-
Notifications
You must be signed in to change notification settings - Fork 994
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
version related changes #144
version related changes #144
Conversation
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.
Please update the makefile to make command make images
to work as well?
pkg/version/version.go
Outdated
) | ||
|
||
var ( | ||
// Version shows the version of kube batch. |
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.
volcano
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.
changed
pkg/version/version.go
Outdated
var ( | ||
// Version shows the version of kube batch. | ||
Version = "Not provided." | ||
// GitSHA shoows the git commit id of kube batch. |
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.
ditto
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.
changed
pkg/version/version.go
Outdated
|
||
// PrintVersionAndExit prints versions from the array returned by Info() and exit | ||
func PrintVersionAndExit() { | ||
apiVersion := "v1alpha1" |
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.
Should be moved into a seperated section outside of the function.
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.
moved
do you want me to change the image tag? |
version related flags are not being passed when execute |
@TommyLike , seems randomly some e2e test cases are failing. as this PR doesn't have any relation with the tcs but still tcs r failed. can we trigger travis once again. |
/lgtm |
/lgtm |
version related changes
added version in vkctl, vk-admission & vk-controller
For #77
sample output
root@root1-HP-EliteBook-840-G2:/home/root1/SriniCH/workspace/src/volcano.sh/volcano# ./_output/bin/vk-admission --version
API Version: v1alpha1
Version: v0.1
Git SHA: e08f588
Built At: 2019-05-08 13:57:25
Go Version: go1.11.5
Go OS/Arch: linux/amd64
root@root1-HP-EliteBook-840-G2:/home/root1/SriniCH/workspace/src/volcano.sh/volcano# ./_output/bin/vk-controllers --version
API Version: v1alpha1
Version: v0.1
Git SHA: e08f588
Built At: 2019-05-08 13:57:22
Go Version: go1.11.5
Go OS/Arch: linux/amd64
root@root1-HP-EliteBook-840-G2:/home/root1/SriniCH/workspace/src/volcano.sh/volcano# ./_output/bin/vkctl version
API Version: v1alpha1
Version: v0.1
Git SHA: e08f588
Built At: 2019-05-08 13:57:28
Go Version: go1.11.5
Go OS/Arch: linux/amd64