pixie icon indicating copy to clipboard operation
pixie copied to clipboard

[Helm Chart] pixie helm chart doesn't support excluding fargate from daemonset

Open xdays opened this issue 2 years ago • 2 comments

Is your feature request related to a problem? Please describe.

When deployed to EKS with fargate enabled, the vizier-pem pod is always in Pending state because it is scheduled to fargate node.

k get pod -A | grep Pending
monitoring                   prometheus-prometheus-node-exporter-s6s99                         0/1     Pending     0          28d
newrelic                     vizier-pem-wq9xq                                                  0/1     Pending     0          114m

Describe the solution you'd like

We enable deploying pixie-chart from newrelic-bundle chart, it will be great if pixie-chart can respect the value global.fargate from that chart.

Describe alternatives you've considered

I see pixie operator chart support patches, but I can't get it to work, it fails the validation.

Error: UPGRADE FAILED: error validating "": error validating data: ValidationError(Vizier.spec.patches.spec): invalid type for dev.px.v1alpha1.Vizier.spec.patches: got "map", expected "string"

This is the patches value I used.

# one line version for helm
{"spec": {"template": {"spec": {"affinity": {"nodeAffinity": {"requiredDuringSchedulingIgnoredDuringExecution": {"nodeSelectorTerms": [ {"matchExpressions": [ {"key": "eks.amazonaws.com/compute-type", "operator": "NotIn", "values": [ "fargate" ]}]}]}}}}}}}

# prettified json
{
  "spec": {
    "template": {
      "spec": {
        "affinity": {
          "nodeAffinity": {
            "requiredDuringSchedulingIgnoredDuringExecution": {
              "nodeSelectorTerms": [
                {
                  "matchExpressions": [
                    {
                      "key": "eks.amazonaws.com/compute-type",
                      "operator": "NotIn",
                      "values": [
                        "fargate"
                      ]
                    }
                  ]
                }
              ]
            }
          }
        }
      }
    }
  }
}

Additional context Add any other context or screenshots about the feature request here.

Related issues:

  • https://github.com/newrelic/helm-charts/issues/945
  • https://github.com/pixie-io/pixie/issues/263

xdays avatar Jun 03 '23 11:06 xdays

Has anyone got this to work? This chart needs to support affinity to be passed to it.

cdenneen avatar Jan 30 '24 22:01 cdenneen