Skip to content

DBC22-7174: Remove Liveness Probe from Tasks - #1445

Merged
wmuldergov merged 2 commits into
mainfrom
task/DBC22-7174-2
Sep 11, 2026
Merged

DBC22-7174: Remove Liveness Probe from Tasks#1445
wmuldergov merged 2 commits into
mainfrom
task/DBC22-7174-2

Conversation

@wmuldergov

Copy link
Copy Markdown
Collaborator

This is causing excessive resource usage and restarts.

📝 Submitter

🔗 JIRA Ticket


✅ Quality Assurance & Requirements

  • Requirements Met: I have confirmed that all acceptance criteria from the JIRA ticket are fulfilled.
  • Tested desktop in local or dev envs
  • Tested mobile in local or dev envs
  • Ran unit tests locally
  • SonarCloud: I have verified that the SonarCloud analysis is clean/passing for this branch.

⚙️ Configuration & Environment

  • New Env Variables: Does this PR require new environment variables? (Yes/No)

    If yes, please list them here and ensure they are added to secret manager, the .env.example. and the Vault by an STA.

🧪 How to Test (if required)

  1. Deploy and confirm that the liveness probe for tasks is no longer in the deployment.

🔍 Reviewer Checklist

  • Reviewed code for logic and cleanliness
  • Re-tested desktop/mobile in local or dev envs
  • Verified no new console warnings/errors
  • Confirmed that any new env variables are understood/documented

@wmuldergov
wmuldergov requested a review from ray-oxd August 20, 2026 22:28
@ray-oxd

ray-oxd commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

@wmuldergov pushed a replacement for the liveness probe script. Please review the changes in leader-election.py

@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2026

Copy link
Copy Markdown

@wmuldergov

wmuldergov commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator Author

@wmuldergov pushed a replacement for the liveness probe script. Please review the changes in leader-election.py

@ray-oxd I tested the change and noticed a few issues due to how timestamps are handled. It was off by 7 hours. I fixed that now and it works well in dev. Feel free to approve and merge to main.

@wmuldergov
wmuldergov merged commit daafaa5 into main Sep 11, 2026
15 checks passed
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