-
Notifications
You must be signed in to change notification settings - Fork 39.6k
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
generated code should pass go vet for go1.10 #62412
Conversation
/sig api-machinery |
/ok-to-test |
lgtm when it's green |
/retest |
cc @roycaihw |
Please run
lgtm when tests are green |
/assign |
/retest |
/test pull-kubernetes-bazel-test |
The bazel test failures are valid. I'm working on fixing them now. |
/lgtm |
/assign @deads2k for approval. |
/approve holding to squash all that generated code into one "generated" commit. |
* Make sure sync.RWMutex values aren't copied.
I rebased to reduce the number of commits; all regenerated fakes are now in a single commit. |
/test pull-kubernetes-e2e-gce-device-plugin-gpu |
/approve |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED Approval requirements bypassed by manually added approval. This pull-request has been approved by: bhcleek, deads2k, roycaihw, sttts 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 |
it's just the generated expression in metrics. tagging. |
/test all [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions here. |
What this PR does / why we need it:
Update code generator and the code it generates to pass
go vet
.go1.10 runs
go vet
whenevergo test
is run. Because of this, generated code for CRDs needs to passgo vet
.Release note: