rallytools / vcon Goto Github PK

If the --name flag is not provided when cloning a VM, the generated name is incorrect: snapshot [GUID].

Basic snapshot functionality is in place, but not complete.

The application should be able to report non-critical error conditions in a way that doesn't effect the meaningful output (i.e., the resulting JSON from a clone), but still allows for messages to be conveyed.

This might mean writing these messages to STDERR. Or setting up all commands to return a JSON that has a optional error section. Other ideas are welcome.

It would be very handy to be able to programmatically rename a VM. If it is also trivial, it would be nice to move that VM within the same datastore & datacenter.

The Gopkg.lock and Gopkg.toml files are missing; regenerate and commit these.

Very few commands have long descriptions, which describe the function, options, examples, etc. This information should be provided for every command.

The reportVM function's comments indicate that it reports to the console, which is no longer the case. Clean up these comments, and change function names to accurately reflect intension & purpose.