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

Make SNS notifications human readable #155

Closed
nclemons78 opened this issue Sep 20, 2023 · 4 comments
Closed

Make SNS notifications human readable #155

nclemons78 opened this issue Sep 20, 2023 · 4 comments
Labels
enhancement triaged Has been triaged by solutions team

Comments

@nclemons78
Copy link

Is your feature request related to a problem? Please describe.

We recently upgraded from the original Limit Monitor solution to the new Quota Monitor solution. The original one provided readable emails like:

"AWS-Account : REDACTED || Timestamp : 2022-10-14T01:33:32Z || Limit-Details : {Status:Yellow,Current Usage:810,Limit Name:Roles,Region:-,Service:IAM,Limit Amount:1000}"

The new solution provides emails with JSON contents like:

{"version":"0","id":"REDACTED","detail-type":"Trusted Advisor Check Item Refresh Notification","source":"aws.trustedadvisor","account":"REDACTED","time":"2023-09-20T12:46:31Z","region":"us-east-1","resources":[],"detail":{"check-name":"IAM Roles","check-item-detail":{"Status":"Yellow","Current Usage":"820","Limit Name":"Roles","Region":"-","Service":"IAM","Limit Amount":"1000"},"status":"WARN","resource_id":"","uuid":"REDACTED"}}

Describe the feature you'd like

I would like the emails to be human readable, or suggestions as to what kind of technology this is designed to be consumed by, because JSON via email doesn't really make much sense to me.

@abewub abewub added the triaged Has been triaged by solutions team label Sep 20, 2023
@rakshb rakshb changed the title Limit Monitor -> Quota Monitor Make SNS notifications human readable Jan 11, 2024
@abewub
Copy link
Member

abewub commented Jan 24, 2024

Thanks, we have added the request to our backlog.

@sanjay-reddy-kandi
Copy link
Member

Hi, We are planning to resolve this issue as part of the next release. What is the human readable format you got in mind? Currently I am thinking of the below. Please let me know if you have any suggestions. Thanks

Quota Monitor Alert

Check Name:
Status:
Time: 
Account: 
Region: 

Details:
- Current Usage: 
- Limit Name: 
- Service: 
- Limit Amount:

@nclemons78
Copy link
Author

That looks good to me, thank you!

@sanjay-reddy-kandi
Copy link
Member

Feature implemented in v6.3.0. Closing issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement triaged Has been triaged by solutions team
Projects
Development

No branches or pull requests

4 participants