chore: retag team ownership to sca-scanners [CMPA-724] - #27
Open
gergo-papp wants to merge 1 commit into
Open
Conversation
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>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What?
catalog-info.yamlownership metadata moves to the merged team:ownersca-scannersgithub.com/team-slugsnyk/<retired>snyk/engines_sca-scannersThe two conventions differ on purpose: only GitHub and CircleCI carry the
engines_groupprefix, so Backstage
owneris the bare slug. Matches siblings such assca-findings(
owner: os-flows) anderror-catalog(owner: cloud-experience).CircleCI context
Also moves the CircleCI context from the
open_source-managedstop-gap to the newengines_sca-scannerscontext (6070a8d4-dd66-4341-ae23-49571225b6da), created by @wayne-granttoday — 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-scannerscurrently provides onlySNYK_TOKEN, whereasopen_source-managedalso holds aCIRCLECI_TOKEN. Every job in this repo that requests thecontext is a Snyk scanning job needing
SNYK_TOKENonly — this repo references no Circle PAT atall, so nothing loses a variable.
Ownership is set in more places than expected
Rather than naming individual files, the transform walks all of
helm/pluscatalog-info.yaml, matching on key name + retired value (never on parent path, which variesper repo):
catalog-info.yamlowner,github.com/team-slughelm/Chart.yamlteamhelm/values.yamlowner,snykowner— underpolaris-datadog-alerting(the monitor tag),polaris-namespace,namespace,global,polarisPostgres,polarisRedis*, bucket blockshelm/values/<env>.yamlhelm/templates/**incl._helpers.tpland subchartssnykowner,snyk.io/owner— these become labels and annotations on the running podsUnrelated
owner:keys are left alone — matching on the retired value means e.g. a secret ownerunder
npmDepsAuthnToken.consumeris untouched.Pruning stale
catalog-info.yamlentriesRemoved rather than updated, since each is either superseded or points somewhere that no longer
resolves:
snyk.io/jira-prefixjira/project-key).## Contactsection added to the README earlier in this migration.team:unify AND service:…— stops resolving once the team tag changes, so the link would silently go stale.pagerduty.com/service-idWhere pruning empties the
links:block, the key itself is removed rather than left with noentries. 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-managedandos-ecosystemshavemerged 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
UNIFYJira prefix, dashboard/SLO URLs, andname:/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
CI context and Backstage metadata only; no application or runtime behavior changes, and scans still rely on SNYK_TOKEN from the new context.
Overview
Team ownership retag for the merged SCA scanners group: Backstage
ownerincatalog-info.yamlis updated fromengines_sca-scannerstosca-scanners, while the GitHub team annotation remainssnyk/engines_sca-scanners(GitHub/CircleCI keep theengines_prefix; Backstage owner uses the bare slug).The Security Scans workflow job in
.circleci/config.ymlnow uses theengines_sca-scannersCircleCI context instead ofopen_source-managed, aligning secrets with the new team for Snyk scanning jobs.Reviewed by Cursor Bugbot for commit 0d0ccec. Bugbot is set up for automated code reviews on this repo. Configure here.