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 date-commit hash as a tag to images #86

Merged
merged 1 commit into from
Feb 27, 2023
Merged

Conversation

89Q12
Copy link

@89Q12 89Q12 commented Feb 22, 2023

Abstract

Enables better versioning and the ability for maintainer of instances to easily rollback a version, as its good practice to version docker images too. Also relates to #55

What does this PR address?

Right now one can only choose:
ghcr.io/pivx-labs/mypivxwallet:latest
or
ghcr.io/pivx-labs/mypivxwallet:master

But what if latest goes bad? There is no way to automatically rollback without manually getting the last known good image digest.

What features or improvements were added?

With this change one can use e.g watchtower to automatically update to the next version tagged e.g
ghcr.io/pivx-labs/mypivxwallet:2023-02-19-223d6a4
If now 2023-02-20-313c5ag goes bad, an automatic rollback can be done.

How does this benefit users?

Minimal down time for selfhosted/external hosted instances and better choosing of features one wants or don't.
It can also be used for easier debugging!

Enables better versioning and the ability for hoster to easily rollback a version
Copy link
Member

@Duddino Duddino left a comment

Choose a reason for hiding this comment

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

ACK

Copy link
Member

@JSKitty JSKitty left a comment

Choose a reason for hiding this comment

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

utACK, looking good, and useful! 🎉

@JSKitty JSKitty requested a review from Liquid369 February 24, 2023 17:08
@JSKitty JSKitty added Enhancement New feature or request Minor (~45 PIV) labels Feb 24, 2023
Copy link
Member

@panleone panleone left a comment

Choose a reason for hiding this comment

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

utACK db472a4

@JSKitty JSKitty merged commit eaae6f3 into PIVX-Labs:master Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
Development

Successfully merging this pull request may close these issues.

4 participants