<div dir="ltr">Hi, sorry if I misunderstood, I waited for more input regarding what areas have to be tested here.<div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 18, 2018 at 11:16 AM, Martin Polednik <span dir="ltr">&lt;<a href="mailto:mpolednik@redhat.com" target="_blank">mpolednik@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 11/04/18 16:52 +0300, Elad Ben Aharon wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We can test this on iSCSI, NFS and GlusterFS. As for ceph and cinder, will<br>
have to check, since usually, we don&#39;t execute our automation on them.<br>
</blockquote>
<br></span>
Any update on this? I believe the gluster tests were successful, OST<br>
passes fine and unit tests pass fine, that makes the storage backends<br>
test the last required piece.<div class="m_-8113256544789027270HOEnZb"><div class="m_-8113256544789027270h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Apr 11, 2018 at 4:38 PM, Raz Tamir &lt;<a href="mailto:ratamir@redhat.com" target="_blank">ratamir@redhat.com</a>&gt; wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+Elad<br>
<br>
On Wed, Apr 11, 2018 at 4:28 PM, Dan Kenigsberg &lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt; wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Apr 11, 2018 at 12:34 PM, Nir Soffer &lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt; wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Apr 11, 2018 at 12:31 PM Eyal Edri &lt;<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>&gt; wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Please make sure to run as much OST suites on this patch as possible<br>
before merging ( using &#39;ci please build&#39; )<br>
<br>
</blockquote>
<br>
But note that OST is not a way to verify the patch.<br>
<br>
Such changes require testing with all storage types we support.<br>
<br>
Nir<br>
<br>
On Tue, Apr 10, 2018 at 4:09 PM, Martin Polednik &lt;<a href="mailto:mpolednik@redhat.com" target="_blank">mpolednik@redhat.com</a>&gt;<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hey,<br>
<br>
I&#39;ve created a patch[0] that is finally able to activate libvirt&#39;s<br>
dynamic_ownership for VDSM while not negatively affecting<br>
functionality of our storage code.<br>
<br>
That of course comes with quite a bit of code removal, mostly in the<br>
area of host devices, hwrng and anything that touches devices; bunch<br>
of test changes and one XML generation caveat (storage is handled by<br>
VDSM, therefore disk relabelling needs to be disabled on the VDSM<br>
level).<br>
<br>
Because of the scope of the patch, I welcome storage/virt/network<br>
people to review the code and consider the implication this change has<br>
on current/future features.<br>
<br>
[0] <a href="https://gerrit.ovirt.org/#/c/89830/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/8<wbr>9830/</a><br>
<br>
</blockquote>
<br>
</blockquote></blockquote>
In particular:  dynamic_ownership was set to 0 prehistorically (as part<br>
of <a href="https://bugzilla.redhat.com/show_bug.cgi?id=554961" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/sh<wbr>ow_bug.cgi?id=554961</a> ) because libvirt,<br>
running as root, was not able to play properly with root-squash nfs mounts.<br>
<br>
Have you attempted this use case?<br>
<br>
I join to Nir&#39;s request to run this with storage QE.<br>
<br>
</blockquote>
<br>
<br>
<br>
--<br>
<br>
<br>
Raz Tamir<br>
Manager, RHV QE<br>
<br>
</blockquote></blockquote>
</div></div></blockquote></div><br></div></div>