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

Feature | Merge EKS clusters into one #659

Open
wants to merge 15 commits into
base: master
Choose a base branch
from

Conversation

angelofenoglio
Copy link
Contributor

@angelofenoglio angelofenoglio commented Nov 26, 2024

What?

  • Merge EKS clusters into one so that we have a single reference for the Leverage EKS Platform

References

@angelofenoglio angelofenoglio requested a review from a team as a code owner November 26, 2024 21:10
@angelofenoglio angelofenoglio changed the title Feature merge eks clusters Feature | Merge EKS clusters into one Nov 26, 2024
Copy link

github-actions bot commented Nov 26, 2024

💰 Infracost report

Monthly estimate increased by $0.80 📈

Changed project Baseline cost Usage cost* Total change New monthly cost
binbashar/le-tf-infra-aws/shared/us-east-1/secrets-manager +$0.80 - +$0.80 (+50%) $2

*Usage costs can be estimated by updating Infracost Cloud settings, see docs for other options.

Estimate details (includes details of unsupported resources and skipped projects due to errors)
Key: * usage cost, ~ changed, + added, - removed

──────────────────────────────────
Project: shared-us-east-1-secrets-manager
Module path: shared/us-east-1/secrets-manager

+ module.secrets.aws_secretsmanager_secret.sm["/devops/monitoring/alertmanager"]
  +$0.40

    + Secret
      +$0.40

    + API requests
      Monthly cost depends on usage
        +$0.05 per 10k requests

+ module.secrets.aws_secretsmanager_secret.sm["/devops/monitoring/grafana/administrator"]
  +$0.40

    + Secret
      +$0.40

    + API requests
      Monthly cost depends on usage
        +$0.05 per 10k requests

Monthly cost change for binbashar/le-tf-infra-aws/shared/us-east-1/secrets-manager (Module path: shared/us-east-1/secrets-manager)
Amount:  +$0.80 ($2 → $2)
Percent: +50%

──────────────────────────────────
Key: * usage cost, ~ changed, + added, - removed
131 projects have no cost estimate changes.
Run the following command to see their breakdown: infracost breakdown --path=/path/to/code

──────────────────────────────────
*Usage costs can be estimated by updating Infracost Cloud settings, see docs for other options.

2625 cloud resources were detected:
∙ 598 were estimated
∙ 1903 were free
∙ 124 are not supported yet, see https://infracost.io/requested-resources:
  ∙ 47 x aws_identitystore_group_membership
  ∙ 38 x aws_identitystore_user
  ∙ 10 x aws_guardduty_member
  ∙ 7 x aws_identitystore_group
  ∙ 5 x aws_fms_policy
  ∙ 4 x aws_guardduty_detector
  ∙ 2 x aws_guardduty_organization_admin_account
  ∙ 2 x aws_guardduty_organization_configuration
  ∙ 2 x aws_organizations_delegated_administrator
  ∙ 1 x aws_cloudtrail_organization_delegated_admin_account
  ∙ 1 x aws_eks_access_entry
  ∙ 1 x aws_fms_admin_account
  ∙ 1 x aws_organizations_organization
  ∙ 1 x aws_route53_resolver_firewall_domain_list
  ∙ 1 x aws_route53_resolver_firewall_rule
  ∙ 1 x aws_route53_resolver_firewall_rule_group

Infracost estimate: Monthly estimate increased by $0.80 ↑
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━┓
┃ Changed project                                            ┃ Baseline cost ┃ Usage cost* ┃ Total change  ┃
┣━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━╋━━━━━━━━━━━━━╋━━━━━━━━━━━━━━━┫
┃ binbashar/le-tf-infra-aws/shared/us-east-1/secrets-manager ┃        +$0.80 ┃           - ┃ +$0.80 (+50%) ┃
┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━┻━━━━━━━━━━━━━┻━━━━━━━━━━━━━━━┛
This comment will be updated when code changes.

@angelofenoglio angelofenoglio self-assigned this Nov 26, 2024
@angelofenoglio angelofenoglio added enhancement New feature or request minor labels Nov 26, 2024
@angelofenoglio angelofenoglio changed the base branch from feat/deploy-emojivoto-in-devstg-cluster to master November 26, 2024 21:12
@angelofenoglio angelofenoglio force-pushed the feature/merge-eks-clusters branch from 5072576 to 4cbdd68 Compare November 26, 2024 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement | Consolidate k8s-eks layers that must result in a single layer
2 participants