On Fri, Aug 28, 2015 at 01:47:39PM +0000, Eren Yagdiran wrote: > Commandline parameters for running a template > --- > virt-sandbox-image/virt-sandbox-image.py | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/virt-sandbox-image/virt-sandbox-image.py b/virt-sandbox-image/virt-sandbox-image.py > index 1da5150..d6b682f 100755 > --- a/virt-sandbox-image/virt-sandbox-image.py > +++ b/virt-sandbox-image/virt-sandbox-image.py > @@ -178,6 +178,18 @@ def gen_create_args(subparser): > help=_("format format for image")) > parser.set_defaults(func=create) > > +def gen_run_args(subparser): > + parser = subparser.add_parser("run", > + help=_("Run a already built image")) > + requires_name(parser) > + requires_source(parser) > + requires_connect(parser) > + parser.add_argument("-t","--template-dir", > + help=_("Template directory for saving templates")) > + parser.add_argument("-i","--igniter", > + help=_("Igniter command for image")) Rather than having an --igniter arg, which only allows a single binary name, we should just allow an arbitrary number of positional args, so the user can pass full arg lists, eg virt-sandbox-image run ubuntu /bin/somefile somearg otherarg .... Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://autobuild.org -o- http://search.cpan.org/~danberr/ :| |: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list