Skip to content

Conversation

@trandafo
Copy link
Contributor

@trandafo trandafo commented Nov 18, 2025

Description

Removing a duplicate list [] to fix a bug

Motivation and Context

Fixing bug in v3.0.0 https://github.com//issues/46

Breaking Changes

How Has This Been Tested?

  • I have updated at least one of the examples/* to demonstrate and validate my change(s)
  • I have tested and validated these changes using one or more of the provided examples/* projects

Terraform plan showed no error and displayed the changes expected and apply run with success

module.emr_apu.module.emr-serverless[0].module.emr-serverless-app-task.aws_emrserverless_application.this[0] will be updated in-place

~ resource "aws_emrserverless_application" "this" {
id = "00f******"
name = "core-2-emrapp-apu-eu-west-2"
tags = {
"Application" = "APU"
"Environment" = "2"
"EnvironmentType" = "Test"
"ManagedByTerraform" = "True"
"terraform-aws-modules" = "emr"
}
# (6 unchanged attributes hidden)

  ~ monitoring_configuration {
      ~ cloudwatch_logging_configuration {
          ~ enabled                = false -> true
          + log_group_name         = "/aws/core-2-cwlog-eu-west-2/emr-serverless"
          + log_stream_name_prefix = "agreement-price-upload"
            # (1 unchanged attribute hidden)

          + log_types {
              + name   = "SPARK_DRIVER"
              + values = [
                  + "STDERR",
                  + "STDOUT",
                ]
            }
          + log_types {
              + name   = "SPARK_EXECUTOR"
              + values = [
                  + "STDERR",
                  + "STDOUT",
                ]
            }
        }

        # (1 unchanged block hidden)
    }

  ~ runtime_configuration {
      ~ properties     = {
          ~ "spark.executor.memory"                            = "18G" -> "20G"
            # (7 unchanged elements hidden)
        }
        # (1 unchanged attribute hidden)
    }

    # (7 unchanged blocks hidden)
}
  • I have executed pre-commit run -a on my pull request

Error: Unsupported attribute
│ 
│   on ../terraform-aws-emr/modules/serverless/main.tf line 170, in resource "aws_emrserverless_application" "this":
│  170:       classification = runtime_configuration.value.classification
│     ├────────────────
│     │ runtime_configuration.value is list of object with 1 element
│ 
│ Can't access attributes on a list of objects. Did you mean to access attribute "classification" for a specific element of the list, or across all elements of the list?
╵
╷
│ Error: Unsupported attribute
│ 
│   on ../terraform-aws-emr/modules/serverless/main.tf line 171, in resource "aws_emrserverless_application" "this":
│  171:       properties     = runtime_configuration.value.properties
│     ├────────────────
│     │ runtime_configuration.value is list of object with 1 element
│ 
│ Can't access attributes on a list of objects. Did you mean to access attribute "properties" for a specific element of the list, or across all elements of the list?
╵
Fix runtime configuration for EMR Serverless
@trandafo trandafo changed the title Fix EMR Serverless runtime_configuration double list fix EMR Serverless runtime_configuration double list Nov 18, 2025
@trandafo trandafo changed the title fix EMR Serverless runtime_configuration double list fix: EMR Serverless runtime_configuration double list Nov 18, 2025
@bryantbiggs bryantbiggs changed the title fix: EMR Serverless runtime_configuration double list fix: EMR Serverless runtime_configuration should support multiple configurtions Nov 18, 2025
@bryantbiggs bryantbiggs merged commit 9e11092 into terraform-aws-modules:master Nov 18, 2025
15 of 17 checks passed
antonbabenko pushed a commit that referenced this pull request Nov 18, 2025
## [3.0.1](v3.0.0...v3.0.1) (2025-11-18)

### Bug Fixes

* EMR Serverless `runtime_configuration` should support multiple configurtions  ([#47](#47)) ([9e11092](9e11092))
@antonbabenko
Copy link
Member

This PR is included in version 3.0.1 🎉

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.

Fix EMR Serverless runtime_configuration double list

3 participants