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 I'm missing about
the patch you're claiming is the guilty one, but it won't be about the
network tests either way.
If it's then not about the network tests, then my immediate suspicion is
that the problem lies somewhere else and isn't in either of the patches. I
am far from able to prove this either way, though, so if reverting the
patch you've mentioned somehow fixes check-merged, then it is of course
fine by me.
On Thu, Dec 22, 2016 at 5:45 PM, Yaniv Bronheim <ybronhei(a)redhat.com> wrote:
On Thu, Dec 22, 2016 at 5:39 PM, Leon Goldberg <lgoldber(a)redhat.com>
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 work (check the jobs
after the merge)
so something in
https://gerrit.ovirt.org/#/c/68078/ broke it
>
> On Thu, Dec 22, 2016 at 5:14 PM, Yaniv Bronheim <ybronhei(a)redhat.com>
> wrote:
>
>> 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
<
http://jenkins.ovirt.org/computer/vm0136.workers-phx.ovirt.org/log>*15...
ERROR: Connection was broken: java.io.IOException: Unexpected termination of the
channel*15:23:34* at
hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:50)*15:23:34*
Caused by: java.io.EOFException*15:23:34* at
java.io.ObjectInputStream$PeekInputStream.readFully(ObjectInputStream.java:2353)*15:23:34*
at
java.io.ObjectInputStream$BlockDataInputStream.readShort(ObjectInputStream.java:2822)*15:23:34*
at java.io.ObjectInputStream.readStreamHeader(ObjectInputStream.java:804)*15:23:34* at
java.io.ObjectInputStream.<init>(ObjectInputStream.java:301)*15:23:34* at
hudson.remoting.ObjectInputStreamEx.<init>(ObjectInputStreamEx.java:48)*15:23:34*
at
hudson.remoting.AbstractSynchronousByteArrayCommandTransport.read(AbstractSynchronousByteArrayCommandTransport.java:34)*15:23:34*
at
hudson.remoting.SynchronousCommandTransport$ReaderThread.run(SynchronousCommandTransport.java:48)*15:23:34*
*15:23:34* Build step 'Execute shell' marked build as failure*15:23:34* Performing
Post build task...
>>
>>
>> I have no clue what causes it.. we need to investigate in the tests code.
>>
>>
>> you can see it in
http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/772/cons...
>>
>>
>> and this for a job run just before it got in, which worked well -
http://jenkins.ovirt.org/job/vdsm_master_check-merged-el7-x86_64/688/console
>>
>>
>> I suggest to revert this patch (and backport the revert to ovirt-4.1 branch as
well) until figuring what causes it
>>
>>
>> --
>> *Yaniv Bronhaim.*
>>
>
>
--
*Yaniv Bronhaim.*