Skip to content

add rbac for cluster alerts#26423

Merged
fspmarshall merged 1 commit intomasterfrom
fspmarshall/alert-rbac
May 17, 2023
Merged

add rbac for cluster alerts#26423
fspmarshall merged 1 commit intomasterfrom
fspmarshall/alert-rbac

Conversation

@fspmarshall
Copy link
Copy Markdown
Contributor

Adds RBAC support for cluster alerts so that the tctl alert * family of commands can be used via remote tctl. Having create/update permissions for cluster_alert allows crating/acknowledging alerts, and having read/list allows tctl alerts ls to show all alerts, even if their labels don't specify the caller as the target audience.

Updates the provided editor and auditor roles to have read/write and read-only alert permissions respectively.

Should be the last item to fully address #17188

@github-actions github-actions Bot added size/sm tctl tctl - Teleport admin tool labels May 17, 2023
@github-actions github-actions Bot requested review from hugoShaka and rosstimothy May 17, 2023 04:15
Comment thread api/proto/teleport/legacy/types/types.proto Outdated
Comment thread lib/auth/auth_with_roles.go Outdated
@fspmarshall fspmarshall force-pushed the fspmarshall/alert-rbac branch 2 times, most recently from 4146df7 to 28c34db Compare May 17, 2023 17:45
@fspmarshall fspmarshall force-pushed the fspmarshall/alert-rbac branch from 28c34db to a492e04 Compare May 17, 2023 21:19
@fspmarshall fspmarshall enabled auto-merge May 17, 2023 21:20
@fspmarshall fspmarshall added this pull request to the merge queue May 17, 2023
Merged via the queue into master with commit b4bc465 May 17, 2023
@fspmarshall fspmarshall deleted the fspmarshall/alert-rbac branch May 17, 2023 21:53
@public-teleport-github-review-bot
Copy link
Copy Markdown

@fspmarshall See the table below for backport results.

Branch Result
branch/v12 Failed
branch/v13 Failed

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

Labels

size/sm tctl tctl - Teleport admin tool

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants