Skip to content
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

azure-quickstart-templates/quickstarts/microsoft.azurestackhci/create-cluster /azuredeploy.json needs update #14271

Open
jscc-hv opened this issue Dec 19, 2024 · 1 comment

Comments

@jscc-hv
Copy link

jscc-hv commented Dec 19, 2024

The template worked in the past and recently stopped working. Potentially due to Portal update

Image
template 8-16




Image

Even if the name is properly filled, the error check would not go away, and it prevents deployment.

After inspection, it is due to lack of conditional logic to make clusterWitnessStorageAccountName optional and skip creating a storage account although the template attempts to set a variable clusterWitnessStorageAccountNameVar to "" when witnessType is "No Witness", it never uses a condition on the storage account resource itself.

Suggested fix is to add "condition": "[equals(parameters('witnessType'), 'Cloud')]", to line 549

Image

I can confirm that the conditional logic removes the error.

@rquarry
Copy link

rquarry commented Feb 12, 2025

@jscc-hv Pretty cool you detailed the fix. A lot of issues here seem to be, "this is broken....". I don't think anyone from Microsoft actually makes the updates in this repo. Since these are community provided examples/demos, the onus is on the public to provide updates via pull requests. Suggest you submit a pull request with the update you mentioned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants