Re: [libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-14 Thread Jamie Strandboge
On Fri, 2012-05-11 at 22:39 -0400, Stefan Berger wrote: +if (virtDriver STREQ(virtDriver, LXC)) +return rc; + ACK once changed to: if (virtDriver STREQ(virtDriver, LXC)) ... -- Jamie Strandboge | http://www.canonical.com signature.asc Description: This is a

Re: [libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-14 Thread Daniel P. Berrange
On Mon, May 14, 2012 at 08:06:03AM -0500, Jamie Strandboge wrote: On Fri, 2012-05-11 at 22:39 -0400, Stefan Berger wrote: +if (virtDriver STREQ(virtDriver, LXC)) +return rc; + ACK once changed to: if (virtDriver STREQ(virtDriver, LXC)) The patch is already like that

[libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-11 Thread Daniel P. Berrange
From: Daniel Walsh dwa...@redhat.com The AppArmour driver does not currently have support for LXC so ensure that when probing, it claims to be disabled Signed-off-by: Daniel P. Berrange berra...@redhat.com --- src/security/security_apparmor.c |5 - 1 file changed, 4 insertions(+), 1

Re: [libvirt] [PATCH 2/6] Don't enable the AppArmour security driver with LXC

2012-05-11 Thread Stefan Berger
On 05/11/2012 06:10 AM, Daniel P. Berrange wrote: From: Daniel Walshdwa...@redhat.com The AppArmour driver does not currently have support for LXC so ensure that when probing, it claims to be disabled Signed-off-by: Daniel P. Berrangeberra...@redhat.com --- src/security/security_apparmor.c |