Shall we use Neuvector 5.3.0?

[e-minguez]

charts/charts/neuvector-core/2.6.6/values.yaml

Line 8 in 48dcba2

tag: 5.2.4

[e-minguez]

Also a diff from the original one https://github.com/neuvector/neuvector-helm/blob/master/charts/core/values.yaml with the one here just bring these changes:

controller:
  replicas: 1
  ranchersso:
    enabled: true
manager:
  enabled: false
cve:
  scanner:
    enabled: false
    replicas: 1
k3s:
  enabled: true
crdwebhook:
  enabled: false

Shall we just minify ours and refer to the original one for more elaborated changes?

[hardys]

Thanks for raising this @e-minguez - my understanding from discussion with @agracey is that we may want to consider removing the suse-edge neuvector chart, as you mention the changes are only to the values, so we can document those instead of maintaining a wrapper chart.

[e-minguez]

I can send a PR to drop the charts here and instead add a README on how to deploy it using the official one. Is that right @hardys @agracey ?