cassandra.nodeSelector ignored about helm-charts

[naseemkullah]

Hi @schmurfy so this is due to the way the dependency cassandra chart works, it has a somewhat custom approach to the values nesting nodeSelector one level deeper: https://github.com/helm/charts/blob/39777352ebc9085cd0d4ed928d4b173f26244c7f/incubator/cassandra/values.yaml#L105-L109

[schmurfy]

Thanks, I read what you linked but I am not sure what I need to write, is it this ?

collector:
  nodeSelector: &selector
    pool: jaeger

query:
  nodeSelector: *selector

selector:
  nodeSelector:
    cassandra: *selector

Edit: I tested this and a few variant and I am sorry but I still don't get what I am supposed to write.

[naseemkullah]

Try

cassandra:
  selector:
    nodeSelector: *selector

To get what you are supposed to write you would need to look at the chart being configured's quirks: https://github.com/helm/charts/blob/39777352ebc9085cd0d4ed928d4b173f26244c7f/incubator/cassandra/templates/statefulset.yaml#L37-L39

[schmurfy]

Thanks but it still does not work.

I tried this:

collector:
  nodeSelector: &selector
    pool: jaeger

query:
  nodeSelector: *selector

cassandra:
  selector:
    nodeselector: *selector

I also tried the following which is in the cassandra chart readme without more luck:

nodes:
  enabled: true
  selector:
    nodeSelector: *selector

and:

cassandra:
  nodes:
    enabled: true
    selector:
      nodeSelector: *selector

I really tried to understand what I was supposed to write by looking at the templates but this is totally cryptic to me :/
query & collector get their node selector correctly set but cassandra never got one...

[schmurfy]

after even more attempts from a colleage after I gave up he finally found the working version:

cassandra:
  selector:
    nodeSelector:
      pool: jaeger

The more I use helm and the more amazed I am this thing even exists, it feels like a bag of junk stitched together...

[naseemkullah]

Thanks but it still does not work.

I tried this:

collector:
  nodeSelector: &selector
    pool: jaeger

query:
  nodeSelector: *selector

cassandra:
  selector:
    nodeselector: *selector

I also tried the following which is in the cassandra chart readme without more luck:

nodes:
  enabled: true
  selector:
    nodeSelector: *selector

and:

cassandra:
  nodes:
    enabled: true
    selector:
      nodeSelector: *selector

I really tried to understand what I was supposed to write by looking at the templates but this is totally cryptic to me :/
query & collector get their node selector correctly set but cassandra never got one...

😏

Sorry the reason that did not work is because i had a typo in nodeSelector (S was not capitalized) I fixed the original suggestion. I think you can still use anchor/alias syntax.

Helm is pretty good imho, but perhaps you would be more interested in deploying with the operator (which itself could be deployed by helm if you wanted):

https://github.com/jaegertracing/jaeger-operator