seldon-core icon indicating copy to clipboard operation
seldon-core copied to clipboard

Seldon v2: Helm chart missing variables and a protocol: TCP

Open FredericJames opened this issue 2 years ago • 5 comments

Describe the bug

In the Helm chart, I have no possibility to add my labels/annotations/nodeSelector/tolerations/affinity/anti-affinity ...

Furthermore, there's protocol: TCP missing line 1661 in file seldon-v2-components.yaml Argocd is raising an error, preventing a good sync of the apps image

To reproduce

Deploy seldon v2 setup with argocd. try to add labels/annotations ... in the value file

Expected behaviour

Be able to add my own labels/annotations/... to any seldon resources Seldon helm chart running with argocd without issues

Environment

  • Cloud Provider: AWS EKS
  • Kubernetes Cluster Version: 1.24
  • Deployed Seldon System Images: 2.5.0

FredericJames avatar Jul 04 '23 20:07 FredericJames

@FredericJames could you provide example of how you see extending the Helm charts without ArgoCD in the picture?

RafalSkolasinski avatar Jul 04 '23 20:07 RafalSkolasinski

We're using argocd and won't be able to do otherwise. Right now, the missing field protocol is preventing me to use argocd properly. So I have to kustomize this helm chart to patch the manifest

FredericJames avatar Jul 05 '23 12:07 FredericJames

This seems strange as TCP is the default. This is also defined with a default inside the CRD also. Maybe ArgoCD does not handle CRD defaults?

ukclivecox avatar Jul 14 '23 13:07 ukclivecox

Maybe related to this ArgoCD issue?

ukclivecox avatar Jul 14 '23 13:07 ukclivecox

+1 for tolerations, nodeSelectors, et al. in the Helm chart. We're using Seldon Core V2, and would like to be able to deploy everything to its own node pool without the need for patching the yaml after the Helm deploy.

peterezetta avatar Jul 10 '24 19:07 peterezetta

closing this issue because the fix is in #5771

Rajakavitha1 avatar Sep 25 '24 14:09 Rajakavitha1