Re: [PATCH] Add libvirt storage browser

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

Cole Robinson wrote:
> Daniel P. Berrange wrote:
>> On Sun, Mar 08, 2009 at 05:10:18PM -0400, Cole Robinson wrote:
>>> The attached patch adds a libvirt-storage-aware browser, to replace the
>>> standard file chooser when adding storage on remote connections.
>>> Screenshot here:
>>> (Greyed out pools are inactive.)
>>> In the near future, this will be enabled for local connections as well,
>>> which will allow provisioning more complex storage on demand, as well as
>>> encourage all storage to go through the libvirt APIs (while still
>>> allowing the use of unmanaged storage through the 'Browse local' option,
>>> or manually entering a path). Before we turn it on in the local case
>>> though, default pool setup needs to be more configurable and robust.
>> Personally I'd turn it on for both local & remote usage right away. With
>> real users trying it out all the time, it'll quickly get debugged & more
>> robust. I think having & documenting 2 different ways of doing storage
>> will cause more trouble that you'll save. 
> Yeah, I'm unsure on the whole thing, but I'm leaning in your direction
> now. The situation with file browsing is just as confusing if running as
> nonroot at the moment, and the storage dialog will enable better
> functionality. The big issue is that this is new UI and there are no
> docs to point at when users have questions. More work for me I suppose :)
> I'll enable this for storage provisioning with the new release, not
> local install media though (since we really don't have anything storage
> pool related hooked up cleanly for this case).

Applied now, with the above defaults change.


et-mgmt-tools mailing list

[Home]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Red Hat 9 Bible]     [Fedora Bible]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]

Powered by Linux