Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Barak Korren
On 13 November 2017 at 12:44, Allon Mureinik wrote: > Both are merged, let's see how OST fares > The 1st one failed, but the 2nd passed: http://jenkins.ovirt.org/job/ovirt-master_change-queue-tester/3792 So we're over this hill. -- Barak Korren RHV DevOps team , RHCE, RHCi Red Hat EMEA redhat.c

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Allon Mureinik
Both are merged, let's see how OST fares On Mon, Nov 13, 2017 at 12:00 PM, Allon Mureinik wrote: > I merged 83920 with dependencies. Perhaps gerrit doesn't handle such > chains properly (or perhaps our CI is misusing the API?). > > WRT current issues, posted two patches upstream: > - https://ger

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Allon Mureinik
I merged 83920 with dependencies. Perhaps gerrit doesn't handle such chains properly (or perhaps our CI is misusing the API?). WRT current issues, posted two patches upstream: - https://gerrit.ovirt.org/#/c/83986/ - fixes the GWT build that was broken this morning, preventing a proper build altoge

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Barak Korren
On 13 November 2017 at 11:10, Allon Mureinik wrote: > Commit 86bf82e746404145e8b97df46f514086e4f82e69 is probably the offending > commit, taking a deeper look now, should have a fix within the hour > Hmm, I see what happened there, that commit is https://gerrit.ovirt.org/c/83920, so it was tested

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Allon Mureinik
Commit 86bf82e746404145e8b97df46f514086e4f82e69 is probably the offending commit, taking a deeper look now, should have a fix within the hour On Mon, Nov 13, 2017 at 9:55 AM, Barak Korren wrote: > Test failed: [ add_master_storage_domain ] > > Link to suspected patches: > - https://gerrit.ovirt.

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Allon Mureinik
It's most definitely not THAT patch. Could be one of the earlier patches in the series, though. Looking. On Mon, Nov 13, 2017 at 10:54 AM, Barak Korren wrote: > On 13 November 2017 at 10:08, Nir Soffer wrote: > > > > > > On Mon, Nov 13, 2017 at 9:56 AM Barak Korren wrote: > >> > >> Test faile

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Barak Korren
On 13 November 2017 at 10:08, Nir Soffer wrote: > > > On Mon, Nov 13, 2017 at 9:56 AM Barak Korren wrote: >> >> Test failed: [ add_master_storage_domain ] >> >> Link to suspected patches: >> - https://gerrit.ovirt.org/#/c/83849/4 > > > This fixes grammar in few error messages, I don't see how thi

Re: [ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-13 Thread Nir Soffer
On Mon, Nov 13, 2017 at 9:56 AM Barak Korren wrote: > Test failed: [ add_master_storage_domain ] > > Link to suspected patches: > - https://gerrit.ovirt.org/#/c/83849/4 This fixes grammar in few error messages, I don't see how this can cause failures unless someone is depending on the error tex

[ovirt-devel] [ OST Failure Report ] [ oVirt master ] [ 2017-11-13 ] [add_master_storage_domain]

2017-11-12 Thread Barak Korren
Test failed: [ add_master_storage_domain ] Link to suspected patches: - https://gerrit.ovirt.org/#/c/83849/4 This seems to be a fairly consistent regression as all patches that follow the patch above exhibit the same issue when tested, this list of similarly failing patches includes: - https://ger