You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Bug Description
The AWS documentation says that account_customizations_name is optional, and so it has been every time I've deployed AFT for a client, but now leaving it out yields an error message.
To Reproduce
Steps to reproduce the behavior:
Submit an account provisioning request not containing account_customizations_name
Expected behavior
I expect an account to be created or updated.
Related Logs
This is the log from ct-aft-account-request, step terraform-apply:
[Container] 2024/09/19 11:41:10.336878 Phase complete: PRE_BUILD State: SUCCEEDED
343 | [Container] 2024/09/19 11:41:10.336901 Phase context status code: Message:
344 | [Container] 2024/09/19 11:41:10.389621 Entering phase BUILD
345 | [Container] 2024/09/19 11:41:10.390509 Running command if [ $TF_DISTRIBUTION = "oss" ]; then
346 | terraform apply -no-color --auto-approve
347 | fi
348 |
349 |
350 | Error: Invalid format of "item": unexpected raw attribute type (<nil>) for data type descriptor: S
351 |
352 | with module.Security-Adm.aws_dynamodb_table_item.account-request,
353 | on modules/aft-account-request/ddb.tf line 5, in resource "aws_dynamodb_table_item" "account-request":
354 | 5: item = jsonencode({
355 | 6: id = { S = lookup(var.control_tower_parameters, "AccountEmail") }
356 | 7: control_tower_parameters = { M = {
357 | 8: AccountEmail = { S = lookup(var.control_tower_parameters, "AccountEmail") }
358 | 9: AccountName = { S = lookup(var.control_tower_parameters, "AccountName") }
359 | 10: ManagedOrganizationalUnit = { S = lookup(var.control_tower_parameters, "ManagedOrganizationalUnit") }
360 | 11: SSOUserEmail = { S = lookup(var.control_tower_parameters, "SSOUserEmail") }
361 | 12: SSOUserFirstName = { S = lookup(var.control_tower_parameters, "SSOUserFirstName") }
362 | 13: SSOUserLastName = { S = lookup(var.control_tower_parameters, "SSOUserLastName") }
363 | 14: }
364 | 15: }
365 | 16: change_management_parameters = { M = {
366 | 17: change_reason = { S = lookup(var.change_management_parameters, "change_reason") }
367 | 18: change_requested_by = { S = lookup(var.change_management_parameters, "change_requested_by") }
368 | 19: }
369 | 20: }
370 | 21: account_tags = { S = jsonencode(var.account_tags) }
371 | 22: account_customizations_name = { S = var.account_customizations_name }
372 | 23: custom_fields = { S = jsonencode(var.custom_fields) }
373 | 24: })
374 |
375 |
376 | [Container] 2024/09/19 11:41:14.312700 Command did not exit successfully if [ $TF_DISTRIBUTION = "oss" ]; then
377 | terraform apply -no-color --auto-approve
378 | fi
379 | exit status 1
380 | [Container] 2024/09/19 11:41:14.318628 Phase complete: BUILD State: FAILED
381
Additional context
Line 350 above refers to account_customizations_name. Unless it is provided, the pipeline step fails.
The text was updated successfully, but these errors were encountered:
Terraform Version & Prov:
oss 1.9.6
AFT Version:
1.13.1
Terraform Version & Provider Versions
Please provide the outputs of
terraform version
andterraform providers
from within your AFT environmentterraform version
terraform providers
Bug Description
The AWS documentation says that
account_customizations_name
is optional, and so it has been every time I've deployed AFT for a client, but now leaving it out yields an error message.To Reproduce
Steps to reproduce the behavior:
account_customizations_name
Expected behavior
I expect an account to be created or updated.
Related Logs
This is the log from ct-aft-account-request, step
terraform-apply
:Additional context
Line 350 above refers to
account_customizations_name
. Unless it is provided, the pipeline step fails.The text was updated successfully, but these errors were encountered: