<div dir="ltr"><div class="gmail_default" style="font-family:monospace,monospace">Yipes.</div><div class="gmail_default" style="font-family:monospace,monospace">[1] should fix that, waiting for the CI to run to merge.</div><div class="gmail_default" style="font-family:monospace,monospace"><br></div><div class="gmail_default" style="font-family:monospace,monospace">[1] <a href="https://gerrit.ovirt.org/#/c/65768/">https://gerrit.ovirt.org/#/c/65768/</a></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 26, 2016 at 3:42 PM, Nadav Goldin <span dir="ltr"><<a href="mailto:ngoldin@redhat.com" target="_blank">ngoldin@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Unfortunately it is still failing, see[1], the repository used was<br>
built from commit [2]. If you want to check the logs same links<br>
apply(just replace build number 2759->2782)<br>
<br>
<br>
<br>
[1] <a href="http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/2782" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/t<wbr>est-repo_ovirt_experimental_ma<wbr>ster/2782</a><br>
[2] <a href="https://gerrit.ovirt.org/#/c/65740/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5740/</a><br>
<div><div class="gmail-m_-4257956841807546249h5"><br>
On Wed, Oct 26, 2016 at 11:33 AM, Tal Nisan <<a href="mailto:tnisan@redhat.com" target="_blank">tnisan@redhat.com</a>> wrote:<br>
> Should be fixed now on master as those patches were just merged:<br>
><br>
> <a href="https://gerrit.ovirt.org/#/c/65738/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5738/</a> - "Move InjectorRule to VdsBroker":<br>
> aligns the InjectorRule's package with the Injector so it can essentially be<br>
> used wherever the Injector is used [+12, -4]<br>
> <a href="https://gerrit.ovirt.org/#/c/65739" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5739</a> - "InjectorRule: override injector<br>
> anyway": Fixes a bug in InjectorRule where the Injector is only mocked if<br>
> you call InjectorRule.bind [+6, -1]<br>
> <a href="https://gerrit.ovirt.org/#/c/65740" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5740</a> - "core: InjectorRule for injecting<br>
> members": Fixes a bug in InjectorRule to allow using Injector.injectMembers<br>
> when using it [+18, -5]<br>
> <a href="https://gerrit.ovirt.org/#/c/65725" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5725</a> - "core: Fix AuditLogging": The actual<br>
> fix. Basically, goes over all the places that create an AuditLoggableBase<br>
> that needs injecting and take care of it [+155, -160]<br>
><br>
><br>
> On Wed, Oct 26, 2016 at 10:04 AM, Nadav Goldin <<a href="mailto:ngoldin@redhat.com" target="_blank">ngoldin@redhat.com</a>> wrote:<br>
>><br>
>> Hi,<br>
>> We have new failure on OST from patches merged to master yesterday,<br>
>> the failure started after the merge of [1], but as there were quite a<br>
>> few patches merged quickly I can't make sure it is the one causing<br>
>> it(OST aren't ran per-patch).<br>
>> The test that fails is [2] when attempting to start the VM.<br>
>><br>
>> The error from the API side:<br>
>><br>
>> RequestError:<br>
>> status: 500<br>
>> reason: Internal Server Error<br>
>> detail: javax.ejb.EJBException: java.lang.NullPointerException<br>
>> at<br>
>> org.jboss.as.ejb3.tx.CMTTxInte<wbr>rceptor.handleExceptionInNoTx(<wbr>CMTTxInterceptor.java:213)<br>
>> at<br>
>> org.jboss.as.ejb3.tx.CMTTxInte<wbr>rceptor.invokeInNoTx(CMTTxInte<wbr>rceptor.java:265)<br>
>> at<br>
>> org.jboss.as.ejb3.tx.CMTTxInte<wbr>rceptor.supports(CMTTxIntercep<wbr>tor.java:374)<br>
>> at<br>
>> org.jboss.as.ejb3.tx.CMTTxInte<wbr>rceptor.processInvocation(CMTT<wbr>xInterceptor.java:243)<br>
>> at<br>
>> org.jboss.invocation.Intercept<wbr>orContext.proceed(InterceptorC<wbr>ontext.java:340)<br>
</div></div>>> ....<br>
<span>>><br>
>> In the engine logs there are a few 'java.lang.NullPointerExceptio<wbr>n'<br>
>> errors:<br>
>><br>
>> 2016-10-25 11:53:52,845 INFO<br>
>> [org.ovirt.engine.core.dal.dbb<wbr>roker.auditloghandling.AuditLo<wbr>gableBase]<br>
>> (org.ovirt.thread.pool-6-threa<wbr>d-2) [5e6a88be] Failed to get vds<br>
>> 'd60db21f-95f0-487b-9f17-44861<wbr>e2610a7', error: null<br>
>> 2016-10-25 11:53:52,864 DEBUG<br>
>> [org.ovirt.engine.core.utils.t<wbr>imer.FixedDelayJobListener]<br>
>> (DefaultQuartzScheduler5) [] Rescheduling<br>
>><br>
>> DEFAULT.org.ovirt.engine.core.<wbr>bll.tasks.AsyncTaskManager.tim<wbr>erElapsed#-9223372036854775787<br>
>> as there is no unfired trigger.<br>
</span>>> ...<br>
<span>>> 2016-10-25 11:53:52,845 DEBUG<br>
>> [org.ovirt.engine.core.dal.dbb<wbr>roker.auditloghandling.AuditLo<wbr>gableBase]<br>
>> (org.ovirt.thread.pool-6-threa<wbr>d-2) [5e6a88be] Exception:<br>
>> java.lang.NullPointerException<br>
>> at<br>
>> org.ovirt.engine.core.dal.dbbr<wbr>oker.auditloghandling.AuditLog<wbr>ableBase.getVdsStatic(AuditLog<wbr>ableBase.java:633)<br>
>> [dal.jar:]<br>
>> at<br>
>> org.ovirt.engine.core.dal.dbbr<wbr>oker.auditloghandling.AuditLog<wbr>ableBase.getVdsName(AuditLogab<wbr>leBase.java:504)<br>
>> [dal.jar:]<br>
</span>>> ...<br>
<span>>> 2016-10-25 11:53:52,837 INFO<br>
>> [org.ovirt.engine.core.dal.dbb<wbr>roker.auditloghandling.AuditLo<wbr>gableBase]<br>
>> (org.ovirt.thread.pool-6-threa<wbr>d-2) [5e6a88be] Failed to get vds<br>
>> 'd60db21f-95f0-487b-9f17-44861<wbr>e2610a7', error: null<br>
>> 2016-10-25 11:53:52,837 DEBUG<br>
>> [org.ovirt.engine.core.dal.dbb<wbr>roker.auditloghandling.AuditLo<wbr>gableBase]<br>
>> (org.ovirt.thread.pool-6-threa<wbr>d-2) [5e6a88be] Exception:<br>
>> java.lang.NullPointerException<br>
>> at<br>
>> org.ovirt.engine.core.dal.dbbr<wbr>oker.auditloghandling.AuditLog<wbr>ableBase.getVdsStatic(AuditLog<wbr>ableBase.java:633)<br>
>> [dal.jar:]<br>
>> at<br>
>> org.ovirt.engine.core.dal.dbbr<wbr>oker.auditloghandling.AuditLog<wbr>ableBase.getVdsName(AuditLogab<wbr>leBase.java:504)<br>
>> [dal.jar:]<br>
</span>>> ...<br>
<div class="gmail-m_-4257956841807546249HOEnZb"><div class="gmail-m_-4257956841807546249h5">>><br>
>> The full engine logs can be found here[3] and the entire test suite<br>
>> logs here[4].<br>
>><br>
>> Can anyone have a look?<br>
>><br>
>> Thanks,<br>
>> Nadav.<br>
>><br>
>><br>
>> [1] <a href="https://gerrit.ovirt.org/#/c/65198/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/6<wbr>5198/</a><br>
>> [2]<br>
>> <a href="https://github.com/oVirt/ovirt-system-tests/blob/master/basic_suite_master/test-scenarios/004_basic_sanity.py#L322" rel="noreferrer" target="_blank">https://github.com/oVirt/ovirt<wbr>-system-tests/blob/master/<wbr>basic_suite_master/test-<wbr>scenarios/004_basic_sanity.py#<wbr>L322</a><br>
>> [3]<br>
>> <a href="http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/2759/artifact/exported-artifacts/basic_suite_master.sh-fc24/exported-artifacts/test_logs/basic_suite_master/post-004_basic_sanity.py/*zip*/post-004_basic_sanity.py.zip" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/t<wbr>est-repo_ovirt_experimental_ma<wbr>ster/2759/artifact/exported-ar<wbr>tifacts/basic_suite_master.sh-<wbr>fc24/exported-artifacts/test_<wbr>logs/basic_suite_master/post-<wbr>004_basic_sanity.py/*zip*/<wbr>post-004_basic_sanity.py.zip</a><br>
>> [4]<br>
>> <a href="http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/2759/artifact/exported-artifacts/basic_suite_master.sh-fc24/exported-artifacts/" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/t<wbr>est-repo_ovirt_experimental_ma<wbr>ster/2759/artifact/exported-ar<wbr>tifacts/basic_suite_master.sh-<wbr>fc24/exported-artifacts/</a><br>
>> ______________________________<wbr>_________________<br>
>> Devel mailing list<br>
>> <a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
>> <a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br>
><br>
><br>
</div></div></blockquote></div><br></div></div>