Skip to content

feature(charts): add liveness probe config of csi daemonset in values #5072

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

oihanitrz
Copy link

@oihanitrz oihanitrz commented Jul 4, 2025

Description

This PR introduces the ability to customize the livenessProbe for the server container of the dynatrace-oneagent-csi-driver DaemonSet via the Helm chart.

Motivation

In certain scenarios, the CSI driver pods enter a CrashLoopBackOff state. After investigation, and based on this article from Dynatrace Community (https://community.dynatrace.com/t5/Troubleshooting/Kubernetes-Operator-CSI-driver-crashes-frequently/ta-p/272021), it appears that tuning the livenessProbe is one of the recommended mitigations.

While adjusting resource limits is currently supported through the Helm chart, the other two workarounds require manual patches, which are not ideal for industrialized or automated deployments. This change makes livenessProbe configuration manageable via values.yaml, enhancing flexibility and reducing operational overhead.

How can this be tested?

  • Install or upgrade the Helm chart with a custom values.yaml including a disabled livenessProbe under csidriver.server.livenessProbe.
  • Confirm that the generated DaemonSet for dynatrace-oneagent-csi-driver reflects the expected probe configuration.

Example values.yaml:

csidriver:
  server:
    livenessProbe:
      enabled: false

@oihanitrz oihanitrz changed the title feature(charts): add livenessProbe for server container of csi daemonset in values feature(charts): add livenessProbe config of csi daemonset in values Jul 4, 2025
@oihanitrz oihanitrz changed the title feature(charts): add livenessProbe config of csi daemonset in values feature(charts): add liveness probe config of csi daemonset in values Jul 4, 2025
@oihanitrz
Copy link
Author

I also added support for customizing the --probe-timeout argument of the liveness-probe container, which I had initially overlooked.

@andriisoldatenko
Copy link
Contributor

Hi @oihanitrz! Thanks for contributing. We will review and test it internally and return to you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants