add nodeselectors to helm

[sadath-12]

No description provided.

[grampelberg]

Are you sure you want the node selectors to be the same for everything? It feels like the extractors should be distinct so that they can land on gpu nodes for example.

[jvidalg]

Are you sure you want the node selectors to be the same for everything? It feels like the extractors should be distinct so that they can land on gpu nodes for example.

Hi @grampelberg @sadath-12 based on the values structure, it seems that each component is already independently customizable. Adding nodeSelector: {} to the values file could make this clearer, for example:

api:
  enabled: true
  nodeSelector: {}

coordinator:
  enabled: true
  nodeSelector: {}

extractors:
  - name: extractor1
    nodeSelector: {}

  - name: extractor2
    nodeSelector: {}

[sadath-12]

Are you sure you want the node selectors to be the same for everything? It feels like the extractors should be distinct so that they can land on gpu nodes for example.

Hi @grampelberg @sadath-12 based on the values structure, it seems that each component is already independently customizable. Adding nodeSelector: {} to the values file could make this clearer, for example:

api:
  enabled: true
  nodeSelector: {}

coordinator:
  enabled: true
  nodeSelector: {}

extractors:
  - name: extractor1
    nodeSelector: {}

  - name: extractor2
    nodeSelector: {}

ya but I wanted in values.yaml itself