Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add additional latest tag to images during release #888

Merged
merged 2 commits into from
Oct 9, 2024

Conversation

anveshreddy18
Copy link
Contributor

@anveshreddy18 anveshreddy18 commented Oct 8, 2024

How to categorize this PR?

/area delivery
/kind enhancement

What this PR does / why we need it:

Add latest tag to every image built as part of the release job in the CI pipeline.

This is to consume the helm charts as it is, currently it expects a latest tag for the etcd-druid image which it cannot find from the europe-docker registry as we currently only have version specific tags. Adding the latest tag would make it better for open-source consumers to quickly test the functionality, as we have seen cases of people reporting the missing tag.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

add additional `latest` tag to released images.

@anveshreddy18 anveshreddy18 requested a review from a team as a code owner October 8, 2024 11:16
@gardener-robot gardener-robot added needs/review Needs review area/delivery Delivery related kind/enhancement Enhancement, improvement, extension size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) labels Oct 8, 2024
@gardener-robot-ci-3 gardener-robot-ci-3 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Oct 8, 2024
@gardener-robot-ci-1 gardener-robot-ci-1 added reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) and removed reviewed/ok-to-test Has approval for testing (check PR in detail before setting this label because PR is run on CI/CD) labels Oct 8, 2024
@anveshreddy18
Copy link
Contributor Author

/test pull-etcd-druid-e2e-kind

Copy link
Member

@renormalize renormalize left a comment

Choose a reason for hiding this comment

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

Thanks for this feature which will be very useful @anveshreddy18!
Will save us a lot of pain during testing.

It'd be great if you could reflect these changes on etcd-backup-restore as well!

@anveshreddy18
Copy link
Contributor Author

It'd be great if you could reflect these changes on etcd-backup-restore as well!

Sure @renormalize, backup-restore helm charts need to be updated, so will do it then if that's okay.

@anveshreddy18 anveshreddy18 merged commit 1007780 into gardener:master Oct 9, 2024
11 checks passed
@gardener-robot gardener-robot added the status/closed Issue is closed (either delivered or triaged) label Oct 9, 2024
shreyas-s-rao pushed a commit that referenced this pull request Oct 23, 2024
* Add latest tag to images during release
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/delivery Delivery related kind/enhancement Enhancement, improvement, extension needs/ok-to-test Needs approval for testing (check PR in detail before setting this label because PR is run on CI/CD) needs/review Needs review size/xs Size of pull request is tiny (see gardener-robot robot/bots/size.py) status/closed Issue is closed (either delivered or triaged)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants