<div dir="ltr">I will update with the <span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:small;font-style:normal;font-variant-ligatures:normal;font-variant-caps:normal;font-weight:400;letter-spacing:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px;background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">results of the </span>next tier1 execution on latest 4.2.3 </div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 23, 2018 at 3:56 PM, Martin Polednik <span dir="ltr"><<a href="mailto:mpolednik@redhat.com" target="_blank">mpolednik@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 23/04/18 01:23 +0300, Elad Ben Aharon wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi, I've triggered another execution [1] due to some issues I saw in the<br>
first which are not related to the patch.<br>
<br>
The success rate is 78% which is low comparing to tier1 executions with<br>
code from downstream builds (95-100% success rates) [2].<br>
</blockquote>
<br></span>
Could you run the current master (without the dynamic_ownership patch)<br>
so that we have viable comparision?<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
>From what I could see so far, there is an issue with move and copy<br>
operations to and from Gluster domains. For example [3].<br>
<br>
The logs are attached.<br>
<br>
<br>
[1]<br></span>
*<a href="https://rhv-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/rhv-4.2-ge-runner-tier1-after-upgrade/7/testReport/" rel="noreferrer" target="_blank">https://rhv-jenkins.rhev-ci-v<wbr>ms.eng.rdu2.redhat.com/job/rhv<wbr>-4.2-ge-runner-tier1-after-upg<wbr>rade/7/testReport/</a><br>
<<a href="https://rhv-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/rhv-4.2-ge-runner-tier1-after-upgrade/7/testReport/" rel="noreferrer" target="_blank">https://rhv-jenkins.rhev-ci-v<wbr>ms.eng.rdu2.redhat.com/job/rhv<wbr>-4.2-ge-runner-tier1-after-upg<wbr>rade/7/testReport/</a>>*<br>
<br>
<br>
<br>
[2]<br>
<a href="https://rhv-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/job/" rel="noreferrer" target="_blank">https://rhv-jenkins.rhev-ci-vm<wbr>s.eng.rdu2.redhat.com/job/</a><div><div class="h5"><br>
rhv-4.2-ge-runner-tier1-after-<wbr>upgrade/7/<br>
<br>
<br>
<br>
[3]<br>
2018-04-22 13:06:28,316+0300 INFO (jsonrpc/7) [vdsm.api] FINISH<br>
deleteImage error=Image does not exist in domain:<br>
'image=cabb8846-7a4b-4244-9835<wbr>-5f603e682f33,<br>
domain=e5fd29c8-52ba-467e-be09<wbr>-ca40ff054dd4'<br>
from=:<br>
:ffff:10.35.161.182,40936, flow_id=disks_syncAction_ba6b2<wbr>630-5976-4935,<br>
task_id=3d5f2a8a-881c-409e-93e<wbr>9-aaa643c10e42 (api:51)<br>
2018-04-22 13:06:28,317+0300 ERROR (jsonrpc/7) [storage.TaskManager.Task]<br>
(Task='3d5f2a8a-881c-409e-93e9<wbr>-aaa643c10e42') Unexpected error (task:875)<br>
Traceback (most recent call last):<br>
File "/usr/lib/python2.7/site-packa<wbr>ges/vdsm/storage/task.py", line 882, in<br>
_run<br>
return fn(*args, **kargs)<br>
File "<string>", line 2, in deleteImage<br>
File "/usr/lib/python2.7/site-packa<wbr>ges/vdsm/common/api.py", line 49, in<br>
method<br>
ret = func(*args, **kwargs)<br>
File "/usr/lib/python2.7/site-packa<wbr>ges/vdsm/storage/hsm.py", line 1503, in<br>
deleteImage<br>
raise se.ImageDoesNotExistInSD(imgUU<wbr>ID, sdUUID)<br>
ImageDoesNotExistInSD: Image does not exist in domain:<br>
'image=cabb8846-7a4b-4244-9835<wbr>-5f603e682f33,<br>
domain=e5fd29c8-52ba-467e-be09<wbr>-ca40ff054dd4'<br>
<br>
2018-04-22 13:06:28,317+0300 INFO (jsonrpc/7) [storage.TaskManager.Task]<br>
(Task='3d5f2a8a-881c-409e-93e9<wbr>-aaa643c10e42') aborting: Task is aborted:<br>
"Image does not exist in domain: 'image=cabb8846-7a4b-4244-9835<wbr>-<br>
5f603e682f33, domain=e5fd29c8-52ba-467e-be09<wbr>-ca40ff054dd4'" - code 268<br>
(task:1181)<br>
2018-04-22 13:06:28,318+0300 ERROR (jsonrpc/7) [storage.Dispatcher] FINISH<br>
deleteImage error=Image does not exist in domain:<br>
'image=cabb8846-7a4b-4244-9835<wbr>-5f603e682f33, domain=e5fd29c8-52ba-467e-be09<br>
-ca40ff054d<br>
d4' (dispatcher:82)<br>
<br>
<br>
<br>
On Thu, Apr 19, 2018 at 5:34 PM, Elad Ben Aharon <<a href="mailto:ebenahar@redhat.com" target="_blank">ebenahar@redhat.com</a>><br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Triggered a sanity tier1 execution [1] using [2], which covers all the<br>
requested areas, on iSCSI, NFS and Gluster.<br>
I'll update with the results.<br>
<br>
[1]<br>
<a href="https://rhv-jenkins.rhev-ci-vms.eng.rdu2.redhat.com/view/4.2" rel="noreferrer" target="_blank">https://rhv-jenkins.rhev-ci-vm<wbr>s.eng.rdu2.redhat.com/view/4.2</a><br>
_dev/job/rhv-4.2-ge-flow-stora<wbr>ge/1161/<br>
<br>
[2]<br>
<a href="https://gerrit.ovirt.org/#/c/89830/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/8<wbr>9830/</a><br>
vdsm-4.30.0-291.git77aef9a.el7<wbr>.x86_64<br>
<br>
<br>
<br>
On Thu, Apr 19, 2018 at 3:07 PM, Martin Polednik <<a href="mailto:mpolednik@redhat.com" target="_blank">mpolednik@redhat.com</a>><br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 19/04/18 14:54 +0300, Elad Ben Aharon wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Martin,<br>
<br>
I see [1] requires a rebase, can you please take care?<br>
<br>
</blockquote>
<br>
Should be rebased.<br>
<br>
At the moment, our automation is stable only on iSCSI, NFS, Gluster and<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
FC.<br>
Ceph is not supported and Cinder will be stabilized soon, AFAIR, it's not<br>
stable enough at the moment.<br>
<br>
</blockquote>
<br>
That is still pretty good.<br>
<br>
<br>
[1] <a href="https://gerrit.ovirt.org/#/c/89830/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/8<wbr>9830/</a><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Thanks<br>
<br>
On Wed, Apr 18, 2018 at 2:17 PM, Martin Polednik <<a href="mailto:mpolednik@redhat.com" target="_blank">mpolednik@redhat.com</a>><br>
wrote:<br>
<br>
On 18/04/18 11:37 +0300, Elad Ben Aharon wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Hi, sorry if I misunderstood, I waited for more input regarding what<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
areas<br>
have to be tested here.<br>
<br>
<br>
</blockquote>
I'd say that you have quite a bit of freedom in this regard. GlusterFS<br>
should be covered by Dennis, so iSCSI/NFS/ceph/cinder with some suite<br>
that covers basic operations (start & stop VM, migrate it), snapshots<br>
and merging them, and whatever else would be important for storage<br>
sanity.<br>
<br>
mpolednik<br>
<br>
<br>
On Wed, Apr 18, 2018 at 11:16 AM, Martin Polednik <<a href="mailto:mpolednik@redhat.com" target="_blank">mpolednik@redhat.com</a><br>
><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
wrote:<br>
<br>
On 11/04/18 16:52 +0300, Elad Ben Aharon wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
We can test this on iSCSI, NFS and GlusterFS. As for ceph and cinder,<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
will<br>
have to check, since usually, we don't execute our automation on<br>
them.<br>
<br>
<br>
Any update on this? I believe the gluster tests were successful, OST<br>
</blockquote>
passes fine and unit tests pass fine, that makes the storage backends<br>
test the last required piece.<br>
<br>
<br>
On Wed, Apr 11, 2018 at 4:38 PM, Raz Tamir <<a href="mailto:ratamir@redhat.com" target="_blank">ratamir@redhat.com</a>><br>
wrote:<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+Elad<br>
<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:28 PM, Dan Kenigsberg <<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>><br>
wrote:<br>
<br>
On Wed, Apr 11, 2018 at 12:34 PM, Nir Soffer <<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>><br>
wrote:<br>
<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 <<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>><br>
wrote:<br>
<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<br>
possible<br>
<br>
before merging ( using 'ci please build' )<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
But note that OST is not a way to verify the patch.<br>
<br>
</blockquote>
<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 <<br>
<a href="mailto:mpolednik@redhat.com" target="_blank">mpolednik@redhat.com</a><br>
><br>
<br>
wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Hey,<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I've created a patch[0] that is finally able to activate<br>
libvirt'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<br>
the<br>
area of host devices, hwrng and anything that touches devices;<br>
bunch<br>
of test changes and one XML generation caveat (storage is<br>
handled<br>
by<br>
VDSM, therefore disk relabelling needs to be disabled on the<br>
VDSM<br>
level).<br>
<br>
Because of the scope of the patch, I welcome<br>
storage/virt/network<br>
people to review the code and consider the implication this<br>
change<br>
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>
<br>
In particular: dynamic_ownership was set to 0 prehistorically<br>
(as<br>
<br>
</blockquote>
<br>
part<br>
</blockquote>
<br>
</blockquote>
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<br>
libvirt,<br>
running as root, was not able to play properly with root-squash nfs<br>
mounts.<br>
<br>
Have you attempted this use case?<br>
<br>
I join to Nir's request to run this with storage QE.<br>
<br>
<br>
<br>
<br>
</blockquote>
--<br>
<br>
<br>
Raz Tamir<br>
Manager, RHV QE<br>
<br>
<br>
<br>
<br>
</blockquote></blockquote></blockquote></blockquote></blockquote></blockquote></blockquote>
<br>
</blockquote></div></div></blockquote>
<br>
<br>
</blockquote></div><br></div>