Azure / AKS-Construction

Accelerate your onboarding to AKS with; Helper Web App, bicep templating and CI/CD samples. Flexible & secure AKS baseline implementations in a Microsoft + community maintained reference implementation.
https://azure.github.io/AKS-Construction/
MIT License
356 stars 165 forks source link

Bicep 0.21.1 onwards detects variables with same name but different case, and throws an error #656

Closed pjlewisuk closed 11 months ago

pjlewisuk commented 1 year ago

Bicep has recently upgraded to 0.21.1, and as per the 0.21.1 release notes, a fix was introduce to detect variables with the same name and different cases: "Catch case-insensitive clashes of type property names (https://github.com/Azure/bicep/pull/11457)"

Codespaces are now coming pre-loaded with Bicep 0.21.1+, and this is causing an error to be thrown when trying to deploy clusters via AKS-C:

/workspaces/AKS-Construction/bicep/network.bicep(173,5) : Error BCP353: The variables "aks_podSubnet", "aks_podsubnet" differ only in casing. The ARM deployments engine is not case sensitive and will not be able to distinguish between them.
/workspaces/AKS-Construction/bicep/network.bicep(192,5) : Error BCP353: The variables "aks_podSubnet", "aks_podsubnet" differ only in casing. The ARM deployments engine is not case sensitive and will not be able to distinguish between them.
/workspaces/AKS-Construction/bicep/network.bicep(409,9) : Warning BCP334: The provided value can have a length as small as 0 and may be too short to assign to a target with a configured minimum length of 3.
/workspaces/AKS-Construction/bicep/main.bicep(130,16) : Error BCP104: The referenced module has errors.

Originally posted by @pjlewisuk in https://github.com/Azure/AKS-Construction/issues/633#issuecomment-1713911592

pjlewisuk commented 11 months ago

Fixed in #657