Skip to content

ci: use Docker Hub OIDC for test jobs - #273

Merged
crazy-max merged 1 commit into
mainfrom
ci-oidc
Aug 6, 2026
Merged

ci: use Docker Hub OIDC for test jobs#273
crazy-max merged 1 commit into
mainfrom
ci-oidc

Conversation

@crazy-max

Copy link
Copy Markdown
Member

No description provided.

@crazy-max
crazy-max requested a review from a team as a code owner August 6, 2026 08:35
@crazy-max
crazy-max requested a review from a team August 6, 2026 08:40
- bake-dockerhub-single
permissions:
contents: read
id-token: write

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we want the comment here as well (I think we did on other repositories?)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this workflow is just for testing and not production so no need imo

sbom: true
set: |
*.args.VERSION={{meta.version}}
target: hello

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What was this block for?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GitHub diff is bad here. It's a duplicate where we had both OIDC auth and PAT auth so I removed the PAT auth basically.

- bake-ghcr-and-dockerhub
permissions:
contents: read
id-token: write

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here (wrt comments)

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, left some minor question and comment

@crazy-max
crazy-max merged commit cf06f13 into main Aug 6, 2026
454 checks passed
@crazy-max
crazy-max deleted the ci-oidc branch August 6, 2026 11:59
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