Error creating Batch Pool from CLI template

themorey commented 3 years ago

This is autogenerated. Please review and update as needed.

Describe the bug

Command Name az batch pool create Extension Name: azure-batch-cli-extensions. Version: 6.0.0.

Errors:

Unexpect element taskSlotsPerNode in template
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 231, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 657, in execute
    raise ex
...
azure/cli/core/__init__.py, ln 807, in default_command_handler
    return op(**command_args)
cliextensions/azure-batch-cli-extensions/azext_batch/custom.py, ln 66, in create_pool
    templates.validate_json_object(json_obj, pool)
azure-batch-cli-extensions/azext/batch/_template_utils.py, ln 1203, in validate_json_object
    raise ValueError('Unexpect element {} in template'.format(item))
ValueError: Unexpect element taskSlotsPerNode in template

To Reproduce:

Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az batch pool create --template {}

Expected Behavior

Environment Summary

Linux-4.15.0-1113-azure-x86_64-with-debian-10.2 (Cloud Shell)
Python 3.6.10
Installer: DEB

azure-cli 2.22.0 *

Extensions:
azure-batch-cli-extensions 6.0.0
aks-preview 0.4.64
hpc-cache 0.1.0
ai-examples 0.2.5
ssh 0.1.4

Additional Context

bitsofinfo commented 3 years ago

@bgklein who maintains this repo? see this error has been here since april and I just ran into it as well

bitsofinfo commented 3 years ago

@themorey you ever solve this?

themorey commented 3 years ago

Yes the template SDK uses an older version of Batch API and taskSlotsPerNode is not compatible. Use the older parameter maxTasksPerNode instead.

bitsofinfo commented 3 years ago

thanks @themorey can you perhaps share a sample? I tried and can't seem to get maxTasksPerNode working either: see #132

dpwatrous commented 3 years ago

Just a quick update that this is on our radar and we do plan to release an updated version of the CLI extensions which will include support for taskSlotsPerNode. Until then I'll see if we can update the docs to reflect the right version of the API.

fblampe commented 2 years ago

It's been another couple months and this still seems to be an issue... :(

dpwatrous commented 2 years ago

Unfortunately the developer working on this is out of the office until 4/11, but I'll follow up with him then and see if we can't unblock a new release.

Azure / azure-batch-cli-extensions