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

BC-8811 - Reconfigure the Probs for etherpad #1092

Merged
merged 1 commit into from
Jan 22, 2025
Merged

Conversation

mamutmk5
Copy link
Member

Description

Set the initialeDelaySeconds for the readinessProbe and livenessProb to 0 so that is hase not effect and Ansible over wirte it.
Add an startupProbe to the Etherpad deployment so that every 5 seconds for an Failuer Thrashold von 60 will be show if Etherpad is up and runing. So we detect the Earlys timepoint where the user cann access the POD via HTTP.
We hatte seen at Production that an initial delay off 60 seconds where to low in some cases and we nead an delay like 120 seconds or 180 seconds on wone hand but on the other hand we hade ssen Etherpad PODs who where up in like 30 seconds.
This Problem had shown that it's easer to add an startupProbe who test it every 5 Seconds for an failure threashod of 60 woud be better, so if the POD is up after 30 second the failure count is at 6 and if the POD is up after 90 seconds the failure count is up to 9.

Links to Tickets or other pull requests

Changes

Datasecurity

Deployment

New Repos, NPM pakages or vendor scripts

Screenshots of UI changes

Approval for review

  • All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

Notice: Please remove the WIP label if the PR is ready to review, otherwise nobody will review it.

@mamutmk5 mamutmk5 self-assigned this Jan 22, 2025
@mamutmk5 mamutmk5 merged commit 17defe4 into main Jan 22, 2025
53 checks passed
@mamutmk5 mamutmk5 deleted the bc-8811-etherpad branch January 22, 2025 15:37
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