From 3450a048bc3f8a05a3833e4b8d8eba360a9693db Mon Sep 17 00:00:00 2001 From: Martin Schimandl Date: Thu, 1 Oct 2020 12:40:32 +0200 Subject: [PATCH] Fix staticchecks in vendor/k8s.io/client-go Kubernetes-commit: 13c017056c924e148113e42786a511ed2fee7594 --- dynamic/fake/simple.go | 6 ++++-- metadata/fake/simple.go | 6 ++++-- restmapper/category_expansion.go | 2 +- restmapper/shortcut.go | 2 +- tools/leaderelection/leaderelection.go | 3 --- 5 files changed, 10 insertions(+), 9 deletions(-) diff --git a/dynamic/fake/simple.go b/dynamic/fake/simple.go index b2c5f6f3..ec204255 100644 --- a/dynamic/fake/simple.go +++ b/dynamic/fake/simple.go @@ -96,7 +96,8 @@ func (c *dynamicResourceClient) Create(ctx context.Context, obj *unstructured.Un Invokes(testing.NewRootCreateAction(c.resource, obj), obj) case len(c.namespace) == 0 && len(subresources) > 0: - accessor, err := meta.Accessor(obj) + var accessor metav1.Object // avoid shadowing err + accessor, err = meta.Accessor(obj) if err != nil { return nil, err } @@ -109,7 +110,8 @@ func (c *dynamicResourceClient) Create(ctx context.Context, obj *unstructured.Un Invokes(testing.NewCreateAction(c.resource, c.namespace, obj), obj) case len(c.namespace) > 0 && len(subresources) > 0: - accessor, err := meta.Accessor(obj) + var accessor metav1.Object // avoid shadowing err + accessor, err = meta.Accessor(obj) if err != nil { return nil, err } diff --git a/metadata/fake/simple.go b/metadata/fake/simple.go index f5f3f946..adf0ce9a 100644 --- a/metadata/fake/simple.go +++ b/metadata/fake/simple.go @@ -114,7 +114,8 @@ func (c *metadataResourceClient) CreateFake(obj *metav1.PartialObjectMetadata, o Invokes(testing.NewRootCreateAction(c.resource, obj), obj) case len(c.namespace) == 0 && len(subresources) > 0: - accessor, err := meta.Accessor(obj) + var accessor metav1.Object // avoid shadowing err + accessor, err = meta.Accessor(obj) if err != nil { return nil, err } @@ -127,7 +128,8 @@ func (c *metadataResourceClient) CreateFake(obj *metav1.PartialObjectMetadata, o Invokes(testing.NewCreateAction(c.resource, c.namespace, obj), obj) case len(c.namespace) > 0 && len(subresources) > 0: - accessor, err := meta.Accessor(obj) + var accessor metav1.Object // avoid shadowing err + accessor, err = meta.Accessor(obj) if err != nil { return nil, err } diff --git a/restmapper/category_expansion.go b/restmapper/category_expansion.go index 2537a2b4..484e4c83 100644 --- a/restmapper/category_expansion.go +++ b/restmapper/category_expansion.go @@ -58,7 +58,7 @@ func NewDiscoveryCategoryExpander(client discovery.DiscoveryInterface) CategoryE // Expand fulfills CategoryExpander func (e discoveryCategoryExpander) Expand(category string) ([]schema.GroupResource, bool) { // Get all supported resources for groups and versions from server, if no resource found, fallback anyway. - apiResourceLists, _ := e.discoveryClient.ServerResources() + _, apiResourceLists, _ := e.discoveryClient.ServerGroupsAndResources() if len(apiResourceLists) == 0 { return nil, false } diff --git a/restmapper/shortcut.go b/restmapper/shortcut.go index 6903ec80..73b317c1 100644 --- a/restmapper/shortcut.go +++ b/restmapper/shortcut.go @@ -84,7 +84,7 @@ func (e shortcutExpander) getShortcutMappings() ([]*metav1.APIResourceList, []re res := []resourceShortcuts{} // get server resources // This can return an error *and* the results it was able to find. We don't need to fail on the error. - apiResList, err := e.discoveryClient.ServerResources() + _, apiResList, err := e.discoveryClient.ServerGroupsAndResources() if err != nil { klog.V(1).Infof("Error loading discovery information: %v", err) } diff --git a/tools/leaderelection/leaderelection.go b/tools/leaderelection/leaderelection.go index d1959fd7..e44fb473 100644 --- a/tools/leaderelection/leaderelection.go +++ b/tools/leaderelection/leaderelection.go @@ -188,9 +188,6 @@ type LeaderElector struct { clock clock.Clock metrics leaderMetricsAdapter - - // name is the name of the resource lock for debugging - name string } // Run starts the leader election loop. Run will not return