Skip to content
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

Webhook PodDistributionBudget created when HighAvailability is set to False via Helm Deployment #4008

Closed
coolmancgfs opened this issue Oct 31, 2024 · 1 comment · Fixed by #4013
Labels
bug Something isn't working

Comments

@coolmancgfs
Copy link

coolmancgfs commented Oct 31, 2024

Describe the bug
When deploying the cloud native fullstack operator v1.3.2 via helm the webhook resource is created with a PodDistributionBudget that has minAvailable: 1 even when you set highAvailability: false in the values.yaml. The webhook then runs as a single replica with PDB that does not allow eviction. This prevents eviction and blocks nodes from being drained for shutdown, resulting in blocking k8s upgrades.

To Reproduce

  1. Deploy the DT K8s Operator Cloud Native via Helm with the webhook values.yaml highAvailability: false
  2. Attempt to drain node with webhook deployment running.

Expected behavior
The webhook deployment with highAvailability: false does not have a PDB that prevents eviction of the single pod

Screenshots
image

Additional context
This appears to be a regression from this commit:
83801f6#diff-67f1e210b8d311b90567c94206d9d6b9d8bfa16967742f8f19fc0ffbeeb3bd17

As a workaround you can deploy the webhook in HA mode to allow multiple replicas and the PDB will not prevent eviction of a 1 pod.

@luhi-DT
Copy link
Collaborator

luhi-DT commented Nov 4, 2024

Hi @coolmancgfs,

thanks for the report, will create an internal bug for this. A fix for this will be available in Operator v1.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants