Skip to content

deprecate errors high alerts #288

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

avlitman
Copy link
Collaborator

@avlitman avlitman commented Apr 15, 2025

follow-up for this pr: kubevirt/kubevirt#13911

This pr also Fix all deprecated runbooks content to this template:

# <alert-name> [Deprecated]

This alert has been deprecated; it does not indicate a genuine issue. If
triggered, it may be safely ignored and silenced.

Release note:

RESTErrorsHigh alerts are deprecated.

follow-up for this pr: kubevirt/kubevirt#13911

Signed-off-by: avlitman <[email protected]>
@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Apr 15, 2025
@kubevirt-bot kubevirt-bot added size/XS dco-signoff: no Indicates the PR's author has not DCO signed all their commits. size/XL and removed dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XS labels Apr 15, 2025
Fix all deprecated runbooks content to this template:
# <alert-name> [Deprecated]

This alert has been deprecated; it does not indicate a genuine issue. If
triggered, it may be safely ignored and silenced.

Signed-off-by: avlitman <[email protected]>
@avlitman avlitman force-pushed the deprecate-high-alerts branch from 488570e to 1cea69b Compare April 15, 2025 11:31
@kubevirt-bot kubevirt-bot added dco-signoff: yes Indicates the PR's author has DCO signed all their commits. and removed dco-signoff: no Indicates the PR's author has not DCO signed all their commits. labels Apr 15, 2025
@sradco
Copy link
Collaborator

sradco commented Apr 28, 2025

Hi @avlitman, I think we should add the deprecation notice at the top of the runbooks, but not remove all the runbooks content.

@sradco
Copy link
Collaborator

sradco commented May 29, 2025

Hi @avlitman , please see my comment above

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dco-signoff: yes Indicates the PR's author has DCO signed all their commits. size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants