Update admin health page with the application readiness endpoints
What does this MR do?
Adds the application readiness endpoint links to the monitoring -> health check view inside the admin section
Screenshots (if relevant)
Before | After |
---|---|
![]() |
![]() |
Does this MR meet the acceptance criteria?
-
Changelog entry added, if necessary - Tests
-
All builds are passing
-
-
Conform by the merge request performance guides -
Conform by the style guides -
Branch has no merge conflicts with master
(if it does - rebase it please) -
Squashed related commits together
What are the relevant issue numbers?
closes #31359 (closed)
Merge request reports
Activity
@joshlambert Can you take a look at the screenshots, is this located in the right place?
@jivanvl yep! But we actually want to remove the old ones and just show the new ones. Those are deprecated now.
@joshlambert All of them?
Yea. We replaced them with the new ones. Maybe we can say "Health information can be retrieved from the following endpoints. More information is available here." for the title.
Then just list the endpoints without the second grouping.
Edited by Joshua Lambert@joshlambert Ohh I see, thanks a lot for the clarification
@joshlambert
I'm going to update the docs as well, they still use the old endpointsNevermind I think it's best we leave there for a couple of releases since there's going to be the need to actually look at the old behavior also, The new endpoints are thereEdited by Jose Ivan Vargas Lopezadded 20 commits
-
12bee1c5...aab904f7 - 18 commits from branch
master
- 5b5d6405 - update admin health page with the application readiness endpoints
- b2c34e93 - Removed old endpoints from the monitoring view
-
12bee1c5...aab904f7 - 18 commits from branch
assigned to @brycepj
@brycepj Could you take a look when you get a chance? Thanks!
- Resolved by Jose Ivan Vargas Lopez
@jivanvl A couple tiny comments. Otherwise, this LGTM
Thanks @brycepj should I correct the double quotes and assign to an endboss?
@jivanvl yep
added 1 commit
- 1f299612 - Removed old endpoints from the monitoring view
assigned to @filipa
Thank you @jivanvl code LGTM!
enabled an automatic merge when the pipeline for 1f299612 succeeds
@jivanvl can you please merge master? Tests keep failing
assigned to @jivanvl
added 72 commits
-
1f299612...43befaf2 - 70 commits from branch
master
- e402000b - update admin health page with the application readiness endpoints
- 94a6c0de - Removed old endpoints from the monitoring view
-
1f299612...43befaf2 - 70 commits from branch
@filipa builds passed
assigned to @filipa
mentioned in commit 55f6b1ed