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

[bitnami/mongodb] metrics readiness/liveness probe #14466

Closed
aeber opened this issue Jan 20, 2023 · 4 comments · Fixed by #14573
Closed

[bitnami/mongodb] metrics readiness/liveness probe #14466

aeber opened this issue Jan 20, 2023 · 4 comments · Fixed by #14573
Assignees

Comments

@aeber
Copy link
Contributor

aeber commented Jan 20, 2023

Name and Version

bitnami/mongodb v13.6.4

What is the problem this feature will solve?

When the amount of data in the database increases, the query to the /metrics endpoint might take longer than the specified timeout of the readiness and liveness probes (for example see: #12717)

The readiness and liveness probes currently query /metrics due to upstream changes in the mongodb-exporter v0.33.0: #11398

What is the feature you are proposing to solve the problem?

Release v0.36.0 of the mongodb-exporter added a handler for / and this version is already distributed since the bitnami/mongodb chart v13.6.3.

So it should be possible to switch back to using / for readiness and liveness probes. This should also improve stability as the readiness probe should no longer fail when the amount of data in the database increases.

What alternatives have you considered?

Increasing timeouts of readiness and liveness probes each time the amount of data in the db increases query time of /metrics over the timeout threshold.

@github-actions github-actions bot added the triage Triage is needed label Jan 20, 2023
@javsalgar javsalgar changed the title bitnami/mongodb - metrics readiness/liveness probe [bitnami/mongodb] metrics readiness/liveness probe Jan 23, 2023
@javsalgar
Copy link
Contributor

Hi,

Thank you so much for the input. Would you like to submit a PR changing the probe?

aeber added a commit to aeber/bitnami-charts that referenced this issue Jan 27, 2023
Fixes bitnami#14466

Signed-off-by: Anselm Eberhardt <aeberhardt@dg-i.net>
@aeber
Copy link
Contributor Author

aeber commented Jan 27, 2023

Yeah sure, i just created one: #14573

@javsalgar
Copy link
Contributor

Thank you so much!

@ckittelmann
Copy link

I had the same issue with timeouts. In 13.6.6 the timeout was increased to 10s by #14539 but I think to change to / is a good solution to reduce requests to /metrics which is done very often (by default every 5 seconds for liveness and readiness probes)

CeliaGMqrz pushed a commit that referenced this issue Jan 31, 2023
…probes (#14573)

Fixes #14466

Signed-off-by: Anselm Eberhardt <aeberhardt@dg-i.net>
nik0811 pushed a commit to nik0811/charts that referenced this issue Feb 4, 2023
…probes (bitnami#14573)

Fixes bitnami#14466

Signed-off-by: Anselm Eberhardt <aeberhardt@dg-i.net>
Signed-off-by: nikhil <nikhil811raj@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants