-
Notifications
You must be signed in to change notification settings - Fork 103
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
Consistent ID across multiple planning #79
Comments
The problem is made worse by the fact that the usual
|
So, would the sha1_fingerprint be a good ID? It would have the positive side-effect, that it would make a change in this property very visible. |
Any new info, does somebody found solutions to this problem? |
Any updates? |
I solve this problem with an update of Terraform to version "0.14.2". |
@amirba-mistral AFAIU this is the line that sets this id: https://github.com/hashicorp/terraform-provider-tls/blob/v3.0.0/internal/provider/data_source_tls_certificate.go#L109 What is the provider version you are using? |
I also solved this problem with update Terraform to version "0.14.2". |
* Closes hashicorp#139 support parsing certificate content hashicorp#4 * Using hash of certificates as ID for 'tls_certificate' data source * Update CHANGELOG.md * Adding BUGFIX + NOTES to the CHANGELOG as we are now fixing issue hashicorp#79. Co-authored-by: Ivan De Marino <ivan.demarino@hashicorp.com>
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Terraform Version
Affected Resource(s)
Please list the resources as a list, for example:
Terraform Configuration Files
Expected Behavior
Consistent
id
across consecutive plans.Actual Behavior
Resource
id
changes every plan even if certificates are the same.Steps to Reproduce
terraform plan .
terraform apply .
terraform plan .
Important Factoids
Obviously, this is the behavior given by this line.
Is this intentional? Could it be something stable, like the certificate's serial number or sha1 fingerprint?
The text was updated successfully, but these errors were encountered: