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

[CC-1328] Custom cookbooks for monitoring/alerting of EC2 credit balances #384

Open
wants to merge 3 commits into
base: next-release
Choose a base branch
from

Conversation

jfuechsl
Copy link
Contributor

Description of your patch

Add a custom cookbook which sets up monitoring and alerting for EC2 CPU credits and IOPS burst credits for all attached EBS volumes.

Recommended Release Notes

See "Description of your patch"

Estimated risk

Medium. It is a custom cookbook and therefore not included by default. However, when included, it touches the core collectd configuration which is a core component.

Components involved

collectd, python3 (requests, boto3), /engineyard/bin

Description of testing done

Booted an environment with the custom cookbooks uploaded.
Checked if the environment works.
Checked if the metrics get reported.

QA Instructions

Test with different cluster configurations and instance types.
Test with load testing tools (cpu and disk) and check if alerts are triggered and reported properly.
Test upload of custom chef for existing environments.

@dvalfre
Copy link
Contributor

dvalfre commented Feb 15, 2019

@jfuechsl iiuc the PR pushes a change to the core recipes, right? no longer needed then to include it as a custom recipe?

@jfuechsl
Copy link
Contributor Author

@dvalfre Correct, this is now part of the core recipes (collectd). Once we merge this it is available to everyone.

DalianisDim pushed a commit to engineyard/ey-cookbooks-stable-v6 that referenced this pull request May 18, 2019
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.

2 participants