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

Add document about CloudWatch metrics #2350

Merged
merged 5 commits into from Jul 10, 2019
Merged

Conversation

mikena-truss
Copy link
Contributor

Description

I recently added some basic dashboards to CloudWatch. This is here to document and surface their existence. They're super basic right now, but it will be a good experiment to see how useful they can be. I will also add it to the Bat Team docs.

Reviewer Notes

Is there an existing location that would be better for this?
Do we need a more in depth description for this?
Have any resources that would be helpful to list?

* [Staging Dashboard](https://us-west-2.console.aws.amazon.com/cloudwatch/home?region=us-west-2#dashboards:name=mil-staging)
* [Experimental Dashboard](https://us-west-2.console.aws.amazon.com/cloudwatch/home?region=us-west-2#dashboards:name=mil-experimental)

### Updating Dashboards
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@chrisgilmerproj can I get some of your time to create a small guide for this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Absolutely, just grab any time on my calendar.

For what to write here I think it depends on how deep you want to go. I might just say "If you want to preserve changes to one of these dashboards have infrastructure update their terraform configurations." And then you could link them to this file: https://github.com/transcom/ppp-infra/blob/master/modules/aws-app-environment/main.tf#L840-L995. All I do is copy the source from the dashboard, paste it in that file, and update it a bit to use templated variables.

Copy link
Contributor

@chrisgilmerproj chrisgilmerproj left a comment

Choose a reason for hiding this comment

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

🚀

@mikena-truss mikena-truss merged commit caed56f into master Jul 10, 2019
@mikena-truss mikena-truss deleted the mw-add-dashboard-docs branch July 10, 2019 01:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants