-
Notifications
You must be signed in to change notification settings - Fork 469
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: backwards incompatible TLS provider restriction #228
Comments
@matelang Thanks for opening this issue! You're right, the restriction should be |
Thanks @lonegunmanb for the ⚡ fix! |
Could 5.1.0 be released please - so that we can get this fix? |
@davidkarlsen in the meantime you can import the module using an https provider source instead of terraform registry: # source = "Azure/aks/azurerm"
# version = "5.x.y"
# Change back after a version greater than 5.0.0 has been released
source = "git::https://github.com/Azure/terraform-azurerm-aks.git?ref=master" |
Description
After the v5 release, a TLS provider restriction has been introduced to the codebase, namely TLS provoder version
~> 3.1
.See: https://github.com/Azure/terraform-azurerm-aks/blob/5.0.0/versions.tf
Please note that there is TLS provider version 4.x already available for a while now, and there are users of this AKS module who are potentially already using AKS module v4 (even in production) with the newer TLS provider (v4). Since there is a restriction in Terraform, that the older TLS provider (v3) is not able to manage the resource managed by the newer TLS provider (v4) the Terraform workspaces in this situation are unable to run to completion (failing at initialization).
The AKS module's version 4 did not have any restrictions on the TLS provider version:
https://github.com/Azure/terraform-azurerm-aks/blob/4.16.0/versions.tf
Even if it is best practice to have version constraints, I recommend also including v4 TLS provider as an option, since there is not much change in the key resource being managed by the provider code.
Thanks.
The text was updated successfully, but these errors were encountered: