alert-mgmt-14: single alert rule update + delete-by-ID#13
Open
sradco wants to merge 1 commit intoalert-mgmt-13-docs-ci-e2efrom
Open
alert-mgmt-14: single alert rule update + delete-by-ID#13sradco wants to merge 1 commit intoalert-mgmt-13-docs-ci-e2efrom
sradco wants to merge 1 commit intoalert-mgmt-13-docs-ci-e2efrom
Conversation
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
133781c to
c1aa685
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
b8ebc48 to
cacc1a3
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
c1aa685 to
bd693a6
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
cacc1a3 to
0a8c965
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
bd693a6 to
10139d8
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
0a8c965 to
a8e9b58
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
10139d8 to
aa409e1
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
a8e9b58 to
7a14299
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
aa409e1 to
63b0924
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
7a14299 to
78f93f8
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
63b0924 to
b1fdfb4
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
78f93f8 to
4d15eeb
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
b1fdfb4 to
3ac18c0
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
4d15eeb to
281349b
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
3ac18c0 to
26ce67f
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
281349b to
7cddc0c
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
26ce67f to
4a09657
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
7cddc0c to
0b1688e
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
4a09657 to
854ed0c
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
0b1688e to
c3cae66
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
854ed0c to
65fe1c8
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
c3cae66 to
6e4c042
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
65fe1c8 to
1340083
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
6e4c042 to
e814a67
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
1340083 to
722eb82
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
e814a67 to
fdf4860
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
722eb82 to
5b5dbc4
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
fdf4860 to
a7a6ffc
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
5b5dbc4 to
eb40e18
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
a7a6ffc to
139b443
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
eb40e18 to
f5efdf2
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
2 times, most recently
from
31b2d04 to
81e5e9c
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
f5efdf2 to
63f77d3
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
81e5e9c to
9ade599
Compare
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
63f77d3 to
49ee8dd
Compare
Signed-off-by: Shirly Radco <sradco@redhat.com> Signed-off-by: João Vilaça <jvilaca@redhat.com> Signed-off-by: Aviv Litman <alitman@redhat.com> Co-authored-by: AI Assistant <noreply@cursor.com>
sradco
force-pushed
the
alert-mgmt-13-docs-ci-e2e
branch
from
49ee8dd to
b97de66
Compare
sradco
force-pushed
the
alert-mgmt-14-single-rule
branch
from
9ade599 to
b8e9698
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Alert Management API — Part 14/14: PATCH/DELETE /rules/{ruleId} (single rule)
Summary
PATCH /api/v1/alerting/rules/{ruleId}handler: single-rule update supporting alerting rule changes, drop/restore toggle, and classification patch in one requestDELETE /api/v1/alerting/rules/{ruleId}handler: single-rule delete by IDNotAllowedError(same pattern as bulk update)Dependencies
This PR is part of a stacked series. Please review in order.
2–13. Pending — alert queries, relabel/health, management foundation, endpoints, CRUD, classification, platform update, bulk update, docs/CI/e2e
14 . → This PR — PATCH/DELETE /rules/{ruleId} (single rule)