<div dir="ltr"><div><div><div>Dear Yedidyah,<br><br></div>Thank you for your response. I have attached the two dump files. The first is the original error for isort, dump.txt. The second dump_isort.txt is after installing python2_isort.<br><br></div>With Best Regards.<br><br></div>Steven Rosenberg.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 26, 2018 at 3:10 PM, Yedidyah Bar David <span dir="ltr">&lt;<a href="mailto:didi@redhat.com" target="_blank">didi@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">Hi,<br>
<div><div class="h5"><br>
On Mon, Feb 26, 2018 at 2:39 PM, Steven Rosenberg &lt;<a href="mailto:srosenbe@redhat.com">srosenbe@redhat.com</a>&gt; wrote:<br>
&gt; Dear Martin,<br>
&gt;<br>
&gt; I should add I am running RHEL 7.4 (Maipo).<br>
&gt;<br>
&gt; With Best Regards.<br>
&gt;<br>
&gt; Steven Rosenberg.<br>
&gt;<br>
&gt; On Mon, Feb 26, 2018 at 2:24 PM, Martin Perina &lt;<a href="mailto:mperina@redhat.com">mperina@redhat.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; Didi/Sandro, any ideas about it? AFAIK on CentOS/RHEL 7 everything works<br>
&gt;&gt; as expected ...<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; On Mon, Feb 26, 2018 at 11:58 AM, Steven Rosenberg &lt;<a href="mailto:srosenbe@redhat.com">srosenbe@redhat.com</a>&gt;<br>
&gt;&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Dear Martin Perina,<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; We found some issues with the oVirt Engine master branch when updating<br>
&gt;&gt;&gt; the version via git pull.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; We then performed the make command:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; make clean install-dev PREFIX=~/ovirt_engine_master<br>
&gt;&gt;&gt; DEV_BUILD_SCL_POSTGRESQL=1<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; It seems a new dependency was added for isort, so the make fails with the<br>
&gt;&gt;&gt; following error:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; ------------------------------<wbr>------------------------------<wbr>------------------------------<wbr>--------------------------<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; packaging/setup/plugins/ovirt-<wbr>engine-rename/ovirt-engine/<wbr>database.py:313:21:<br>
&gt;&gt;&gt; E126 continuation line over-indented for hanging indent<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; packaging/setup/plugins/ovirt-<wbr>engine-rename/ovirt-engine/<wbr>database.py:314:21:<br>
&gt;&gt;&gt; E126 continuation line over-indented for hanging indent<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; packaging/setup/plugins/ovirt-<wbr>engine-common/base/core/<wbr>duplicated_constants_check.py:<wbr>109:17:<br>
&gt;&gt;&gt; E124 closing bracket does not match visual indentation<br>
&gt;&gt;&gt; + ret=1<br>
&gt;&gt;&gt; + which isort<br>
&gt;&gt;&gt; + echo &#39;WARNING: tool &#39;\&#39;&#39;isort&#39;\&#39;&#39; is missing&#39;<br>
&gt;&gt;&gt; WARNING: tool &#39;isort&#39; is missing<br>
<br>
</div></div>This is just a warning<br>
<br>
&gt;&gt;&gt; + exit 1<br>
<br>
Most likely the error causing it to exit 1 happened earlier, please check/share<br>
more of the output.<br>
<span class=""><br>
&gt;&gt;&gt; make[1]: *** [validations] Error 1<br>
&gt;&gt;&gt; make[1]: Leaving directory `/home/srosenbe/Documents/git/<wbr>ovirt-engine&#39;<br>
&gt;&gt;&gt; make: *** [all-dev] Error 2<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; ------------------------------<wbr>------------------------------<wbr>------------------------------<wbr>--------------------------<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; To attempt to address this issue, we guessed and installed python2-isort<br>
&gt;&gt;&gt; through yum, though the readme states it is optional.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Though the process continued, it then failed in the python module:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; /home/srosenbe/Documents/git/<wbr>ovirt-engine/packaging/<wbr>pythonlib/ovirt_engine/<wbr>service.py<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; That error is here:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; ------------------------------<wbr>------------------------------<wbr>------------------------------<wbr>------------------------------<wbr>---<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; ---<br>
&gt;&gt;&gt; /home/srosenbe/Documents/git/<wbr>ovirt-engine/packaging/<wbr>pythonlib/ovirt_engine/<wbr>service.py:before<br>
&gt;&gt;&gt; 2018-02-12 12:17:26<br>
&gt;&gt;&gt; +++<br>
&gt;&gt;&gt; /home/srosenbe/Documents/git/<wbr>ovirt-engine/packaging/<wbr>pythonlib/ovirt_engine/<wbr>service.py:after<br>
&gt;&gt;&gt; 2018-02-26 12:07:11.233915<br>
&gt;&gt;&gt; @@ -31,6 +31,7 @@<br>
&gt;&gt;&gt;  import time<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;  import daemon<br>
&gt;&gt;&gt; +<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;  from dateutil import tz<br>
&gt;&gt;&gt;<br>
<br>
</span>This is an output of &#39;isort --diff&#39;, and not the actual error.<br>
<br>
(isort --check, which is what we run to test the code, does not fail if<br>
the diff it would have suggested consists of only difference in the number<br>
of empty lines. It would still be nice to patch service.py to have another<br>
empty line, but not needed).<br>
<br>
&gt;&gt;&gt; + exit 1<br>
<br>
Same. Error was most likely earlier.<br>
<br>
Best regards,<br>
<div class="HOEnZb"><div class="h5"><br>
&gt;&gt;&gt; make[1]: *** [validations] Error 1<br>
&gt;&gt;&gt; make[1]: Leaving directory `/home/srosenbe/Documents/git/<wbr>ovirt-engine&#39;<br>
&gt;&gt;&gt; make: *** [all-dev] Error 2<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; ------------------------------<wbr>------------------------------<wbr>------------------------------<wbr>------------------------------<wbr>---<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Please advise if you have a quick fix or when this issue will be<br>
&gt;&gt;&gt; addressed.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Thank you for your time and consideration.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; With Best Regards.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Steven Rosenberg.<br>
&gt;&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Martin Perina<br>
&gt;&gt; Associate Manager, Software Engineering<br>
&gt;&gt; Red Hat Czech s.r.o.<br>
&gt;<br>
&gt;<br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Didi<br>
</font></span></blockquote></div><br></div>