On Tue, 19 Feb 2019 at 10:56, Tal Nisan <tni...@redhat.com> wrote: > OK seems like I've found the reason, Ori you merged this patch between the > time when the version bumps were pushed and their actual merge which cause > them to rebase on the broken patch - https://gerrit.ovirt.org/#/c/96610/ > Note that Jenkins failed on patchset 12 while CI was marked as +1 manually > (I guess wrongly based on the fact the artifacts were successfully build > but that does not mean tests passed) > > As a next step I'd like to know how do we avoid it next time?
> > On Tue, Feb 19, 2019 at 10:50 AM Tal Nisan <tni...@redhat.com> wrote: > >> I don't get it, both version bump patches passed Jenkins with all unit >> tests: >> https://gerrit.ovirt.org/#/c/97835 >> And the Jenkins job: >> >> https://jenkins.ovirt.org/job/ovirt-engine_master_check-patch-el7-x86_64/48195/ >> >> Fred, what is the suspected patch that broke the tests? I suspect Gerrit >> auto rebase might broke something in case the version bump patches were not >> fully rebased on master when they were pushed >> >> On Tue, Feb 19, 2019 at 9:51 AM Fred Rolland <froll...@redhat.com> wrote: >> >>> This fixes the failing test: >>> https://gerrit.ovirt.org/#/c/97865/ >>> >>> On Tue, Feb 19, 2019 at 9:28 AM Benny Zlotnik <bzlot...@redhat.com> >>> wrote: >>> >>>> I created a revert patch last night: >>>> https://gerrit.ovirt.org/#/c/97854/ >>>> >>>> frolland is currently checking if it's easy to fix without reverting >>>> >>>> On Tue, Feb 19, 2019, 09:18 Sandro Bonazzola <sbona...@redhat.com >>>> wrote: >>>> >>>>> Hi, >>>>> please note engine build failed with: >>>>> >>>>> *17:45:54* [ERROR] Tests run: 11, Failures: 0, Errors: 3, Skipped: 1, >>>>> Time elapsed: 0.258 s <<< FAILURE! - in >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest*17:45:54* >>>>> [ERROR] testAdd Time elapsed: 0.027 s <<< ERROR!*17:45:54* >>>>> org.ovirt.engine.api.restapi.utils.MappingException: >>>>> java.lang.reflect.InvocationTargetException*17:45:54* at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testAdd(BackendAttachedStorageDomainsResourceTest.java:75)*17:45:54* >>>>> Caused by: java.lang.reflect.InvocationTargetException*17:45:54* >>>>> at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testAdd(BackendAttachedStorageDomainsResourceTest.java:75)*17:45:54* >>>>> Caused by: java.lang.NullPointerException*17:45:54* at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testAdd(BackendAttachedStorageDomainsResourceTest.java:75)*17:45:54* >>>>> *17:45:54* [ERROR] testList Time elapsed: 0.018 s <<< ERROR!*17:45:54* >>>>> org.ovirt.engine.api.restapi.utils.MappingException: >>>>> java.lang.reflect.InvocationTargetException*17:45:54* at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.getCollection(BackendAttachedStorageDomainsResourceTest.java:251)*17:45:54* >>>>> at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testList(BackendAttachedStorageDomainsResourceTest.java:179)*17:45:54* >>>>> Caused by: java.lang.reflect.InvocationTargetException*17:45:54* at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.getCollection(BackendAttachedStorageDomainsResourceTest.java:251)*17:45:54* >>>>> at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testList(BackendAttachedStorageDomainsResourceTest.java:179)*17:45:54* >>>>> Caused by: java.lang.NullPointerException*17:45:54* at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.getCollection(BackendAttachedStorageDomainsResourceTest.java:251)*17:45:54* >>>>> at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testList(BackendAttachedStorageDomainsResourceTest.java:179)*17:45:54* >>>>> *17:45:54* [ERROR] testAddByName Time elapsed: 0.022 s <<< >>>>> ERROR!*17:45:54* org.ovirt.engine.api.restapi.utils.MappingException: >>>>> java.lang.reflect.InvocationTargetException*17:45:54* at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testAddByName(BackendAttachedStorageDomainsResourceTest.java:116)*17:45:54* >>>>> Caused by: java.lang.reflect.InvocationTargetException*17:45:54* >>>>> at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testAddByName(BackendAttachedStorageDomainsResourceTest.java:116)*17:45:54* >>>>> Caused by: java.lang.NullPointerException*17:45:54* at >>>>> org.ovirt.engine.api.restapi.resource.BackendAttachedStorageDomainsResourceTest.testAddByName(BackendAttachedStorageDomainsResourceTest.java:116)*17:45:54* >>>>> *17:45:54* [INFO] Running >>>>> org.ovirt.engine.api.restapi.util.LinkFollowerTest*17:45:54* [INFO] Tests >>>>> run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.175 s - in >>>>> org.ovirt.engine.api.restapi.util.LinkFollowerTest*17:45:54* [INFO] >>>>> Running >>>>> org.ovirt.engine.api.restapi.util.FenceOptionsParserTest*17:45:54* ERROR >>>>> org.ovirt.engine.api.restapi.util.FenceOptionsParser - No type specified >>>>> for fencing option "one".*17:45:54* ERROR >>>>> org.ovirt.engine.api.restapi.util.FenceOptionsParser - Invalid fencing >>>>> option description "three".*17:45:54* ERROR >>>>> org.ovirt.engine.api.restapi.util.FenceOptionsParser - Invalid fencing >>>>> agent description "foo:::one=1,two=2,three=3".*17:45:54* [INFO] Tests >>>>> run: 11, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.012 s - in >>>>> org.ovirt.engine.api.restapi.util.FenceOptionsParserTest*17:45:54* [INFO] >>>>> Running org.ovirt.engine.api.restapi.util.LinkHelperTest*17:45:54* [INFO] >>>>> Tests run: 23, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.009 s >>>>> - in org.ovirt.engine.api.restapi.util.LinkHelperTest*17:45:54* [INFO] >>>>> Running org.ovirt.engine.api.restapi.util.QueryHelperTest*17:45:54* >>>>> [INFO] Tests run: 20, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: >>>>> 0.001 s - in org.ovirt.engine.api.restapi.util.QueryHelperTest*17:45:55* >>>>> [INFO] *17:45:55* [INFO] Results:*17:45:55* [INFO] *17:45:55* [ERROR] >>>>> Failures: *17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainResourceTest.testRemoveCantDo:181->doTestBadRemove:203 >>>>> Unexpected exception type thrown ==> expected: >>>>> <javax.ws.rs.WebApplicationException> but was: >>>>> <org.ovirt.engine.api.restapi.utils.MappingException>*17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainResourceTest.testRemoveFailed:186->doTestBadRemove:203 >>>>> Unexpected exception type thrown ==> expected: >>>>> <javax.ws.rs.WebApplicationException> but was: >>>>> <org.ovirt.engine.api.restapi.utils.MappingException>*17:45:55* [ERROR] >>>>> BackendStorageDomainResourceTest.testConflictedUpdate:175 Unexpected >>>>> exception type thrown ==> expected: <javax.ws.rs.WebApplicationException> >>>>> but was: <org.ovirt.engine.api.restapi.utils.MappingException>*17:45:55* >>>>> [ERROR] >>>>> BackendStorageDomainResourceTest.testUpdateCantDo:145->doTestBadUpdate:164 >>>>> Unexpected exception type thrown ==> expected: >>>>> <javax.ws.rs.WebApplicationException> but was: >>>>> <org.ovirt.engine.api.restapi.utils.MappingException>*17:45:55* [ERROR] >>>>> BackendStorageDomainResourceTest.testUpdateFailed:150->doTestBadUpdate:164 >>>>> Unexpected exception type thrown ==> expected: >>>>> <javax.ws.rs.WebApplicationException> but was: >>>>> <org.ovirt.engine.api.restapi.utils.MappingException>*17:45:55* [ERROR] >>>>> Errors: *17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainResourceTest.testGet:69 ? Mapping >>>>> java.lang.reflec...*17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainResourceTest.testRemove:152 ? Mapping >>>>> java.lang.re...*17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainResourceTest.testRemoveLocalStorage:176 ? >>>>> Mapping ...*17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainsResourceTest.testAdd:75 ? Mapping >>>>> java.lang.refle...*17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainsResourceTest.testAddByName:116 ? Mapping >>>>> java.lan...*17:45:55* [ERROR] >>>>> BackendAttachedStorageDomainsResourceTest.testList:179->getCollection:251 >>>>> ? Mapping*17:45:55* [ERROR] BackendStorageDomainResourceTest.testGet:79 >>>>> ? Mapping java.lang.reflect.Invoca...*17:45:55* [ERROR] >>>>> BackendStorageDomainResourceTest.testGetFcp:91 ? Mapping >>>>> java.lang.reflect.Inv...*17:45:55* [ERROR] >>>>> BackendStorageDomainResourceTest.testUpdate:140 ? Mapping >>>>> java.lang.reflect.In...*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddExistingStorageDomain:169->doTestAddStorageDomain:211 >>>>> ? Mapping*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddIscsiStorageDomain:348 ? Mapping >>>>> java...*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddIscsiStorageDomainAssumingConnection:392 >>>>> ? Mapping*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddLocalStorageDomain:251 ? Mapping >>>>> java...*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddPosixStorageDomain:301 ? Mapping >>>>> java...*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddStorageDomain:82->doTestAddStorageDomain:211 >>>>> ? Mapping*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddStorageDomainWithExistingConnectionId:128 >>>>> ? Mapping*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest.testAddStorageDomainWithHostName:162->doTestAddStorageDomain:211 >>>>> ? Mapping*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest>AbstractBackendCollectionResourceTest.testList:72->getCollection:634 >>>>> ? Mapping*17:45:55* [ERROR] >>>>> BackendStorageDomainsResourceTest>AbstractBackendCollectionResourceTest.testQuery:81->getCollection:634 >>>>> ? Mapping*17:45:55* [INFO] *17:45:55* [ERROR] Tests run: 1849, Failures: >>>>> 5, Errors: 19, Skipped: 65*17:45:55* [INFO] *17:45:55* [INFO] >>>>> ------------------------------------------------------------------------*17:45:55* >>>>> [INFO] Reactor Summary:*17:45:55* [INFO] *17:45:55* [INFO] oVirt >>>>> Findbugs Filters ............................ SUCCESS [1.970s]*17:45:55* >>>>> [INFO] ovirt-root ........................................ SUCCESS >>>>> [3.307s]*17:45:55* [INFO] oVirt Build Tools root >>>>> ............................ SUCCESS [1.213s]*17:45:55* [INFO] oVirt >>>>> checkstyle .................................. SUCCESS [2.121s]*17:45:55* >>>>> [INFO] oVirt Checkstyle Checks ........................... SUCCESS >>>>> [3.430s]*17:45:55* [INFO] oVirt Modules - backend >>>>> ........................... SUCCESS [0.734s]*17:45:55* [INFO] oVirt >>>>> Manager ..................................... SUCCESS [0.738s]*17:45:55* >>>>> [INFO] oVirt Engine dependencies ......................... SUCCESS >>>>> [0.891s]*17:45:55* [INFO] oVirt Engine common dependencies >>>>> .................. SUCCESS [3.350s]*17:45:55* [INFO] oVirt Engine tools >>>>> dependencies ................... SUCCESS [0.813s]*17:45:55* [INFO] >>>>> Utilities to extend java.logging framework ........ SUCCESS >>>>> [1.682s]*17:45:55* [INFO] Extensions API root >>>>> ............................... SUCCESS [0.005s]*17:45:55* [INFO] >>>>> ovirt-engine-extensions-api ....................... SUCCESS >>>>> [9.274s]*17:45:55* [INFO] oVirt Modules - manager >>>>> ........................... SUCCESS [2.056s]*17:45:55* [INFO] Universal >>>>> utilities ............................... SUCCESS [20.282s]*17:45:55* >>>>> [INFO] Extensions manager ................................ SUCCESS >>>>> [5.412s]*17:45:55* [INFO] GWT UI Compatibility >>>>> .............................. SUCCESS [6.178s]*17:45:55* [INFO] Common >>>>> Code ....................................... SUCCESS [36.381s]*17:45:55* >>>>> [INFO] Common utilities .................................. SUCCESS >>>>> [35.291s]*17:45:55* [INFO] Branding package >>>>> .................................. SUCCESS [9.146s]*17:45:55* [INFO] >>>>> ovirt-engine sso .................................. SUCCESS >>>>> [5.829s]*17:45:55* [INFO] Data Access Layer >>>>> ................................. SUCCESS [30.876s]*17:45:55* [INFO] >>>>> engine scheduler bean ............................. SUCCESS >>>>> [6.888s]*17:45:55* [INFO] Vds broker >>>>> ........................................ SUCCESS [32.916s]*17:45:55* >>>>> [INFO] builtin-extensions ................................ SUCCESS >>>>> [2.320s]*17:45:55* [INFO] Search Backend >>>>> .................................... SUCCESS [10.064s]*17:45:55* [INFO] >>>>> Backend Authentication, Authorization and Accounting SUCCESS >>>>> [7.355s]*17:45:55* [INFO] Custom Application Server Authentication Plugin >>>>> ... SUCCESS [3.320s]*17:45:55* [INFO] Backend Logic @Service bean >>>>> ....................... SUCCESS [2:51.560s]*17:45:55* [INFO] oVirt >>>>> RESTful API Backend Integration ............. SUCCESS [1.084s]*17:45:55* >>>>> [INFO] oVirt RESTful API interface ....................... SUCCESS >>>>> [1.005s]*17:45:55* [INFO] oVirt Engine API Definition >>>>> ....................... SUCCESS [50.498s]*17:45:55* [INFO] oVirt Engine >>>>> API Commom Parent POM ................ SUCCESS [1.037s]*17:45:55* [INFO] >>>>> oVirt Engine API Common JAX-RS .................... SUCCESS >>>>> [8.077s]*17:45:55* [INFO] oVirt RESTful API Backend Integration Type >>>>> Mappers SUCCESS [22.779s]*17:45:55* [INFO] oVirt RESTful API Backend >>>>> Integration JAX-RS Resources FAILURE [1:52.179s] >>>>> >>>>> >>>>> Job is here: >>>>> https://jenkins.ovirt.org/job/ovirt-engine_master_build-artifacts-el7-x86_64/8532/consoleFull >>>>> >>>>> So we need a respin once fixed. >>>>> >>>>> -- >>>>> >>>>> SANDRO BONAZZOLA >>>>> >>>>> MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV >>>>> >>>>> Red Hat EMEA <https://www.redhat.com/> >>>>> >>>>> sbona...@redhat.com >>>>> <https://red.ht/sig> >>>>> _______________________________________________ >>>>> Devel mailing list -- devel@ovirt.org >>>>> To unsubscribe send an email to devel-le...@ovirt.org >>>>> Privacy Statement: https://www.ovirt.org/site/privacy-policy/ >>>>> oVirt Code of Conduct: >>>>> https://www.ovirt.org/community/about/community-guidelines/ >>>>> List Archives: >>>>> https://lists.ovirt.org/archives/list/devel@ovirt.org/message/2GJAJHWAD5X435IFQWIABZF5KXFQ5NHM/ >>>>> >>>> _______________________________________________ >>>> Devel mailing list -- devel@ovirt.org >>>> To unsubscribe send an email to devel-le...@ovirt.org >>>> Privacy Statement: https://www.ovirt.org/site/privacy-policy/ >>>> oVirt Code of Conduct: >>>> https://www.ovirt.org/community/about/community-guidelines/ >>>> List Archives: >>>> https://lists.ovirt.org/archives/list/devel@ovirt.org/message/32XDCPM6IPOD7SACIFJKB74PP3HSDHI2/ >>>> >>> _______________________________________________ >>> Devel mailing list -- devel@ovirt.org >>> To unsubscribe send an email to devel-le...@ovirt.org >>> Privacy Statement: https://www.ovirt.org/site/privacy-policy/ >>> oVirt Code of Conduct: >>> https://www.ovirt.org/community/about/community-guidelines/ >>> List Archives: >>> https://lists.ovirt.org/archives/list/devel@ovirt.org/message/LV6XFFQPI4ZFZ2DNJZNHEVBGIASEGYPV/ >>> >> _______________________________________________ > Devel mailing list -- devel@ovirt.org > To unsubscribe send an email to devel-le...@ovirt.org > Privacy Statement: https://www.ovirt.org/site/privacy-policy/ > oVirt Code of Conduct: > https://www.ovirt.org/community/about/community-guidelines/ > List Archives: > https://lists.ovirt.org/archives/list/devel@ovirt.org/message/THBBN6G2ETAEEGCFGIU7CEKUNBZVCCEF/ >
_______________________________________________ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-le...@ovirt.org Privacy Statement: https://www.ovirt.org/site/privacy-policy/ oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/devel@ovirt.org/message/EJJDUS3GSNPLY34JLVIQ2LLKUE3HAKTM/