Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-26 Thread Barak Korren
> > https://gerrit.ovirt.org/#/c/68078/ broke the check-merged job with > really tough exception: > > 15:23:34 sh: [17766: 1 (255)] tcsetattr: Inappropriate ioctl for device > 15:23:34 Took 2586 seconds > 15:23:34 Slave went offline during the build This isa what I was

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-25 Thread Dan Kenigsberg
On Thu, Dec 22, 2016 at 5:59 PM, Leon Goldberg wrote: > So it's not about the added network tests (these were added in the one I've > mentioned, the one who had successful runs after being merged), and I don't > see how the patch you've mentioned breaks anything (as it merely

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-23 Thread Barak Korren
> > http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/778/console > > 11:55:54 RuntimeError: Failed to run reposync 3 times for repoid: > ovirt-master-snapshot-static-el7, aborting. > > It is wrong to pick up on the first patch that happened to see the > "Groovy thread" exception.

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-23 Thread Barak Korren
בתאריך 23 בדצמ׳ 2016 12:46 PM,‏ "Dan Kenigsberg" כתב: On Thu, Dec 22, 2016 at 5:45 PM, Yaniv Bronheim wrote: > On Thu, Dec 22, 2016 at 5:39 PM, Leon Goldberg wrote: > >> Doesn't seem related; the patch does nothing but move pieces

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-23 Thread Dan Kenigsberg
Yaniv, Maybe you can look at http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/791/consoleText Unlike other, this was actually run by CI, and failed to start vdsm on the lago host. "A dependency job for vdsmd.service failed. See 'journalctl -xe' for details." The cause might be

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-23 Thread Dan Kenigsberg
On Thu, Dec 22, 2016 at 5:45 PM, Yaniv Bronheim wrote: > On Thu, Dec 22, 2016 at 5:39 PM, Leon Goldberg wrote: > >> Doesn't seem related; the patch does nothing but move pieces around. >> >> Judging by the title I guess you're referring to >>

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-22 Thread Leon Goldberg
Doesn't seem related; the patch does nothing but move pieces around. Judging by the title I guess you're referring to https://gerrit.ovirt.org/#/c/67787/ ? On Thu, Dec 22, 2016 at 5:14 PM, Yaniv Bronheim wrote: > Hi guys and Leon, > > https://gerrit.ovirt.org/#/c/68078/

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-22 Thread Leon Goldberg
So it's not about the added network tests (these were added in the one I've mentioned, the one who had successful runs after being merged), and I don't see how the patch you've mentioned breaks anything (as it merely moves pieces around). I could of course be wrong entirely and there's something

Re: [ovirt-devel] Check-merged is broken since we added network test to it

2016-12-22 Thread Yaniv Bronheim
On Thu, Dec 22, 2016 at 5:39 PM, Leon Goldberg wrote: > Doesn't seem related; the patch does nothing but move pieces around. > > Judging by the title I guess you're referring to > https://gerrit.ovirt.org/#/c/67787/ ? > no.. you can see that after this patch it used to

[ovirt-devel] Check-merged is broken since we added network test to it

2016-12-22 Thread Yaniv Bronheim
Hi guys and Leon, https://gerrit.ovirt.org/#/c/68078/ broke the check-merged job with really tough exception: *15:23:34* sh: [17766: 1 (255)] tcsetattr: Inappropriate ioctl for device*15:23:34* Took 2586 seconds*15:23:34* Slave went offline during the build