Skip to content

chore: remove backstage.yaml and internal audit report from public repo (SRE-3790) - #8

Merged
PiyushT-pattern merged 2 commits into
mainfrom
SRE-3790-remove-backstage-yaml
Nov 2, 2026
Merged

PiyushT-pattern merged 2 commits into
mainfrom
SRE-3790-remove-backstage-yaml

Conversation

@PiyushT-pattern

@PiyushT-pattern PiyushT-pattern commented Nov 2, 2026 •

Copy link
Copy Markdown

Why

This repo is public. Two files in it expose internal information to the internet (SRE-3790):

  • backstage.yaml: catalog metadata (owner team, cost center, department, system).
  • docs/engineering-best-practices-audit.md: the internal engineering audit. Audit reports are internal and are not kept in public repos. The goal is to turn its findings into fixes in the repo instead.

What

  • Removes backstage.yaml. The catalog entity moves, unchanged, to patterninc/backstage-catalog-public-repos (catalog/public-repo-components.yaml, backstage-catalog-public-repos#1), so this repo stays in Backstage.
  • Removes docs/engineering-best-practices-audit.md. Nothing else in the repo links to it.

Merge order

This PR is a draft until backstage-catalog-public-repos#1 is merged and this repo's entity shows in Backstage from there. Merging it first would leave this repo's Backstage entry pointing at a missing file.

🤖 Generated with Claude Code

PiyushT-pattern and others added 2 commits November 2, 2026 15:28
The catalog entity now lives in patterninc/backstage-portal
(public-repo-components.yaml), so this public repo no longer exposes
owner, cost center and system metadata.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The engineering best-practices audit is for internal use and is not kept
in public repos.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@PiyushT-pattern PiyushT-pattern changed the title chore: remove backstage.yaml from public repo (SRE-3790) chore: remove backstage.yaml and internal audit report from public repo (SRE-3790) Nov 2, 2026
@PiyushT-pattern
PiyushT-pattern marked this pull request as ready for review November 2, 2026 11:20
@PiyushT-pattern
PiyushT-pattern merged commit 2b49d77 into main Nov 2, 2026
11 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants