fix: Added client_authentication in ignore_changes

terraform-aws-modules / terraform-aws-msk-kafka-cluster

Terraform module to create AWS MSK (Managed Streaming for Kafka) resources 🇺🇦

https://registry.terraform.io/modules/terraform-aws-modules/msk-kafka-cluster/aws

Apache License 2.0

55 stars 53 forks source link

fix: Added client_authentication in ignore_changes #25

Closed vadim-kubasov closed 5 months ago

vadim-kubasov commented 5 months ago

Description

When try to run terraform apply where TF code wasn't changed. Terraform run is failed

Motivation and Context

The change resolves the problem MSK Cluster TLS config.

Breaking Changes

How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects
[x] I have executed pre-commit run -a on my pull request

bryantbiggs commented 5 months ago

it does not resolve the problem, it merely ignores the problem. this needs to be fixed in the provder

carbaugh-politico commented 4 months ago

it does not resolve the problem, it merely ignores the problem. this needs to be fixed in the provder

@bryantbiggs is there an issue tracking the provider fix?

github-actions[bot] commented 3 months ago

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.