klich
Enthusiast
Enthusiast

vmclone.pl missing validation for required arguments

So, I found this one the hard way. Passed my command-line parameters to vmclone.pl, and it just wouldn't work.

Turns out, the required parameter, vmhost_destination is not included within %args for validation.

Please add that as an item to correct.

Here is what I added:

vmhost_destination => {

type => "=s",

help => "The name of the host to place the target vm",

required => 1,

},

0 Kudos
2 Replies
SaranshG
Enthusiast
Enthusiast

Not sure what you are looking for. Is this a suggestion to add a parameter (vmhost_destination) to vmclone.cl file of you need find out how to add a parameter? Please give more details

0 Kudos
klich
Enthusiast
Enthusiast

This post was simply to notify of an issue with the sample vmclone.pl script, that should be corrected.

The parameter is required (per the usage dialog), however the script does not check for the requirement. The code I listed details how I corrected the issue.

Please correct this in the next release, so others trying to evaluate the toolkit do not run into any errors.

Thank you.

0 Kudos