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

[v13] Add in audit review recurrence presets. #32961

Merged
merged 1 commit into from
Oct 5, 2023

Conversation

mdwn
Copy link
Contributor

@mdwn mdwn commented Oct 4, 2023

Backport of #32843 to branch/v13.

Access Lists now have audit review recurrence presets. These allow users to specify review frequencies of 1, 3, 6, or 12 months, and specify the 1st, 15th, or last days of the target month. Presets have been used for their simplicity over other various recurrence definition mechanisms, as these presets are much clearer than many of the other options.

Note: This was a manual backport. The two conflicts were the ordering in the cache_tests.go, and that unified_resource_test.go doesn't exist in v13.

Access Lists now have audit review recurrence presets. These allow users to
specify review frequencies of 1, 3, 6, or 12 months, and specify the 1st,
15th, or last days of the target month. Presets have been used for their
simplicity over other various recurrence definition mechanisms, as these
presets are much clearer than many of the other options.
@github-actions github-actions bot added backport size/md tctl tctl - Teleport admin tool labels Oct 4, 2023
@espadolini espadolini changed the title Add in audit review recurrence presets. [v13] Add in audit review recurrence presets. Oct 4, 2023
@espadolini espadolini requested review from r0mant and jakule and removed request for espadolini and rudream October 4, 2023 16:02
@mdwn mdwn added this pull request to the merge queue Oct 5, 2023
Merged via the queue into branch/v13 with commit f85e477 Oct 5, 2023
28 checks passed
@mdwn mdwn deleted the mike.wilson/v13-access-list-recurrence-presets branch October 5, 2023 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport size/md tctl tctl - Teleport admin tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants