[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [libvirt-designer PATCHv2 1/9] Improve VM skeleton created by gvir_designer_domain_setup_guest()



On Thu, Apr 18, 2013 at 05:18:55PM +0100, Daniel P. Berrange wrote:
> On Thu, Apr 18, 2013 at 06:12:25PM +0200, Christophe Fergeau wrote:
> > Add various devices/configuration to libvirt XML config when
> > creating the VM. This configuration is generic enough that it
> > should be useful on all created VMs, that's why no public API is
> > added to set them up. However, they are split in several helpers
> > that can easily be exported if needed.
> > 
> > What this commit adds is:
> > - clock
> > - sound card
> > - input device (a tablet which will act as a mouse)
> > - a console (Boxes is adding one, not sure if it's required...)
> > 
> > For maximum flexibility, we may want to make the last 2 public.
> > I'm undecided on sound card, I don't think it's terribly interesting
> > to configure, but I wouldn't mind exposing this publicly either.
> 
> Commit message is outdated - you're not adding sound cards
> anymore.

Thanks, I've changed it to:

    Improve VM skeleton created by gvir_designer_domain_setup_guest()

    Add various devices/configuration to libvirt XML config when
    creating the VM. This configuration is generic enough that it
    should be useful on all created VMs, that's why no public API is
    added to set them up. However, they are split in several helpers
    that can easily be exported if needed.

    What this commit adds is:
    - clock
    - input device (a tablet which will act as a mouse)
    - a console (Boxes is adding one, not sure if it's required...)

    For maximum flexibility, we may want to let applications decide whether
    they want a console or not.

    In addition to these devices, gvir_designer_domain_setup_guest()
    now disables power management, and sets the domain to stop on
    power off/crash, and to restart on reboots.

Christophe

Attachment: pgplNtIhQiidy.pgp
Description: PGP signature


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]