Skip to content

[CLOUDGA-31599] [Terraform] Fix - AWS S3 integration doesn't handle optional params.#196

Open
NiyatiSinha-yb wants to merge 1 commit intomainfrom
fix-CLOUDGA-31599
Open

[CLOUDGA-31599] [Terraform] Fix - AWS S3 integration doesn't handle optional params.#196
NiyatiSinha-yb wants to merge 1 commit intomainfrom
fix-CLOUDGA-31599

Conversation

@NiyatiSinha-yb
Copy link
Contributor

@NiyatiSinha-yb NiyatiSinha-yb commented Feb 25, 2026

[CLOUDGA-31599] [Terraform] AWS S3 integration doesn't handle optional params. Apply fails with : inconsistent values for sensitive attribute.

niyatisinha@ns-mbp-hkxf3 devcloud-only % terraform plan          

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # ybm_integration.s3_devcloud will be created
  + resource "ybm_integration" "s3_devcloud" {
      + account_id  = (known after apply)
      + aws_s3_spec = {
          + access_key_id      = (sensitive value)
          + bucket_name        = "sushil.kumar-2"
          + file_prefix        = "helloworld"
          + partition_strategy = (known after apply)
          + path_prefix        = "/"
          + region             = "us-west-2"
          + secret_access_key  = (sensitive value)
        }
      + config_id   = (known after apply)
      + config_name = "s3-telemetry-final-working-test-1003"
      + is_valid    = (known after apply)
      + project_id  = (known after apply)
      + type        = "AWS_S3"
    }

Plan: 1 to add, 0 to change, 0 to destroy.

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

Note: You didn't use the -out option to save this plan, so Terraform can't guarantee to take exactly these actions if you run "terraform apply"
now.
niyatisinha@ns-mbp-hkxf3 devcloud-only % terraform apply

Terraform used the selected providers to generate the following execution plan. Resource actions are indicated with the following symbols:
  + create

Terraform will perform the following actions:

  # ybm_integration.s3_devcloud will be created
  + resource "ybm_integration" "s3_devcloud" {
      + account_id  = (known after apply)
      + aws_s3_spec = {
          + access_key_id      = (sensitive value)
          + bucket_name        = "sushil.kumar-2"
          + file_prefix        = "helloworld"
          + partition_strategy = (known after apply)
          + path_prefix        = "/"
          + region             = "us-west-2"
          + secret_access_key  = (sensitive value)
        }
      + config_id   = (known after apply)
      + config_name = "s3-telemetry-final-working-test-1003"
      + is_valid    = (known after apply)
      + project_id  = (known after apply)
      + type        = "AWS_S3"
    }

Plan: 1 to add, 0 to change, 0 to destroy.

Do you want to perform these actions?
  Terraform will perform the actions described above.
  Only 'yes' will be accepted to approve.

  Enter a value: yes

ybm_integration.s3_devcloud: Creating...
ybm_integration.s3_devcloud: Creation complete after 3s

Apply complete! Resources: 1 added, 0 changed, 0 destroyed.
niyatisinha@ns-mbp-hkxf3 devcloud-only % terraform plan
ybm_integration.s3_devcloud: Refreshing state...

No changes. Your infrastructure matches the configuration.

Terraform has compared your real infrastructure against your configuration and found no differences, so no changes are needed.

…l params. Apply fails with : inconsistent values for sensitive attribute.
@NiyatiSinha-yb NiyatiSinha-yb requested a review from a team as a code owner February 25, 2026 10:22
@NiyatiSinha-yb NiyatiSinha-yb changed the title [CLOUDGA-31599] [Terraform] AWS S3 integration doesn't handle optiona… [CLOUDGA-31599] [Terraform] AWS S3 integration doesn't handle optional params. Feb 25, 2026
@NiyatiSinha-yb NiyatiSinha-yb changed the title [CLOUDGA-31599] [Terraform] AWS S3 integration doesn't handle optional params. [CLOUDGA-31599] [Terraform] Fix - AWS S3 integration doesn't handle optional params. Feb 25, 2026
@NiyatiSinha-yb NiyatiSinha-yb self-assigned this Feb 25, 2026
Copy link
Collaborator

@posriniv posriniv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Copy link
Collaborator

@posriniv posriniv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

SumoLogicSpec *SumoLogicSpec `tfsdk:"sumologic_spec"`
GoogleCloudSpec *GCPServiceAccount `tfsdk:"googlecloud_spec"`
NewRelicSpec *NewRelicSpec `tfsdk:"newrelic_spec"`
AwsS3Spec *AwsS3Spec `tfsdk:"aws_s3_spec"`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@NiyatiSinha-yb This tempState is added specifically to not fail the apply when AWS_S3 FF is disabled in the env.

Adding it in tempState will cause the same issue like https://yugabyte.atlassian.net/browse/CLOUDGA-31409

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.

3 participants