Skip to content

configurable settings for liveness and readiness probes

Aaron Blythe requested to merge aaronblythe/gitlab-ce:liveness_readiness into master

What does this MR do?

Coincides with charts/auto-deploy-app!19 (merged) The point of this is to pass into the exposed variables for the livenessprobe and readinessprobe to make them configurable, since not all applications will serve a 200 on the root ("/") and/or the root ("/") may not be the appropriate health check for the application.

This is being submitted as a supplement to charts/auto-deploy-app!19 (merged) which is more important. Exhibiting how to test/exercise that code is the initial point here. If this is the proper way forward then I will work on the checklist below, however would prefer feedback on the original helm chart issue first.

What are the relevant issue numbers?

charts/auto-deploy-app!19 (merged) https://gitlab.com/gitlab-org/gitlab-ce/issues/46536

Does this MR meet the acceptance criteria?

Edited by 🤖 GitLab Bot 🤖

Merge request reports