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

add basic alerting for Kueue #258

Merged
merged 1 commit into from
May 10, 2024

Conversation

KPostOffice
Copy link

@KPostOffice KPostOffice commented May 8, 2024

severity: warning
annotations:
description: This alert fires when the Kueue Operator is not running.
triage: 'https://gitlab.cee.redhat.com/service/managed-tenants-sops/-/blob/main/RHODS/Distributed-Workloads/kuberay-operator-availability.md' # TODO: what link here?
Copy link

Choose a reason for hiding this comment

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

i think you will need to create a new file for kueue in https://gitlab.cee.redhat.com/service/managed-tenants-sops/-/blob/main/RHODS/Distributed-Workloads and link it here.

Copy link

Choose a reason for hiding this comment

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

as long as this is updated, i am fine with this PR

Signed-off-by: Kevin <kpostlet@redhat.com>
severity: warning
annotations:
description: This alert fires when the Kueue Operator is not running.
triage: 'https://gitlab.cee.redhat.com/service/managed-tenants-sops/-/blob/main/RHODS/Distributed-Workloads/kuberay-operator-availability.md' # TODO: what link here?
Copy link

Choose a reason for hiding this comment

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

Suggested change
triage: 'https://gitlab.cee.redhat.com/service/managed-tenants-sops/-/blob/main/RHODS/Distributed-Workloads/kuberay-operator-availability.md' # TODO: what link here?
triage: 'https://gitlab.cee.redhat.com/service/managed-tenants-sops/-/blob/main/RHODS/Distributed-Workloads/kueue-operator-availability.md'

@zdtsw
Copy link

zdtsw commented May 9, 2024

once this is in "main" we need backport to rhoai-2.9

@KPostOffice
Copy link
Author

This should all be g2g now. The SOPs PR has been merged

@zdtsw zdtsw merged commit 0fbd57c into red-hat-data-services:main May 10, 2024
3 checks passed
zdtsw pushed a commit to zdtsw-forking/rhods-operator that referenced this pull request May 10, 2024
Signed-off-by: Kevin <kpostlet@redhat.com>
zdtsw added a commit that referenced this pull request May 10, 2024
Signed-off-by: Kevin <kpostlet@redhat.com>
Co-authored-by: Kevin Postlethwait <k.postlethwait24@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants