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

Test #97

Closed
wants to merge 7 commits into from
Closed

Test #97

wants to merge 7 commits into from

Conversation

alikhajeh1
Copy link
Member

No description provided.

Copy link

github-actions bot commented Feb 16, 2024

Infracost report

💰 Monthly cost will decrease by $90 📉

Project Cost change New monthly cost
infracost/gh-actions-demo/terraform -$90 (-46%) $107
Cost details
──────────────────────────────────
Project: infracost/gh-actions-demo/terraform
Module path: terraform

~ aws_instance.my_web_app
  -$90 ($197 → $107)

    ~ root_block_device
    
        ~ Storage (general purpose SSD, gp2)
          -$90 ($100 → $10)

Monthly cost change for infracost/gh-actions-demo/terraform (Module path: terraform)
Amount:  -$90 ($197 → $107)
Percent: -46%

──────────────────────────────────
Key: ~ changed, + added, - removed

2 cloud resources were detected:
∙ 2 were estimated, all of which include usage-based costs, see https://infracost.io/usage-file

Infracost estimate: Monthly cost will decrease by $90 ↓
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━┓
┃ Project                                            ┃ Cost change ┃ New monthly cost ┃
┣━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━━━━┫
┃ infracost/gh-actions-demo/terraform                ┃ -$90 (-46%) ┃ $107             ┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━━━━┛

Governance checks

🔴 3 failures
EBS - consider upgrading gp2 volumes to gp3: gp3 volumes are the latest generation of general-purpose SSD-based EBS volumes that enable you to provision performance independent of storage capacity, while providing up to 20% lower price per GB than existing gp2 volumes. With gp3 volumes, you can scale IOPS (input/output operations per second) and throughput without needing to provision additional block storage capacity. This means you only pay for the storage you need.

aws_instance.my_web_app at terraform/main.tf:5

  • Set root_block_device.volume_type to gp3.

in project infracost/gh-actions-demo/terraform

EC2 - consider using latest generation instances for m family instances: Upgrade to the Amazon EC2 m5 instance family to get a higher performing CPU with support for AVX-512 instructions, up to 384 GiB of RAM, and up to 25 Gbps of networking bandwidth, all at a lower price point than previous generation instances. For example an `m3.large` instance with 7.5 GiB of memory and 2 vCPUs has a monthly cost of $97. Whereas a `m5.large` instance with 8 GiB of memory and 2 vCPUs costs just $70. That's a 27% saving for a more performant machine.

aws_instance.my_web_app at terraform/main.tf:5

  • Switch instance_type from m3.large to m5.large

in project infracost/gh-actions-demo/terraform

FinOps tags: FinOps tags enable us to report on and optimize cloud costs, please see https://acme-inc.atlassian.com/finops/tagging for more information.

aws_instance.my_web_app at terraform/main.tf:5

  • Environment has invalid value production. Must be one of Dev, Stage, Prod
  • Missing mandatory tags: Tst, root_block_device.Service, root_block_device.Environment, root_block_device.Tst

in project infracost/gh-actions-demo/terraform

🟢 47 passed
46 FinOps policies, 0 Tagging policies, and 1 Guardrail passed.

View in Infracost Cloud. This comment will be updated when code changes.

@alikhajeh1 alikhajeh1 closed this Feb 16, 2024
@alikhajeh1 alikhajeh1 reopened this Feb 16, 2024
@alikhajeh1 alikhajeh1 closed this Feb 16, 2024
@alikhajeh1 alikhajeh1 reopened this Feb 16, 2024
@alikhajeh1 alikhajeh1 closed this Feb 16, 2024
@alikhajeh1 alikhajeh1 reopened this Feb 16, 2024
@alikhajeh1 alikhajeh1 closed this Feb 16, 2024
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