On Thu, Feb 21, 2013 at 9:55 AM, Peter Krempa <pkrempa@xxxxxxxxxx> wrote: > On 02/18/13 06:11, Doug Goldstein wrote: >> >> On Fri, Jan 11, 2013 at 2:51 PM, Guido Günther <agx@xxxxxxxxxxx> wrote: >>> >>> On Tue, Jan 08, 2013 at 04:41:58PM -0700, Eric Blake wrote: >>>> >>>> On 01/08/2013 02:36 PM, Doug Goldstein wrote: >>>>> >>>>> I often find myself doing virsh "define blah.xml; start blah". I >>>>> figured >>>>> adding this would be a easier^Hlazier way to do it. >>>>> --- >>>>> tools/virsh-domain.c | 20 ++++++++++++++++++-- >>>>> 1 files changed, 18 insertions(+), 2 deletions(-) >>>> >>>> > >> I'd like to still add "virsh define --start" in the end. Possibly when >> --start is passed, virsh would actually call virDomainCreateXML(). My >> feeling is that virsh started out really demoing the API but its >> really a fundamental user tool now and we should strive to make it >> friendly to use. > > > I think that the original version of your patch is usable although it's > lacking documentation in the man page. If you mind reposting it with that > added it might be a intermediate road until somebody will be motivated > enough to add the flag for virDomainCreateXML. > > Peter > > Locally I've implemented VIR_DOMAIN_START_PERSISTENT for virDomainCreateXML() and the qemu driver. I'm tweaking virsh to call that when you do virsh define --start instead of virDomainDefine(). It still needs some massaging and docs fix ups locally before I'm ready to push it for review. But basically there's: virsh create --persistent domain.xml virsh define --start domain.xml I'll let you guys decide what parts you want to keep and what parts you don't. -- Doug Goldstein -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list