Skip to content

Add OctoSTS org trust policy for Argo CD on dev-aks - #21

Merged
Nashon-Steffen merged 1 commit into
mainfrom
Nashon-Steffen/octosts-argocd-dev-aks
Nov 4, 2026
Merged

Nashon-Steffen merged 1 commit into
mainfrom
Nashon-Steffen/octosts-argocd-dev-aks

Conversation

@Nashon-Steffen

Copy link
Copy Markdown
Contributor

Adds .github/chainguard/argocd-dev-aks.sts.yaml, an org-level OctoSTS trust policy for Argo CD on the new dev-aks AKS cluster.

dev-aks is a dev cluster in West US 3 that mirrors a customer who deploys with Argo CD. Argo CD must read stacklok/infra, which is internal. A CronJob in the argocd namespace will exchange its projected service account token here every 30 minutes. It writes the token into the Argo CD repo-creds Secret. No GitHub App key is stored. This follows the toolhive-doc-refresh setup (#19).

The policy accepts exactly one caller. That caller must:

  • come from the dev-aks OIDC issuer https://westus3.oic.prod-aks.azure.com/6a88f3cf-e238-4742-95ab-8aafca0b2b87/caa0aab0-2fcb-4088-ad5c-bbe477e9e5a5/. The trailing slash is part of the iss claim, and the issuer's discovery document returns that exact value.
  • have the subject system:serviceaccount:argocd:argocd-repo-token.
  • present a token with audience octo-sts.stacklok.dev. OctoSTS enforces this.

The token carries contents: read on infra only.

Prerequisite: stacklok-octosts (app 4886385) must be installed on stacklok/infra. I could not check the installation list without admin:org.

Infra side:

  • stacklok/infra#6165 (merged, applied) creates the cluster.
  • stacklok/infra#6166 removes the GitHub App placeholder secret.
  • The Argo CD bootstrap PR adds the CronJob.

🤖 Generated with Claude Code

Argo CD on the dev-aks cluster reads stacklok/infra. A CronJob in the
argocd namespace exchanges its projected service account token for a
read-only stacklok-octosts token. No GitHub App key is stored.

The policy accepts the dev-aks OIDC issuer and the service account
argocd:argocd-repo-token. The token carries contents: read on infra.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@Nashon-Steffen
Nashon-Steffen requested a review from a team as a code owner November 4, 2026 20:08
@github-actions

github-actions Bot commented Nov 4, 2026

Copy link
Copy Markdown

Missing Issue Link

This PR does not appear to reference a GitHub issue.
Please add a closing keyword (e.g. Closes #123) if this PR fully resolves the issue, or a non-closing reference (e.g. Part of #123) if it's one of several PRs implementing it.
Cross-repo references are also supported (e.g. Part of owner/repo#123) for issues tracked outside this repository.
Supported closing keywords: closes, fixes, resolves, close, fix, resolve (case insensitive)
Supported non-closing references: relates to, related to, part of, refs, ref, references, reference, contributes to, follow-up to (case insensitive)
Note: closing keywords only auto-close issues in this repository — cross-repo references never auto-close, regardless of keyword.

@Nashon-Steffen
Nashon-Steffen merged commit 4cbcb75 into main Nov 4, 2026
2 checks passed
@Nashon-Steffen
Nashon-Steffen deleted the Nashon-Steffen/octosts-argocd-dev-aks branch November 4, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants