Skip to content

chore(ci): add id-token permission to system-tests subworkflow#209

Merged
nccatoni merged 7 commits into
mainfrom
nccatoni/system-tests-dd-sts
Apr 13, 2026
Merged

chore(ci): add id-token permission to system-tests subworkflow#209
nccatoni merged 7 commits into
mainfrom
nccatoni/system-tests-dd-sts

Conversation

@nccatoni

@nccatoni nccatoni commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Migrates system-tests CI to use dd-sts for Datadog Test Optimization instead of long-lived API keys.

All repositories now share a single system-tests policy (see dd-source#408172) — no per-repo policy is needed.

Depends on DataDog/system-tests#6726.

Changes

  • Add id-token: write permission to the parametric job (which calls the run-parametric reusable workflow)
  • Pin system-tests to 1e5d6b709 (current main, pre-migration) to allow a controlled rollout: repos stay on the pre-migration workflow until their pin is explicitly updated to the post-merge SHA

How to review

The only functional changes are adding id-token: write and enabling push_to_test_optimization. Both have no effect at the pinned SHA (dd-sts is not yet used there) but will be activated once the pin is updated after DataDog/system-tests#6726 merges.

@nccatoni
nccatoni marked this pull request as ready for review April 13, 2026 13:29
@nccatoni
nccatoni requested a review from a team as a code owner April 13, 2026 13:29
@nccatoni
nccatoni merged commit 9614409 into main Apr 13, 2026
27 checks passed
@nccatoni
nccatoni deleted the nccatoni/system-tests-dd-sts branch April 13, 2026 15:46
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