Fix #792: fix findings

apache · Sep 18, 2019 · 4fa3d1a · 4fa3d1a
1 parent 21cc250
commit 4fa3d1a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/pkg/client/client.go b/pkg/client/client.go
@@ -23,10 +23,10 @@ import (
 	"os/user"
 	"path/filepath"
 
-	"k8s.io/apimachinery/pkg/api/meta"
 	"github.com/apache/camel-k/pkg/apis"
 	"github.com/operator-framework/operator-sdk/pkg/k8sutil"
 	"github.com/pkg/errors"
+	"k8s.io/apimachinery/pkg/api/meta"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/schema"
 	"k8s.io/client-go/kubernetes"

diff --git a/pkg/client/fastmapper.go b/pkg/client/fastmapper.go
@@ -40,9 +40,10 @@ func newFastDiscoveryRESTMapperWithFilter(config *rest.Config, filter func(*meta
 	wg := wait.Group{}
 	totalCount := 0
 	pickedCount := 0
-	var grs []*restmapper.APIGroupResources
+	grs := make([]*restmapper.APIGroupResources, 0)
 	for _, group := range groups.Groups {
-		pick := filter(&group)
+		pinnedGroup := group
+		pick := filter(&pinnedGroup)
 		logrus.Debugf("Group: %s %v", group.Name, pick)
 		totalCount++
 		if !pick {