NetApp / trident

Storage orchestrator for containers
Apache License 2.0
761 stars 223 forks source link

Helm Trident Operator - "tridentNodePluginTolerations" will not be used for Windows Daemonset #828

Closed burkhat closed 10 months ago

burkhat commented 1 year ago

Describe the bug We're using the latest helm version 23.4.0 to rollout the trident operator with windows support. In Openshift it is necessary to set tolerations in the Windows DaemonSet that Pods gets started.

We tried following config in the values.yaml:

tridentNodePluginTolerations:
  - operator: Exists
    effect: NoExecute
  - operator: Exists
    effect: NoSchedule

But the toleration will only be set in the Linux DaemonSet and not in the Windows DaemonSet. Workaround: Deploy the DaemonSet with the tolerations manually

Environment Provide accurate information about the environment to help us reproduce the issue.

To Reproduce Steps to reproduce the behavior: See Bug desscription

Expected behavior Tolerations will be set in the Windows DaemonSet, too

Additional context Add any other context about the problem here.

travisghansen commented 1 year ago

I also need this one. It may make sense to have a different set of tolerations per OS though..

travisghansen commented 1 year ago

Can we get an update on this?

travisghansen commented 1 year ago

Can we get this looked at before the 23.10 release?

mravi-na commented 10 months ago

This issue is fixed with https://github.com/NetApp/trident/commit/5efa9949b0863566c17014a41723088b14e32b99 and is included in 23.10