[v15] Fix opsgenie plugin to use correct annotation to determine whether or not to create alerts #40121
Conversation
|
🤖 Vercel preview here: https://docs-dief57u5o-goteleport.vercel.app/docs/ver/preview |
|
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 |
|
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 |
|
🤖 Vercel preview here: https://docs-h3xhga9h6-goteleport.vercel.app/docs/ver/preview |
|
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 |
|
🤖 Vercel preview here: https://docs-lptqduvyp-goteleport.vercel.app/docs/ver/preview |
|
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 |
|
🤖 Vercel preview here: https://docs-6z3b310r5-goteleport.vercel.app/docs/ver/preview |
|
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 |
|
🤖 Vercel preview here: https://docs-hwbj8oywo-goteleport.vercel.app/docs/ver/preview |
And add tests for accessrequest app
|
🤖 Vercel preview here: https://docs-gct8pt17y-goteleport.vercel.app/docs/ver/preview |
|
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 |
|
🤖 Vercel preview here: https://docs-ouw657eke-goteleport.vercel.app/docs/ver/preview |
|
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 |
|
🤖 Vercel preview here: https://docs-df4bgzm7m-goteleport.vercel.app/docs/ver/preview |
|
@zmb3 can you take a look at this when you get a chance |
zmb3
left a comment
zmb3
left a comment
There was a problem hiding this comment.
Looks good, assuming you fix the lint failures.
public-teleport-github-review-bot
Bot
removed the request for review
from r0mant
|
🤖 Vercel preview here: https://docs-889gznw2k-goteleport.vercel.app/docs/ver/preview |
|
🤖 Vercel preview here: https://docs-dmv66d28i-goteleport.vercel.app/docs/ver/preview |
|
🤖 Vercel preview here: https://docs-5oigu2ioh-goteleport.vercel.app/docs/ver/preview |
|
🤖 Vercel preview here: https://docs-m3ifnblhc-goteleport.vercel.app/docs/ver/preview |
3 participants
backport of #38435
Tested with the backwards compatible version of the behaviors for the annotations
Wont change auto approval behavior until 3823 is merged to /e after this is merged
changelog: Updated Opsgenie annotations so approve-schedules is used for both alert creation and auto approval if notify schedules is not set.