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

Update Health check alerts in Teletraan with Runbook info #1223

Merged
merged 4 commits into from
Aug 1, 2023

Conversation

rfleur01
Copy link
Contributor

@rfleur01 rfleur01 commented Jul 24, 2023

This PR is being submitted to provide a way for Teletraan operators to configure additional oncall information in the form of a runbook link for health check alerts.

Test Plan:

  1. Navigate to the Group Configuration page for a cluster
  2. Confirm that the group has Runbook Link blank in the UI and in the DB
Screenshot_2023-07-24_at_10_24_19_AM Screenshot_2023-07-24_at_10_24_33_AM
  1. Create a health check that will eventually timeout

  2. Confirm that the Email received is formatted correctly without the runbook link

Screenshot_2023-07-24_at_10_15_38_AM
  1. Populate the Runbook Link in the UI

  2. Confirm that the Link is populated in the UI and DB

Screenshot_2023-07-24_at_10_22_16_AM Screenshot_2023-07-24_at_10_22_48_AM
  1. Create a health check that will eventually timeout

  2. Confirm that the Email received is formatted correctly without the runbook link

Screenshot_2023-07-24_at_10_15_50_AM

@rfleur01 rfleur01 changed the title Rfleurinord/healthcheck alert info Update Health check alerts in Teletraan with Runbook info Jul 24, 2023
@rfleur01 rfleur01 marked this pull request as ready for review July 24, 2023 22:29
@rfleur01 rfleur01 requested a review from a team as a code owner July 24, 2023 22:29
Comment on lines +367 to +369
$("#runbooklinkExpressionId").click(function() {
$("#runbooklinkExpressionDetailsId").collapse('toggle');
});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

non-blocking. This and all above can share the same class and this only need to apply to the class once.

@rfleur01 rfleur01 merged commit 6a9b5b9 into master Aug 1, 2023
4 checks passed
@rfleur01 rfleur01 deleted the rfleurinord/healthcheck-alert-info branch August 1, 2023 17:58
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