Skip to content

Implement auto-approvals for datadog#47475

Merged
bernardjkim merged 4 commits intomasterfrom
bernard/datadog-auto-approval
Oct 16, 2024
Merged

Implement auto-approvals for datadog#47475
bernardjkim merged 4 commits intomasterfrom
bernard/datadog-auto-approval

Conversation

@bernardjkim
Copy link
Copy Markdown
Contributor

@bernardjkim bernardjkim commented Oct 11, 2024

Paired with https://github.com/gravitational/teleport.e/pull/5150

This PR extends the Datadog plugin to support automatic approvals.

Example Usage: A user with the following role will have access requests automatically approved if they are on-call for the datadog-team at the time of making the access request.

kind: role
metadata:
  name: requester
spec:
  allow:
    request:
      annotations:
        teleport.dev/schedules:
        - datadog-team
...

changelog: Extend the Datadog plugin to support automatic approvals.

Comment thread integrations/access/accessrequest/app.go
Comment thread integrations/access/accessrequest/app.go
Comment thread integrations/access/datadog/bot.go Outdated
Comment thread integrations/access/datadog/client.go Outdated
Copy link
Copy Markdown
Contributor

@hugoShaka hugoShaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you for adding the auto-approval capability into the common app.

- Use standard teleport.dev/schedules annotation
- Link Datadog API docs
@bernardjkim bernardjkim requested a review from hugoShaka October 11, 2024 22:26
Comment thread integrations/access/datadog/bot.go Outdated
@bernardjkim bernardjkim requested a review from hugoShaka October 15, 2024 16:22
@bernardjkim bernardjkim added this pull request to the merge queue Oct 16, 2024
Merged via the queue into master with commit e9f296f Oct 16, 2024
@bernardjkim bernardjkim deleted the bernard/datadog-auto-approval branch October 16, 2024 02:35
@public-teleport-github-review-bot
Copy link
Copy Markdown

@bernardjkim See the table below for backport results.

Branch Result
branch/v16 Create PR

mvbrock pushed a commit that referenced this pull request Oct 16, 2024
* Implement auto-approvals for datadog

* Address feedback

- Use standard teleport.dev/schedules annotation
- Link Datadog API docs

* Check annotations before api calls
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