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

Remove HealthCheckPolicy's #976

Merged
merged 2 commits into from
Dec 12, 2024
Merged

Remove HealthCheckPolicy's #976

merged 2 commits into from
Dec 12, 2024

Conversation

tnotheis
Copy link
Member

@tnotheis tnotheis commented Dec 12, 2024

Readiness checklist

  • I added/updated unit tests.
  • I added/updated integration tests.
  • I ensured that the PR title is good enough for the changelog.
  • I labeled the PR.

We removed the HealthCheckPolicy's again, because according to the official documentation, it defaults to the pod's readiness probe, which is perfect for us.

the official documentation.

@tnotheis tnotheis added the enhancement New feature or request label Dec 12, 2024
@tnotheis tnotheis merged commit 35687c4 into main Dec 12, 2024
18 checks passed
@tnotheis tnotheis deleted the remove-healthcheckpolicys branch December 12, 2024 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants