Skip to content

ci: Add support for OIDC - #3313

Merged
michi-covalent merged 1 commit into
mainfrom
pr/aanm/quay-oidc-robot-variables
Oct 6, 2026
Merged

michi-covalent merged 1 commit into
mainfrom
pr/aanm/quay-oidc-robot-variables

Conversation

@aanm

@aanm aanm commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

This commit adds support for OIDC, replacing long-lived tokens. Quay.io
has been configured accordingly.

The robot account names are read from the QUAY_ROBOT_CI and
QUAY_ROBOT_RELEASE variables, defined on the GitHub environments the
jobs pin rather than at repository or organisation level, so the value a
job receives is the one belonging to its environment. The environment
name also forms the OIDC subject that the robot is configured to trust,
so renaming an environment breaks the login.

@aanm
aanm marked this pull request as ready for review September 25, 2026 13:00
@aanm
aanm requested review from a team as code owners September 25, 2026 13:00
@aanm
aanm requested review from nbusseneau and tklauser September 25, 2026 13:00
@aanm
aanm marked this pull request as draft September 25, 2026 13:15
@aanm
aanm force-pushed the pr/aanm/quay-oidc-robot-variables branch from 8a7bf9a to 747bc15 Compare September 25, 2026 13:29
This commit adds support for OIDC, replacing long-lived tokens. Quay.io
has been configured accordingly.

The robot account names are read from the QUAY_ROBOT_CI and
QUAY_ROBOT_RELEASE variables, defined on the GitHub environments the
jobs pin rather than at repository or organisation level, so the value a
job receives is the one belonging to its environment. The environment
name also forms the OIDC subject that the robot is configured to trust,
so renaming an environment breaks the login.

Signed-off-by: André Martins <andre@cilium.io>
@aanm
aanm force-pushed the pr/aanm/quay-oidc-robot-variables branch from 747bc15 to ac0a15c Compare September 25, 2026 14:47
@aanm

aanm commented Sep 28, 2026

Copy link
Copy Markdown
Member Author

Tested with a temporary DO NOT MERGE commit that switches images-dev.yaml to pull_request: https://github.com/cilium/cilium-cli/actions/runs/36422913345. The run got the robot token through OIDC and pushed quay.io/cilium/cilium-cli-ci:abaed675af4fdce2b4b25a93cacdf410c4badc2f@sha256:0f48a5c6b838aaa59133a1eececccb90153a3250e7ad2f033e3a0c07cf833622.

@aanm
aanm force-pushed the pr/aanm/quay-oidc-robot-variables branch from abaed67 to ac0a15c Compare September 28, 2026 12:43
@aanm
aanm marked this pull request as ready for review September 28, 2026 14:12
@michi-covalent
michi-covalent merged commit 6bf00cf into main Oct 6, 2026
14 of 15 checks passed
@michi-covalent
michi-covalent deleted the pr/aanm/quay-oidc-robot-variables branch October 6, 2026 20:32
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.

4 participants