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

DEVOPS-30642 disable liveness probes #395

Merged
merged 1 commit into from
Jan 22, 2025
Merged

Conversation

drewwells
Copy link
Contributor

@drewwells drewwells commented Jan 22, 2025

dsnexec does not use the standard mount volumes of dbproxy. So
it can't use probes pointing to those mount volumes. Removed
the liveness probe and setup the readiness probe to point
to DBPROXY_CREDENTIAL env variable.

https://infoblox.atlassian.net/browse/DEVOPS-30642

@drewwells drewwells marked this pull request as draft January 22, 2025 18:03
    dsnexec does not use the standard mount volumes of dbproxy. So
    it can't use probes pointing to those mount volumes. Removed
    the liveness probe and setup the readiness probe to point
    to DBPROXY_CREDENTIAL env variable.
@drewwells drewwells force-pushed the hotfix/DEVOPS-30642 branch from 0a4da47 to f31a28f Compare January 22, 2025 20:50
@drewwells drewwells marked this pull request as ready for review January 22, 2025 20:50
@drewwells drewwells merged commit f8e4c46 into release/v1.10.x Jan 22, 2025
3 checks passed
@drewwells drewwells deleted the hotfix/DEVOPS-30642 branch January 22, 2025 20:56
drewwells added a commit that referenced this pull request Jan 31, 2025
* release/v1.10.x:
  DEVOPS-30642 disable liveness probes (#395)
  DEVOPS-30870: fix password rotation logic (#393)
  DEVOPS-30046 optional sidecar probes (#384)
  DEVOPS-30046 DEVOPS-30239 dsnexec conversion injects dbproxy (#381)
  update start-pgbouncer to exit after 60seconds (#379)
  PTEUDO-2142 remove validation of DSNName (#370)
  update CI pipeline for gitflow branch names
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