[virt-tools-list] virt-manager: show read-only check button when add fs for lxc container

Chen Hanxiao chenhanxiao at cn.fujitsu.com
Tue Nov 26 01:42:47 UTC 2013



> -----Original Message-----
> From: Cole Robinson [mailto:crobinso at redhat.com]
> Sent: Monday, November 25, 2013 11:18 PM
> To: Chen Hanxiao; virt-tools-list at redhat.com
> Subject: Re: [virt-tools-list] virt-manager: show read-only check button
when
> add fs for lxc container
> 
> On 11/25/2013 01:51 AM, Chen Hanxiao wrote:
> > From: Chen Hanxiao <chenhanxiao at cn.fujitsu.com>
> >
> > show read-only check button when add fs for lxc container.
> >
> > Signed-off-by: Chen Hanxiao <chenhanxiao at cn.fujitsu.com>
> > ---
> >  virtManager/addhardware.py | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/virtManager/addhardware.py b/virtManager/addhardware.py
> > index fb7c236..fa534cc 100644
> > --- a/virtManager/addhardware.py
> > +++ b/virtManager/addhardware.py
> > @@ -307,6 +307,7 @@ class vmmAddHardware(vmmGObjectUI):
> >          simple_store_set("fs-wrpolicy-combo",
> VirtualFilesystem.WRPOLICIES)
> >          self.show_pair_combo("fs-type", self.conn.is_openvz())
> >          self.show_check_button("fs-readonly", self.conn.is_qemu())
> > +        self.show_check_button("fs-readonly", self.conn.is_lxc())
> >
> >          # Smartcard widgets
> >          combo = self.widget("smartcard-mode")
> >
> 
> Just use 'self.conn.is_qemu() or self.conn.is_lxc()'. The patch as is will
> actually hide the readonly checkbox for qemu.
> 
> Please also use [PATCH virt-manager] as a subject prefix for virt-manager
> patches. git format-patch --subject-prefix="PATCH virt-manager"

Thanks for your help.
v2 patch will fit the new subject style

> 
> Thanks,
> Cole





More information about the virt-tools-list mailing list