chore: retag team ownership to sca-scanners [CMPA-724] - #381
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>
Contributor
✅ 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
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.owneris nowsca-scanners;github.com/team-slugissnyk/engines_sca-scanners(GitHub-style slug withengines_prefix, bare owner slug in Backstage).CircleCI: the Security Scans workflow job drops the
open_source-managedcontext and usesengines_sca-scannersinstead (alongsideprodsec-orb-runtime), so Snyk scans still getSNYK_TOKENfrom the new team context.Reviewed by Cursor Bugbot for commit 5994cb5. Bugbot is set up for automated code reviews on this repo. Configure here.