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

Fix empty groups, add launch template support and show potential cost savings in the report #117

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

timothy-cloudopsguy
Copy link

Let me know your thoughts. The Launch Template addition is a little more chatty in boto3, due to the extra call you must make to get the actual launch template version. The original pull only retrieves $Latest or $Default to the version which is unhelpful as I believe we need the integer value for the version.

I didn't try to make it neat in terms of a fancy one-liner here, because it got more complex than the original Launch Configuration.

I ran it in my current production and was able to recover ~$1,900/mo in savings. YMMV.

AMIs to be removed:
+--------------------+------------+-----------------------+--------------+
|     Group name     | candidates | EBS Storage recovered | Cost Savings |
+--------------------+------------+-----------------------+--------------+
| no-tags (excluded) |    256     |         19419         |  $1941.9/mo  |
+--------------------+------------+-----------------------+--------------+

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.

1 participant