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

feature: allow monitoring multiple RDS instances #33

Merged
merged 28 commits into from
Jul 3, 2022

Conversation

joe-niland
Copy link
Member

@joe-niland joe-niland commented Jun 20, 2021

what

  • adds instance prefix to SNS alarms

why

  • Otherwise multiple instances overwrite each other

references

@joe-niland joe-niland marked this pull request as ready for review June 21, 2021 00:15
@joe-niland joe-niland requested review from a team as code owners June 21, 2021 00:15
@joe-niland joe-niland requested review from dotCipher and woz5999 and removed request for a team June 21, 2021 00:15
@joe-niland joe-niland changed the title fix: TF 0.12 test harness feature: allow monitoring multiple RDS instances Jun 21, 2021
@joe-niland
Copy link
Member Author

/test all

@joe-niland
Copy link
Member Author

/test bats

@joe-niland
Copy link
Member Author

/test terratest

@joe-niland
Copy link
Member Author

/test bats

@JohnShortland
Copy link

JohnShortland commented Dec 9, 2021

@joe-niland Can you run /test bats again as the results have expired, would be interested to see why this is failing.

@joe-niland
Copy link
Member Author

/test bats

@JohnShortland
Copy link

@joe-niland I think its an issue with the test itself, isn't it? Can't configure provider.

@joe-niland
Copy link
Member Author

/test bats

@joe-niland
Copy link
Member Author

@JohnShortland yes that's true. Looking into it.

@bg46z
Copy link

bg46z commented May 16, 2022

bump

@mergify
Copy link

mergify bot commented Jul 2, 2022

This pull request is now in conflict. Could you fix it @joe-niland? 🙏

@joe-niland joe-niland requested a review from a team as a code owner July 2, 2022 02:02
@joe-niland
Copy link
Member Author

/test all

@joe-niland
Copy link
Member Author

/test bats

@joe-niland
Copy link
Member Author

/test bats

@joe-niland
Copy link
Member Author

/test all

@joe-niland
Copy link
Member Author

/test all

alarms.tf Show resolved Hide resolved
@joe-niland
Copy link
Member Author

/test all

@joe-niland joe-niland merged commit d575e86 into cloudposse:master Jul 3, 2022
@joe-niland joe-niland deleted the DevopsCare-master branch July 3, 2022 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

alarm names should be unique
7 participants