Skip to content

chore: retag team ownership to sca-scanners [CMPA-724] - #381

Open
gergo-papp wants to merge 1 commit into
mainfrom
chore/migrate-team-tag-sca-scanners
Open

chore: retag team ownership to sca-scanners [CMPA-724]#381
gergo-papp wants to merge 1 commit into
mainfrom
chore/migrate-team-tag-sca-scanners

Conversation

@gergo-papp

@gergo-papp gergo-papp commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

What?

catalog-info.yaml ownership metadata moves to the merged team:

Field Before After
owner retired team name sca-scanners
github.com/team-slug snyk/<retired> snyk/engines_sca-scanners

The two conventions differ on purpose: only GitHub and CircleCI carry the engines_ group
prefix, so Backstage owner is the bare slug. Matches siblings such as sca-findings
(owner: os-flows) and error-catalog (owner: cloud-experience).

CircleCI context

Also moves the CircleCI context from the open_source-managed stop-gap to the new
engines_sca-scanners context (6070a8d4-dd66-4341-ae23-49571225b6da), created by @wayne-grant
today — see the #ask-prodsec thread.
Per that discussion the plan is to go straight to the correct team rather than leave the stop-gap in place,
since these repos are being touched anyway.

Note contexts do carry the engines_ prefix, unlike the Polaris/Backstage owner values above.

Why this is safe here: engines_sca-scanners currently provides only SNYK_TOKEN, whereas
open_source-managed also holds a CIRCLECI_TOKEN. Every job in this repo that requests the
context is a Snyk scanning job needing SNYK_TOKEN only — this repo references no Circle PAT at
all, so nothing loses a variable.

Ownership is set in more places than expected

Rather than naming individual files, the transform walks all of helm/ plus
catalog-info.yaml, matching on key name + retired value (never on parent path, which varies
per repo):

Location Keys
catalog-info.yaml owner, github.com/team-slug
helm/Chart.yaml team
helm/values.yaml owner, snykowner — under polaris-datadog-alerting (the monitor tag), polaris-namespace, namespace, global, polarisPostgres, polarisRedis*, bucket blocks
helm/values/<env>.yaml per-environment overrides
helm/templates/** incl. _helpers.tpl and subcharts snykowner, snyk.io/owner — these become labels and annotations on the running pods

Unrelated owner: keys are left alone — matching on the retired value means e.g. a secret owner
under npmDepsAuthnToken.consumer is untouched.

Pruning stale catalog-info.yaml entries

Removed rather than updated, since each is either superseded or points somewhere that no longer
resolves:

Entry Why removed
snyk.io/jira-prefix Names a Jira project, which is not renamed when a GitHub team is. It was being updated on the wrong trigger, and it is not the annotation the Backstage Jira plugin reads (jira/project-key).
Ask-channel link Superseded by the ## Contact section added to the README earlier in this migration.
Notion links Notion is no longer used.
Datadog links naming a retired team e.g. an SLO query team:unify AND service:… — stops resolving once the team tag changes, so the link would silently go stale.
pagerduty.com/service-id Unused.

Where pruning empties the links: block, the key itself is removed rather than left with no
entries. Datadog links that do not reference a retired team are left alone.

Every one of these is checked per repo and reported, so a repo that simply does not set them is
distinguishable from one that was skipped.

Why?

open-source_unify, open-source_analysis-platform, os-managed and os-ecosystems have
merged into @snyk/engines_sca-scanners (CODEOWNERS already migrated under CMPA-724).

Risk

None to alerting. This repo has no Polaris or Datadog configuration, so nothing about monitor
tagging or paging changes. Safe to merge at any time.

Not touched: Slack channels/handles/mentions, the UNIFY Jira prefix, dashboard/SLO URLs, and
name: / project-slug: fields — some repo names legitimately contain a retired team name.


Note

Low Risk
Metadata and CI context only; no code or runtime change.


Note

Low Risk
Metadata and CI context only; no application code or runtime behavior changes.

Overview
Reassigns snyk-python-plugin from the retired os-managed / open_source-managed setup to engines_sca-scanners / sca-scanners after the team merge.

Backstage (catalog-info.yaml): spec.owner is now sca-scanners; github.com/team-slug is snyk/engines_sca-scanners (GitHub-style slug with engines_ prefix, bare owner slug in Backstage).

CircleCI: the Security Scans workflow job drops the open_source-managed context and uses engines_sca-scanners instead (alongside prodsec-orb-runtime), so Snyk scans still get SNYK_TOKEN from the new team context.

Reviewed by Cursor Bugbot for commit 5994cb5. Bugbot is set up for automated code reviews on this repo. Configure here.

The teams behind the retired 'unify', 'os-managed', 'os-ecosystems' and
'open-source_analysis-platform' names have merged into @snyk/engines_sca-scanners.

Covers ownership wherever it is set: catalog-info.yaml, helm/Chart.yaml,
helm/values.yaml, the per-environment helm/values/ overrides, and helm/templates
(snykowner / snyk.io/owner, which land as labels and annotations on the pods).

Also moves the CircleCI context from the 'open_source-managed' stop-gap to
'engines_sca-scanners', and prunes stale catalog-info entries where present
(snyk.io/jira-prefix, ask-channel links, Notion links, Datadog links naming a
retired team, pagerduty.com/service-id).

Package metadata only — no Polaris/Datadog config is touched, so alert routing is
unaffected and this is safe to merge at any time.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@gergo-papp
gergo-papp requested a review from a team as a code owner September 1, 2026 19:55
@snyk-io

snyk-io Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

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.

1 participant