-
Notifications
You must be signed in to change notification settings - Fork 4.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
Fixes go vet complain in package upup/pkg/fi/cloudup/awstasks #5669
Conversation
This commit fixes the result from `go vet k8s.io/kops/upup/pkg/fi/cloudup/awstasks`: upup/pkg/fi/cloudup/awstasks/iamrole.go:80: Errorf format %q has arg e.Name of wrong type *string upup/pkg/fi/cloudup/awstasks/iamrole.go:88: Errorf format %q has arg e.Name of wrong type *string upup/pkg/fi/cloudup/awstasks/iamrole.go:93: Errorf format %q has arg e.Name of wrong type *string upup/pkg/fi/cloudup/awstasks/iamrole.go:98: Errorf format %q has arg e.Name of wrong type *string upup/pkg/fi/cloudup/awstasks/iamrolepolicy.go:78: Errorf format %q has arg e.Name of wrong type *string upup/pkg/fi/cloudup/awstasks/load_balancer.go:115: Warningf format %q has arg lb.LoadBalancerName of wrong type *string upup/pkg/fi/cloudup/awstasks/load_balancer.go:328: Verbose.Info call has possible formatting directive %+v upup/pkg/fi/cloudup/awstasks/load_balancer.go:382: Verbose.Infof format %q has arg actual.LoadBalancerName of wrong type *string upup/pkg/fi/cloudup/awstasks/natgateway.go:189: Errorf format %q has arg id of wrong type *string upup/pkg/fi/cloudup/awstasks/natgateway.go:193: Verbose.Infof format %q has arg id of wrong type *string upup/pkg/fi/cloudup/awstasks/natgateway.go:197: Errorf format %q has arg id of wrong type *string upup/pkg/fi/cloudup/awstasks/natgateway.go:283: Errorf format %q has arg e.Name of wrong type *string
Hmm govet passes fine on master for me. We can get this testing though. /ok-to-test |
@mikesplain I try on go1.10.3, it doesn't complain anything but still happen on go tip. This is the output:
This commit doesn't fixes on all package that go vet complain. It's only upup/pkg/fi/cloudup/awstasks. But I think it would be good if we starting fixes all of complains before go release 1.11. |
Got it! Cool thanks @wingyplus! /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mikesplain, wingyplus 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 |
No description provided.