-
Notifications
You must be signed in to change notification settings - Fork 2
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
VIH-10175 Refactor health check to improve startup and liveness #213
Conversation
* VIH-0000 - Renovate - Update Helm release java to v4.2.0 * Bumping chart version --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: hmcts-ado-cnp <174565+hmcts-ado-cnp[bot]@users.noreply.github.com>
… an ejud username (for applicable roles only) (#214) This applies to Judge and Judicial Office Holders only
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
health checks seem good. It looks like this PR also fixes an issue with notifications? Might be worth adding the ticket number to the PR description, just for reference
The merge-base changed after approval.
The bug fix was merged in PR #214 . I just took a branch off that for this branch |
The merge-base changed after approval.
replaced with #215 |
JIRA link (if applicable)
VIH-10175
Change description
Refactor health check
Does this PR introduce a breaking change? (check one with "x")