Skip to content

Add octo-sts trust policy for datadog-api-spec merge-dependents - #659

Merged
MintsInc merged 1 commit into
mainfrom
ulysse.mavrocordatos/AAWF-1384/octo-sts-merge-dependents-policy
Jul 20, 2026
Merged

Add octo-sts trust policy for datadog-api-spec merge-dependents#659
MintsInc merged 1 commit into
mainfrom
ulysse.mavrocordatos/AAWF-1384/octo-sts-merge-dependents-policy

Conversation

@MintsInc

Copy link
Copy Markdown
Member

Context

datadog-api-spec generates pup's CLI commands from the OpenAPI spec and pushes them here as PRs. When a spec PR also regenerates the Rust client (datadog-api-client, pinned in Cargo.toml by commit rev), pup must be repointed at the new Rust commit before/at merge. datadog-api-spec's merge queue now runs a pipeline merge-dependents step (in .github/workflows/merge-queue.yml) that, after the Rust client merges, rewrites pup's Cargo.toml rev and merges pup's generated PR. That step authenticates to pup with an octo-sts token, which requires a trust policy in this repo.

This is the merge-time sibling of the existing .github/chainguard/datadog-api-spec.ci-cd.generate.sts.yaml. That policy trusts the PR-time generation workflow (ci-cd.yml on pull_request); this one trusts the merge-queue workflow (merge-queue.yml on push to the MergeQueue working branch). They must be separate because the event and workflow differ.

Changes

Adds one octo-sts trust policy: .github/chainguard/datadog-api-spec.merge-queue.merge-dependents.sts.yaml, granting contents: write and pull_requests: write to the merge-queue.yml workflow running on mq-working-branch-* refs in DataDog/datadog-api-spec.

Required for datadog-api-spec's pup generation merge step to authenticate and land the repointed Cargo.toml.

@MintsInc
MintsInc marked this pull request as ready for review July 20, 2026 12:08
@MintsInc
MintsInc requested a review from a team as a code owner July 20, 2026 12:08
@MintsInc
MintsInc merged commit fe0397c into main Jul 20, 2026
6 checks passed
@MintsInc
MintsInc deleted the ulysse.mavrocordatos/AAWF-1384/octo-sts-merge-dependents-policy branch July 20, 2026 12:19
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