Allow fully-qualified resource names to be used with --exclude

semekh · semekh · commit 3ede2013e084 · 2020-02-02T19:06:00.000+03:30
Signed-off-by: Mehran Kholdi &lt;semekh@hamravesh.com&gt;
diff --git a/internal/client/client.go b/internal/client/client.go
@@ -142,6 +142,7 @@ func extractRelevantResources(grs []groupResource, exclusions []string) []groupR
 		name := r.fullName()
 		resourceIds := r.APIResource.ShortNames
 		resourceIds = append(resourceIds, r.APIResource.Name)
+		resourceIds = append(resourceIds, name)
 		if forbidden.HasAny(resourceIds...) {
 			logrus.Debugf("Excluding %s", name)
 			continue
diff --git a/internal/client/client_test.go b/internal/client/client_test.go
@@ -63,6 +63,13 @@ func TestExtractRelevantResourceNames(t *testing.T) {
 			exclude:   []string{"baz"},
 			expected:  []string{"bar.puorg"},
 		},
+		{
+			testName:  "two filtered by fully-qualified resource name",
+			resources: []v1.APIResource{{Name: "foo"}, {Name: "bar"}},
+			groups:    []string{"group", "puorg"},
+			exclude:   []string{"foo.group"},
+			expected:  []string{"bar.puorg"},
+		},
 	}
 
 	for _, test := range tests {
