fix #314

cmd/version.go

@@ -1,3 +1,6 @@
+// SPDX-License-Identifier: Apache-2.0
+// Copyright Authors of Popeye
+
 package cmd
 
 import (

internal/alias.go

@@ -16,6 +16,9 @@ import (
 
 var (
 	ClusterGVR = types.NewGVR("cluster")
+	groups     = map[R]string{
+		NP: "networking.k8s.io",
+	}
 )
 
 // ResourceMetas represents a collection of resource metadata.
@@ -165,6 +168,10 @@ func (a *Aliases) loadPreferred(c types.Connection) error {
 			continue
 		}
 		for _, r := range l.APIResources {
+			if g, ok := groups[R(r.Name)]; ok && g != gv.Group {
+				continue
+			}
+
 			gvr := types.NewGVRFromAPIRes(gv, r)
 			if !a.cilium && strings.Contains(gvr.G(), "cilium.io") {
 				a.cilium = true

internal/report/writer.go

@@ -99,7 +99,7 @@ func (s *ScanReport) Error(msg string, err error) {
 
 // Comment writes a comment line.
 func (s *ScanReport) Comment(msg string) {
-	fmt.Fprintf(s, "  · "+msg+"\n")
+	fmt.Fprint(s, "  · "+msg+"\n")
 }
 
 // Dump all errors to output.

internal/report/writer_test.go

@@ -5,6 +5,7 @@ package report
 
 import (
 	"bytes"
+	"errors"
 	"fmt"
 	"io"
 	"strings"
@@ -31,12 +32,12 @@ func TestError(t *testing.T) {
 		e   string
 	}{
 		{
-			fmt.Errorf("crapola"),
-			"\n💥 \x1b[38;5;196mblee: crapola\x1b[0m\n",
+			err: fmt.Errorf("crapola"),
+			e:   "\n💥 \x1b[38;5;196mblee: crapola\x1b[0m\n",
 		},
 		{
-			fmt.Errorf(strings.Repeat("#", 200)),
-			"\n💥 \x1b[38;5;196mblee: " + strings.Repeat("#", Width-9) + "\x1b[0m\n\x1b[38;5;196m" + strings.Repeat("#", Width-3) + "\x1b[0m\n\x1b[38;5;196m" + strings.Repeat("#", Width-88) + "\x1b[0m\n",
+			err: errors.New(strings.Repeat("#", 200)),
+			e:   "\n💥 \x1b[38;5;196mblee: " + strings.Repeat("#", Width-9) + "\x1b[0m\n\x1b[38;5;196m" + strings.Repeat("#", Width-3) + "\x1b[0m\n\x1b[38;5;196m" + strings.Repeat("#", Width-88) + "\x1b[0m\n",
 		},
 	}
 

internal/stringset_test.go

@@ -1,3 +1,6 @@
+// SPDX-License-Identifier: Apache-2.0
+// Copyright Authors of Popeye
+
 package internal_test
 
 import (

main.go

@@ -1,3 +1,6 @@
+// SPDX-License-Identifier: Apache-2.0
+// Copyright Authors of Popeye
+
 package main
 
 import (

pkg/popeye.go

@@ -263,10 +263,9 @@ func (p *Popeye) lint() (int, int, error) {
 		if gvr == internal.Glossary[internal.NO] && p.client().ActiveNamespace() != client.AllNamespaces {
 			continue
 		}
-		if !p.aliases.IsNamespaced(gvr) {
-			ctx = context.WithValue(ctx, internal.KeyNamespace, client.ClusterScope)
-		}
+
 		runners[gvr] = fn(ctx, cache, codes)
+
 	}
 
 	total, errCount := len(runners), 0
@@ -305,7 +304,12 @@ func (p *Popeye) runLinter(ctx context.Context, gvr types.GVR, l scrub.Linter, c
 		}
 	}()
 
-	if err := l.Lint(ctx); err != nil {
+	callCtx := ctx
+	if !p.aliases.IsNamespaced(gvr) {
+		callCtx = context.WithValue(ctx, internal.KeyNamespace, client.ClusterScope)
+	}
+
+	if err := l.Lint(callCtx); err != nil {
 		p.builder.AddError(err)
 	}
 	o := l.Outcome().Filter(rules.Level(p.config.LintLevel))