Skip to content

Conversation

rh-hemartin
Copy link
Contributor

Description

Our SLO dashboard was using konflux_up without filtering it by cluster, leading to confusion. This PR solves it. Closes KAR-612.

Pre-Submission Checklist

  • Jira Ticket: If a corresponding Jira ticket exists, it is linked in the description above, in the PR name, and/or in the commit message.
  • Alert Tests: New or modified alerts include tests to ensure they function correctly.
  • SOP / Runbook: Any required SOP has been created or updated as needed. If it has direct connection to the dashboard or alert - It should be included within the dashboard or alert's runbook label respectively.
  • Dashboards Addition: New dashboards or significant changes to them should have a link to the staging instance for validation purposes.
  • Contribution Guides: This submission follows the guidelines in our CONTRIBUTING.md - specifically about the commit conventions and PR instructions - together with our README.md which explains contents of this repository with useful examples for contribution.
  • Pipeline Finished Successfully

Deployment Notice

  • Production Deployment: For any changes to alerts, recording rules or dashboards, I understand that the deployment of changes to production requires updating the commit reference to o11y in app-interface repository.

Review

Once your PR is ready please let us know in the #forum-konflux-o11y slack channel. Tag @konflux-o11y-ic for assistance.

Copy link
Collaborator

@pacho-rh pacho-rh left a comment

Choose a reason for hiding this comment

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

LGTM

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