Skip to content

Comments

Fix multiple tag error on main and release builds#3186

Merged
tklauser merged 1 commit intomainfrom
pr/feroz/multiple-tag-fix
Feb 17, 2026
Merged

Fix multiple tag error on main and release builds#3186
tklauser merged 1 commit intomainfrom
pr/feroz/multiple-tag-fix

Conversation

@ferozsalam
Copy link
Contributor

On main and release builds, the build workflow was assigning multiple tags at the image build time, which was then causing errors when pushing.

This change resolves the issue by using a single temporary tag at build time, and only using the final repo tags on push.

@ferozsalam
Copy link
Contributor Author

Set the trigger to pull_request to test the workflow: https://github.com/cilium/cilium-cli/actions/runs/22020292929/job/63628416766?pr=3186

On main and release builds, the build workflow was assigning multiple tags at
the image build time, which was then causing errors when pushing.

This change resolves the issue by using a single temporary tag at build time,
and only using the final repo tags on push.

Signed-off-by: Feroz Salam <feroz.salam@isovalent.com>
@ferozsalam ferozsalam force-pushed the pr/feroz/multiple-tag-fix branch from 589bb1e to 319b76f Compare February 14, 2026 16:14
@ferozsalam ferozsalam marked this pull request as ready for review February 14, 2026 16:34
@ferozsalam ferozsalam requested review from a team as code owners February 14, 2026 16:34
Copy link
Member

@tklauser tklauser left a comment

Choose a reason for hiding this comment

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

Thanks @ferozsalam!

@tklauser tklauser merged commit d663ac3 into main Feb 17, 2026
7 checks passed
@tklauser tklauser deleted the pr/feroz/multiple-tag-fix branch February 17, 2026 08:48
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.

3 participants