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

[BUG] opensearch_channel_configuration config_id field change detected in TF plans #88

Open
akennealy opened this issue Sep 28, 2023 · 6 comments
Assignees
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@akennealy
Copy link

akennealy commented Sep 28, 2023

What is the bug?

Persistent Terraform plan change detected with field config_id

How can one reproduce the bug?

  1. Create a new opensearch_channel_configuration resource
resource "opensearch_channel_configuration" "escalations" {
  body = <<EOF
{
  "config": {
    "name": "name",
    "description": "description",
    "config_type": "slack",
    "is_enabled": true,
    "slack": {
      "url": "${var.slack_escalations_channel_webhook}"
    }
  }
}
EOF
}
  1. Plan and apply
  2. Then trigger a new plan
  3. Bug: The plan will try to remove the config_id field (That was added by OpenSearch dashboard)

What is the expected behavior?

No changes detected in plan

What is your host/environment?

Operating system, version.

Do you have any screenshots?

Screenshot of TF cloud UI showing diff

image

Do you have any additional context?

Add any other context about the problem.

@akennealy akennealy added bug Something isn't working untriaged labels Sep 28, 2023
@prudhvigodithi prudhvigodithi added good first issue Good for newcomers and removed untriaged labels Oct 9, 2023
@prudhvigodithi
Copy link
Collaborator

[Untriage]
Hey @akennealy I assume you have tested with latest supported release of the provider?
The opensearch_channel_configuration configuration was introduced by @alexconlin #68, @alexconlin can you please a look and see whats this bug about ?
Adding @rblcoder @afrodidact @premkirank
Thank you

@akennealy
Copy link
Author

@prudhvigodithi yes Im on the latest version.

@afrodidact
Copy link

Hi @prudhvigodithi, can you assign this issue to me?

@prudhvigodithi
Copy link
Collaborator

Thanks assigned to your @afrodidact.

@afrodidact
Copy link

Thanks!

@afrodidact
Copy link

I’ve implemented a fix for this in the pull request here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
Status: Backlog
Development

No branches or pull requests

3 participants