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

Implement periodic check to notify access list owners of reviews due soon #51581

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

rudream
Copy link
Contributor

@rudream rudream commented Jan 29, 2025

Purpose

e counterpart: https://github.com/gravitational/teleport.e/pull/5974

This PR adds a periodic check which will look for access lists due for review within certain thresholds (14d, 7d, 3d, 0d, -3d, -7d) and create reminder notifications for those capable of reviewing those access lists.

Duplicate notifications are prevented by the use of unique notification identifiers, ie. a particular access list will never trigger more than one 7d reminder. It is however possible for two access lists to trigger 7d reminder notifications at different times.

Base automatically changed from yassine/access-list-notification-type to master January 29, 2025 05:47
@rudream rudream force-pushed the yassine/access-list-reminders branch from 2e0990c to 0ad0040 Compare January 29, 2025 20:47
@rudream rudream marked this pull request as ready for review January 29, 2025 20:47
@rudream rudream requested review from kimlisa and removed request for flyinghermit January 29, 2025 20:48
@rudream rudream added the no-changelog Indicates that a PR does not require a changelog entry label Jan 29, 2025
@rudream rudream force-pushed the yassine/access-list-reminders branch from 0ad0040 to d403fd5 Compare January 29, 2025 21:48
@rudream rudream requested a review from zmb3 January 30, 2025 00:08
@rudream rudream force-pushed the yassine/access-list-reminders branch from e1d4901 to 9aecb23 Compare January 30, 2025 00:10
@rudream rudream requested review from fspmarshall and zmb3 January 30, 2025 17:37
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from kimlisa January 30, 2025 21:52
@rudream rudream force-pushed the yassine/access-list-reminders branch from ab213d3 to 2a11012 Compare January 31, 2025 15:21
@rudream rudream added this pull request to the merge queue Jan 31, 2025
@rudream rudream removed this pull request from the merge queue due to a manual request Jan 31, 2025
@rudream rudream enabled auto-merge January 31, 2025 15:42
@rudream rudream added this pull request to the merge queue Jan 31, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jan 31, 2025
@rudream rudream added this pull request to the merge queue Jan 31, 2025
Merged via the queue into master with commit 5b4dee8 Jan 31, 2025
47 checks passed
@rudream rudream deleted the yassine/access-list-reminders branch January 31, 2025 16:19
@public-teleport-github-review-bot

@rudream See the table below for backport results.

Branch Result
branch/v17 Failed

github-merge-queue bot pushed a commit that referenced this pull request Jan 31, 2025
…reviews due soon (#51713)

* add access list reminder notification type (#51426)

* periodically create access list reminder notifications when needed (#51581)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/branch/v17 no-changelog Indicates that a PR does not require a changelog entry size/md ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants