Change position of the node-id argument #272

Merged

lx merged 1 commits from Steam/garage:main into main

2022-03-21 10:57:11 +00:00

Steam commented

2022-03-19 17:06:08 +00:00

Contributor

If the command is entered as in your the example, the following error message appears:

error: The following required arguments were not provided:
    <node-id>

USAGE:
    garage layout assign <node-id> --capacity <capacity> --tag <tags>... --zone <zone>

After moving the <node-id> argument to the first position it works without problems

If the command is entered as in your the example, the following error message appears: ``` error: The following required arguments were not provided: <node-id> USAGE: garage layout assign <node-id> --capacity <capacity> --tag <tags>... --zone <zone> ``` After moving the `<node-id>` argument to the first position it works without problems

Steam added 1 commit 2022-03-19 17:06:11 +00:00

continuous-integration/drone/pr Build is passing Details

continuous-integration/drone/push Build is passing Details

15da2156f6 Change position of the node-id argument

lx commented

2022-03-21 10:56:48 +00:00

Owner

Thanks!

lx merged commit 15da2156f6 into main

2022-03-21 10:57:09 +00:00

quentin commented

2022-03-21 11:12:09 +00:00

Owner

For the record, the problem was due to --tag capturing the <node-id> parameter.
Other possible syntaxes are:

# use a double dash to signal the end of the tag list
garage layout assign -z par1 -c 10 -t mercury -- 563e

# declare the tags before another parameter (here the capacity), so there is no ambiguity
garage layout assign -z par1 -t mercury -c 10 563e

For the record, the problem was due to `--tag` capturing the `<node-id>` parameter. Other possible syntaxes are: ```bash # use a double dash to signal the end of the tag list garage layout assign -z par1 -c 10 -t mercury -- 563e # declare the tags before another parameter (here the capacity), so there is no ambiguity garage layout assign -z par1 -t mercury -c 10 563e ```