-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Makefile,version,cmd/operator-sdk/version: add git compile-time variables #1431
Makefile,version,cmd/operator-sdk/version: add git compile-time variables #1431
Conversation
… for git version and commit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
on green tests
New changes are detected. LGTM label has been removed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Thanks for this, it will be a lot easier indeed to debug things.
lgtm
…bles (operator-framework#1431) * Makefile,version,cmd/operator-sdk/version: add compile-time variables for git version and commit * release.sh: new format for Version in version.go
…bles (operator-framework#1431) * Makefile,version,cmd/operator-sdk/version: add compile-time variables for git version and commit * release.sh: new format for Version in version.go
Description of the change:
version.GitVersion
andversion.GitCommit
operator-sdk version
command to print themMotivation for the change:
Many users seem to be using non-release versions of
operator-sdk
, so lots of issues typically only report the version as something likev0.7.0+git
.To make it easier to troubleshoot issues, it would be nice to have the
operator-sdk version
command print the git version and commit strings.