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: bump Datadog module to one with fixed dependencies #211

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

stefanwb
Copy link
Contributor

@stefanwb stefanwb commented Oct 2, 2024

Signed-off-by: Stefan Wessels Beljaars swesselsbeljaars@schubergphilis.com

Signed-off-by: Stefan Wessels Beljaars <swesselsbeljaars@schubergphilis.com>
@github-actions github-actions bot added the bug Something isn't working label Oct 2, 2024
Copy link
Contributor

github-actions bot commented Oct 2, 2024

Terraform Format and Style 🖌success

Terraform Initialization ⚙️``

Terraform Lint 📖success

Terraform Validation 🤖failure

Validation Output

�[0m�[1mInitializing the backend...�[0m
�[0m�[1mInitializing modules...�[0m
- landing_zone in ../..
Downloading registry.terraform.io/schubergphilis/mcaf-s3/aws 0.14.1 for landing_zone.audit_manager_reports...
- landing_zone.audit_manager_reports in .terraform/modules/landing_zone.audit_manager_reports
Downloading registry.terraform.io/schubergphilis/mcaf-s3/aws 0.14.1 for landing_zone.aws_config_s3...
- landing_zone.aws_config_s3 in .terraform/modules/landing_zone.aws_config_s3
- landing_zone.aws_sso_permission_sets in ../../modules/permission-set
Downloading registry.terraform.io/schubergphilis/mcaf-kms/aws 0.3.0 for landing_zone.kms_key...
- landing_zone.kms_key in .terraform/modules/landing_zone.kms_key
Downloading registry.terraform.io/schubergphilis/mcaf-kms/aws 0.3.0 for landing_zone.kms_key_audit...
- landing_zone.kms_key_audit in .terraform/modules/landing_zone.kms_key_audit
Downloading registry.terraform.io/schubergphilis/mcaf-kms/aws 0.3.0 for landing_zone.kms_key_logging...
- landing_zone.kms_key_logging in .terraform/modules/landing_zone.kms_key_logging
Downloading registry.terraform.io/schubergphilis/mcaf-ses/aws 0.1.4 for landing_zone.ses-root-accounts-mail-alias...
- landing_zone.ses-root-accounts-mail-alias in .terraform/modules/landing_zone.ses-root-accounts-mail-alias
Downloading git::https://github.com/schubergphilis/terraform-aws-mcaf-user.git?ref=v0.1.12 for landing_zone.ses-root-accounts-mail-alias.smtp_users...
- landing_zone.ses-root-accounts-mail-alias.smtp_users in .terraform/modules/landing_zone.ses-root-accounts-mail-alias.smtp_users
Downloading registry.terraform.io/schubergphilis/mcaf-ses-forwarder/aws 0.3.1 for landing_zone.ses-root-accounts-mail-forward...
- landing_zone.ses-root-accounts-mail-forward in .terraform/modules/landing_zone.ses-root-accounts-mail-forward
Downloading registry.terraform.io/schubergphilis/mcaf-lambda/aws 1.4.1 for landing_zone.ses-root-accounts-mail-forward.lambda...
- landing_zone.ses-root-accounts-mail-forward.lambda in .terraform/modules/landing_zone.ses-root-accounts-mail-forward.lambda
Downloading git::https://github.com/schubergphilis/terraform-aws-mcaf-role.git?ref=v0.3.3 for landing_zone.ses-root-accounts-mail-forward.lambda.lambda_role...
- landing_zone.ses-root-accounts-mail-forward.lambda.lambda_role in .terraform/modules/landing_zone.ses-root-accounts-mail-forward.lambda.lambda_role
Downloading registry.terraform.io/schubergphilis/mcaf-s3/aws 0.14.1 for landing_zone.ses-root-accounts-mail-forward.s3_bucket...
- landing_zone.ses-root-accounts-mail-forward.s3_bucket in .terraform/modules/landing_zone.ses-root-accounts-mail-forward.s3_bucket
- landing_zone.tag_policy_assignment in ../../modules/tag-policy-assignment
�[33m╷�[0m�[0m
�[33m│�[0m �[0m�[1m�[33mWarning: �[0m�[0m�[1mReference to undefined provider�[0m
�[33m│�[0m �[0m
�[33m│�[0m �[0m�[0m  on ../../ses_accounts_mail_alias.tf line 20, in module "ses-root-accounts-mail-forward":
�[33m│�[0m �[0m  20:   providers = { aws = aws, �[4maws�[0m.lambda = aws }�[0m
�[33m│�[0m �[0m
�[33m│�[0m �[0mThere is no explicit declaration for local provider name "aws.lambda" in
�[33m│�[0m �[0mmodule.landing_zone.module.ses-root-accounts-mail-forward, so Terraform is
�[33m│�[0m �[0massuming you mean to pass a configuration for "hashicorp/aws".
�[33m│�[0m �[0m
�[33m│�[0m �[0mIf you also control the child module, add a required_providers entry named
�[33m│�[0m �[0m"aws.lambda" with the source address "hashicorp/aws".
�[33m╵�[0m�[0m

@stefanwb stefanwb merged commit 84d69a8 into master Oct 2, 2024
@stefanwb stefanwb deleted the fix-depending-version branch October 2, 2024 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants