upgrading from 8.0.2 to 8.1.0 breaks namespaces sync about kube-mgmt HOT 10 CLOSED

I don't think "simpler" and "helm chart" is something I'd use in the same sentence, but it's good that they are there as an option for those who prefer that :)

from kube-mgmt.

Thanks @bjhaid 👍

from kube-mgmt.

Does it seem like it is trying to do this ? and this is not a supported query for NS

Error from server (BadRequest): Unable to find "/v1, Resource=namespaces" that match label selector "", field selector "metadata.namespace=kube-system": field label not supported: metadata.namespace

from kube-mgmt.

Helo @alex0z1
Please provide values.yaml you are uaing for installing the chart.

from kube-mgmt.

I don't use any helm charts, it is from https://www.openpolicyagent.org/docs/latest/kubernetes-tutorial but with auth enabled

from kube-mgmt.

@anderseknert that k8s guide would be much simpier with the helm chart as an example.

from kube-mgmt.

I just ran into this issue after upgrading to 8.1.0

from kube-mgmt.

I believe this line:

8.0.0...8.1.0#diff-10432976b832c2bf005c1a2b317bff34f985023e0e2e43650c2cfd1fbcb1f7faR139

is the problem it will not work with cluster scoped resources like namespaces.

from kube-mgmt.

This:

diff --git a/pkg/data/generic.go b/pkg/data/generic.go
index 091fbc16..0ca41f7c 100644
--- a/pkg/data/generic.go
+++ b/pkg/data/generic.go
@@ -136,11 +136,15 @@ func (s *GenericSync) setup(ctx context.Context) (cache.Store, workqueue.Delayin
 	store, controller := cache.NewInformer(
 		&cache.ListWatch{
 			ListFunc: func(options metav1.ListOptions) (runtime.Object, error) {
-				options.FieldSelector = ignoreNs
+				if s.ns.Namespaced {
+					options.FieldSelector = ignoreNs
+				}
 				return resource.List(ctx, options)
 			},
 			WatchFunc: func(options metav1.ListOptions) (watch.Interface, error) {
-				options.FieldSelector = ignoreNs
+				if s.ns.Namespaced {
+					options.FieldSelector = ignoreNs
+				}
 				return resource.Watch(ctx, options)
 			},
 		},

should fix the issue, happy to send in a PR

from kube-mgmt.

sent a patch here:

#198

from kube-mgmt.