brunopadz / amictl Goto Github PK

All the logic to get the used/not used AMIs today is made through EC2 APIs. Adding support to ASGs would give more security to users when deleting not used AMIs.

Proposal

Implement support of unused AMIs. It'd also be nice to delete by region or delete all AMIs.

References

None.

Although the app is very simple, it'd be cool to add some documentation.

Every application need tests. :D

Describe the solution you'd like
Display AMI info when running amictl aws inspect <AMI_ID>, such as, creation date, size, region and etc...

AWS is the only cloud provider supported. It'd be nice to support more providers such as:

• Azure
• GCP
• Digital Ocean

and so on...

Proposal

Error messages implemented are too simple, like:

Couldn't create an AWS client

or

Couldn't render the results

It'd be nice to improve these messages and/or show the returned error.

References

None

Proposal

In v0.1.0 the flag --cost was introduced to give the user an estimated costs of not used AMIs. In the next version it'd be nice to support infracost.

The main idea is to make requests to an already provisioned/installed infracost and return the costs of those AMIs.

We also need provide some references and examples on how to get started with infracost (how to deploy on Kubernetes or ECS) and how to configure amictl to integrate with it.

References

• Infracost Docs

Proposal

Stylize outputs based on number of unused AMIs.

E.g: If unused AMIs >= 50, print the output in red, if > 20 and < 50 print in yellow or something like that. The main idea is to represent with colors that the number of unused AMIs is not a best practice.

References

As pterm is already used in the project, it'd be nice to use pterm.Println.