-
Notifications
You must be signed in to change notification settings - Fork 528
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
Added flag for version information output #44
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.
Thx for the PR. could you refer to this PR in EBS CSI driver: kubernetes-sigs/aws-ebs-csi-driver@182211d for the sake of consistency ?
I've added a test and made the implementation more closely match that of the sister-ebs implementation for consistency |
thx for making the change. /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: djcass44, leakingtapan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…ERS-and-OWNERS_ALIASES OCPBUGS-16783: Chore: Update OWNERS
Is this a bug fix or adding new feature?
Fixes: #25
What is this PR about? / Why do we need it?
This PR adds the ability to run the built binary with
--version
which prints some information about the application:What testing is done?
make verify
make test
docker build -t aws-efs-csi-test . && docker run aws-efs-csi-test --version
(to ensure no pollution from my workspace)Assumptions
Since the Makefile contained
GOOS=linux
I didn't add any portability for getting this information on non-Linux platforms.Please let me know if you wish for me to change the names of any of the json fields or the way that the date is formatted.