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

Create DeployMaster role in Security #554

Merged
merged 1 commit into from
Feb 23, 2024

Conversation

diego-ojeda-binbash
Copy link
Contributor

What?

  • Create DeployMaster role in the Security account

Why?

  • This is needed for the NATGW workflows fix

References

@diego-ojeda-binbash diego-ojeda-binbash self-assigned this Feb 23, 2024
@diego-ojeda-binbash diego-ojeda-binbash requested a review from a team as a code owner February 23, 2024 14:39
Copy link

Infracost report

💰 Monthly cost will not change

Cost details
──────────────────────────────────
Project: binbashar/le-tf-infra-aws/apps-devstg/us-east-1/databases-aurora/rds-export-to-s3
Module path: apps-devstg/us-east-1/databases-aurora/rds-export-to-s3
Errors:
  Passed var file does not exist:
    apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.
  Diff baseline error:
    Passed var file does not exist:
      apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.

──────────────────────────────────
Project: binbashar/le-tf-infra-aws/apps-devstg/us-east-1/databases-mysql --/rds-export-to-s3
Module path: apps-devstg/us-east-1/databases-mysql --/rds-export-to-s3
Errors:
  Passed var file does not exist:
    apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.
  Diff baseline error:
    Passed var file does not exist:
      apps-devstg/us-east-1/config/backend.tfvars. Make sure you are passing the var file path relative to the --path flag.

──────────────────────────────────

2541 cloud resources were detected:
∙ 595 were estimated, 456 of which include usage-based costs, see https://infracost.io/usage-file
∙ 1789 were free, rerun with --show-skipped to see details
∙ 157 are not supported yet, rerun with --show-skipped to see details
This comment will be updated when code changes.

@diego-ojeda-binbash diego-ojeda-binbash merged commit 2176c9e into master Feb 23, 2024
6 checks passed
@diego-ojeda-binbash diego-ojeda-binbash deleted the 42-security-deploy-master branch February 23, 2024 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants