Add workaround for k8s bug concerning componentstatus and selectors

Cornelius Weig · Cornelius Weig · commit 44bc720ff807 · 2019-04-13T01:15:59.000+02:00
diff --git a/pkg/ketall/client/client.go b/pkg/ketall/client/client.go
@@ -52,7 +52,7 @@ func GetAllServerResources(flags *genericclioptions.ConfigFlags) (runtime.Object
 		return nil, errors.Wrap(err, "fetch available group resources")
 	}
 
-	resources := extractRelevantResources(grs, viper.GetStringSlice(constants.FlagExclude))
+	resources := extractRelevantResources(grs, getExclusions())
 
 	start := time.Now()
 	response, err := fetchResourcesBulk(flags, resources...)
@@ -64,6 +64,17 @@ func GetAllServerResources(flags *genericclioptions.ConfigFlags) (runtime.Object
 	return fetchResourcesIncremental(flags, resources...)
 }
 
+func getExclusions() []string {
+	exclusions := viper.GetStringSlice(constants.FlagExclude)
+
+	// This is a workaround for a k8s bug where componentstatus is reported even though the label selector does not apply
+	if selector := viper.GetString(constants.FlagSelector); selector != "" {
+		exclusions = append(exclusions, "componentstatuses")
+	}
+
+	return exclusions
+}
+
 func fetchAvailableGroupResources(cache bool, scope string, flags *genericclioptions.ConfigFlags) ([]groupResource, error) {
 	client, err := flags.ToDiscoveryClient()
 	if err != nil {
