<div dir="ltr"><div>Ping - opinions on this or is it ready to be committed?</div><div><br></div>On this reply setting Guido on CC as he has experience on apparmor patches in libvirt and commit rights.<div><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 11, 2017 at 8:58 PM, intrigeri <span dir="ltr"><<a href="mailto:intrigeri+libvirt@boum.org" target="_blank">intrigeri+libvirt@boum.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Christian Ehrhardt:<br>
<span class="">> With that qemu change in place the rules generated for the image<br>
> and backing files need the allowance to also lock (k) the files.<br>
<br>
</span>I'm not a C developer so I cannot review the proposed patch, but FWIW<br>
Christian's reasoning makes sense to me and from a high-level AppArmor<br>
perspective, the proposed change seems entirely harmless.<br>
<br>
Cheers,<br>
<span class="HOEnZb"><font color="#888888">--<br>
intrigeri<br>
<br>
--<br>
libvir-list mailing list<br>
<a href="mailto:libvir-list@redhat.com">libvir-list@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/libvir-list" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/libvir-list</a><br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div></div></div>