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

example JSON in doc results in : ValueError: Unexpect element taskSlotsPerNode in template #76519

Closed
bitsofinfo opened this issue Jun 7, 2021 · 6 comments

Comments

@bitsofinfo
Copy link

bitsofinfo commented Jun 7, 2021

the example shows you can use taskSlotsPerNode but we get this error trying to specify that ValueError: Unexpect element taskSlotsPerNode in template


Document Details

Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

@PRMerger15 PRMerger15 added the Pri3 label Jun 7, 2021
@mikeurnun
Copy link
Contributor

Hi @bitsofinfo - Thank you for sharing your feedback! Since this channel is for driving improvements towards MS Docs, could you provide us with a specific URL of the MS Doc that your feedback is most applicable to? That way, we're able to connect you with the right team that is best suited to address your feedback.

@mikeurnun
Copy link
Contributor

@bitsofinfo Just following up on my comment above. Let us know if there is a doc that correlates to your feedback, otherwise, we will proceed to close this issue for now.

@bitsofinfo
Copy link
Author

https://docs.microsoft.com/en-us/azure/batch/batch-cli-templates

@bitsofinfo
Copy link
Author

@JnHs
Copy link
Contributor

JnHs commented Jun 14, 2021

Hi @bitsofinfo - thanks so much for raising this issue. After investigation it looks like taskSlotsPerNode is not yet supported in Azure CLI. I have updated the document to use maxTasksPerNode instead (as it did previously), which should work. Once support for taskSlotsPerNode is available we'll update this again. I believe this should address the issue, but if you are still experiencing any issues or have further feedback please let us know. Thanks again for sharing your feedback!
#please-close

@fblampe
Copy link

fblampe commented Apr 1, 2022

The linked doc now mentions "taskSlotsPerNode": 1, again, which still does not work.
Azure/azure-batch-cli-extensions#132 has not changed since this issue was closed.
Can someone please provide a working sample template that will configure taskSlotsPerNode?

Tested with
azure-cli 2.34.1
azure-batch-cli-extensions 6.0.0

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

No branches or pull requests

7 participants