-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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,cmd/operator-sdk/version: add go version, os, and arch #1863
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.
/lgtm
Any concerns with quoting the version/commit strings? If anyone is parsing this, it will break them. I'd consider this porcelain though, so I wouldn't call this a breaking change. Thoughts? Also, I'll add another commit for the CHANGELOG. |
/test e2e-aws-helm |
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
/test images |
Description of the change:
Adds the version, OS, and architecture of the Go compiler that was used to build the
operator-sdk
binaryMotivation for the change:
To make it easier to debug problems that may be a result of differences in Go versions used to build and run the SDK.