Skip to content

Add check to AMR watcher to handle subject removing requests#42171

Merged
EdwardDowling merged 6 commits intomasterfrom
edwarddowling/access-monitoring-rule-watcher-filter
May 30, 2024
Merged

Add check to AMR watcher to handle subject removing requests#42171
EdwardDowling merged 6 commits intomasterfrom
edwarddowling/access-monitoring-rule-watcher-filter

Conversation

@EdwardDowling
Copy link
Copy Markdown
Contributor

@EdwardDowling EdwardDowling commented May 30, 2024

Part of 3132

Prevents watcher that updates the cache of rules for the plugins from discarding updates that remove KindAccessRequest from the subjects.

changelog: Fix bug where the plugins AMR cache is not updated in the event Access requests are removed from the subject of an existing rule

@github-actions
Copy link
Copy Markdown
Contributor

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

@github-actions
Copy link
Copy Markdown
Contributor

The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with changelog: followed by the changelog entries for the PR.

Comment thread integrations/access/slack/testlib/suite.go
@EdwardDowling EdwardDowling enabled auto-merge May 30, 2024 16:38
@EdwardDowling EdwardDowling added this pull request to the merge queue May 30, 2024
Merged via the queue into master with commit 2108041 May 30, 2024
@EdwardDowling EdwardDowling deleted the edwarddowling/access-monitoring-rule-watcher-filter branch May 30, 2024 17:10
@public-teleport-github-review-bot
Copy link
Copy Markdown

@EdwardDowling See the table below for backport results.

Branch Result
branch/v15 Create PR
branch/v16 Create PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants