arista-eosplus / packer-veos

Packer.io automation of the vEOS nodes
30 stars 12 forks source link

Documentation is not quite accurate #5

Closed advornic closed 9 years ago

advornic commented 9 years ago
$ ./create-veos.py --help
usage: create-veos.py [-h]
[-n {vEOS1,vEOS2,vEOS3,vEOS4} [{vEOS1,vEOS2,vEOS3,vEOS4} ...]]
[-b BOOTTIME] [-N VMNAME]
{vmware,virtualbox}'

suggests that this would work:

$ ./create-veos.py -N ztps -n vEOS1 -n vEOS2 -n vEOS3 -n vEOS4 virtualbox
usage: create-veos.py [-h]
                      [-n {vEOS1,vEOS2,vEOS3,vEOS4} [{vEOS1,vEOS2,vEOS3,vEOS4} ...]]
                      [-b BOOTTIME] [-N VMNAME]
                      {vmware,virtualbox}
create-veos.py: error: argument -n/--nodes: invalid choice: 'virtualbox' (choose from 'vEOS1', 'vEOS2', 'vEOS3', 'vEOS4')

The order or arguments should not matter anyhow - we need to improve this.

Workaround for now is to specify the hypervisor at the beginning of the argument list.

advornic commented 9 years ago

Same applies to packer-ztpserver.