Change position of the node-id argument #272

Merged

lx merged 1 commit 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

Change position of the node-id argument

continuous-integration/drone/pr Build is passing

Details

continuous-integration/drone/push Build is passing

Details

15da2156f6

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 ```

No reviewers

No milestone

No project

No assignees

3 participants

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Deuxfleurs/garage#272

No description provided.