-
Notifications
You must be signed in to change notification settings - Fork 184
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
Managed app ARM templates failing while validation with latest version of ARM-TTK #703
Comments
If a full copy of ARM is required, the sample can be found on below URL, since that too is failing and there as well we have vmSize as a parameter: https://learn.microsoft.com/en-us/azure/aks/learn/quick-kubernetes-deploy-rm-template?tabs=azure-cli Thanks, |
It appears we are not allowing the VMSize parameter to be anything other than VMSize: I believe we should adjust this. But how? Should we require it to be named anything particular? If not, what heuristic do we use to determine if the top-level parameter is a VMSize? |
It should have no naming constraints - the parameter can be named anything. We just need to verify that any resource property named |
This should be fixed by recent changes to the VMSize tests. Please confirm when you get the chance. |
Hi Team, Hope you are doing good.
Since the upgrade of ARM-TTK our template validation is failing. If I remember it correctly with v0.15 I never encountered this issue.
Also, I believe the reason is wrong or misleading since we already are using the practices which are shared over the documentation.
Ref: https://learn.microsoft.com/en-us/azure/azure-resource-manager/templates/template-test-cases
Section: VM size uses parameter
Error:
We are already using vmSize as a parameter, please find below snippets from our respective arm templates:
createUiDefinition.json [Ref: https://learn.microsoft.com/en-us/azure/azure-resource-manager/managed-applications/microsoft-compute-sizeselector]
mainTemplate's parameters:
mainTemplate's resources (this is a part of AKS agent pool):
Could you please suggest OR correct me if I am doing something wrong?
Thank you,
Sunny
The text was updated successfully, but these errors were encountered: