[virt-tools-list] [virt-viewer v2] ovirt: Only use active ISO domains for foreign menu

Fabiano Fidêncio fidencio at redhat.com
Wed Mar 2 12:04:30 UTC 2016


On Wed, Mar 2, 2016 at 12:02 PM, Christophe Fergeau <cfergeau at redhat.com> wrote:
> On Wed, Mar 02, 2016 at 08:52:28AM +0100, Christophe Fergeau wrote:
>> oVirt storage domains can be in various states (inactive, in
>> maintainance, ...). We only want to show the ISOs it contains in the
>> foreign menu when the storage domain is actually active, not in the
>> other states.
>>
>> https://bugzilla.redhat.com/show_bug.cgi?id=1310450
>> ---
>> Changes since v1:
>>      - added new condition to an existing if() block
>
> Fwiw, I prefer the v1 way which I find much easier to read.

Same here.

>
> Christophe
>
> _______________________________________________
> virt-tools-list mailing list
> virt-tools-list at redhat.com
> https://www.redhat.com/mailman/listinfo/virt-tools-list




More information about the virt-tools-list mailing list