Skip to content

ci: Add support for OIDC - #440

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

aanm 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 name is read from the QUAY_ROBOT_RELEASE variable,
defined on the GitHub environment the job pins 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 the environment
breaks the login.

@aanm
aanm force-pushed the pr/aanm/quay-oidc-robot-variables branch from 82b8a5a to 391e8d6 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 name is read from the QUAY_ROBOT_RELEASE variable,
defined on the GitHub environment the job pins 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 the 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 391e8d6 to bcc6797 Compare September 25, 2026 14:47
@aanm
aanm marked this pull request as ready for review September 28, 2026 14:13
@aanm
aanm requested a review from joestringer September 29, 2026 08:20
Comment thread .github/workflows/build-image.yaml
@aanm
aanm requested a review from joestringer September 30, 2026 08:29
@aanm
aanm merged commit 14786e3 into main Oct 16, 2026
1 check passed
@aanm
aanm deleted the pr/aanm/quay-oidc-robot-variables branch October 16, 2026 09:10
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