feat(cloudwatch-actions): add ssm opsitem action for cloudwatch alarm #16923
Conversation
|
|
peterwoodworth
changed the title
There was a problem hiding this comment.
This is great, thank you for contributing, @vportascarta ! I just have a couple extremely minor comments on the docs.
Co-authored-by: Madeline Kusters <80541297+madeline-k@users.noreply.github.com>
|
@madeline-k It's all good for me. |
mergify
bot
dismissed
madeline-k’s stale review
Pull request has been modified.
There was a problem hiding this comment.
Sorry, I approved prematurely! We need an update to the README to describe this new feature as well. https://github.com/aws/aws-cdk/blob/master/packages/@aws-cdk/aws-cloudwatch-actions/README.md
mergify
bot
dismissed
madeline-k’s stale review
Pull request has been modified.
|
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
|
The build is failing now because this module has been migrated to use the Assertions library for unit tests. You will need to update the |
mergify
bot
dismissed
madeline-k’s stale review
Pull request has been modified.
madeline-k
changed the title
There was a problem hiding this comment.
Thank you, @vportascarta! This is ready to merge, and should go out in the next weekly release.
|
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
Fix compilation errors in README example
mergify
bot
dismissed
madeline-k’s stale review
Pull request has been modified.
Accidentally removed them in prior commit.
mergify
bot
dismissed
madeline-k’s stale review
Pull request has been modified.
|
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
Thank you for contributing! Your pull request will be updated from master and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
…aws#16923) This small PR will add SSM OpsItem action to cloudwatch alarm. The arn format was taken from the alarm UI (in view source) and with all the parameters (severity and category) closes aws#16861 ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
This small PR will add SSM OpsItem action to cloudwatch alarm.
The arn format was taken from the alarm UI (in view source) and with all the parameters (severity and category)
closes #16861
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license