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

[CI] Tag docker images with the commit sha #2418

Merged
merged 1 commit into from
Jan 9, 2022
Merged

[CI] Tag docker images with the commit sha #2418

merged 1 commit into from
Jan 9, 2022

Conversation

quentinus95
Copy link
Contributor

We do not release stable tags often. To avoid using latest, for instance in Kubernetes manifests, this change allows to reference a repository commit. This way it is easy to lock image to a specific commit, and try and rollback if a change to a newer commit brings regressions.

We do not release stable tags often. To avoid using `latest`, for instance in Kubernetes manifests, this change allows to reference a repository commit. This way it is easy to lock image to a specific commit, and try and rollback if a change to a newer commit brings regressions.
@quentinus95 quentinus95 changed the title tag docker images with the commit sha [CI] tag docker images with the commit sha Jan 9, 2022
@quentinus95 quentinus95 changed the title [CI] tag docker images with the commit sha [CI] Tag docker images with the commit sha Jan 9, 2022
@em92 em92 merged commit 2f957b6 into RSS-Bridge:master Jan 9, 2022
@em92
Copy link
Contributor

em92 commented Jan 9, 2022

Hi, @quentinus95! Are you deploying RSS-Bridge for your business related tasks in your company?
Asking 'cos kubernetes thing seems to be SaaS specific thing.

@quentinus95 quentinus95 deleted the patch-1 branch January 9, 2022 15:31
@quentinus95
Copy link
Contributor Author

@em92 hello! It is for personal use on an experimental personal cluster. Thanks!

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