After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 722336 - tls/connection/close-during-handshake has become unreliable
tls/connection/close-during-handshake has become unreliable
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: network
2.39.x
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
: 719727 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2014-01-16 13:56 UTC by Iain Lane
Modified: 2014-01-20 17:17 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
tls/tests: try to fix a flaky test (2.27 KB, patch)
2014-01-18 20:34 UTC, Dan Winship
committed Details | Review

Description Iain Lane 2014-01-16 13:56:43 UTC
In Ubuntu, we run glib-networking's installed-tests automatically as part of regression testing.

Lately the test /tls/connection/close-during-handshake started failing intermittently:

  /tls/connection/close-during-handshake: **
GLib-Net:ERROR:connection.c:131:teardown_connection: assertion failed: (!(test->client_connection))
Test glib-networking/connection.test failed: Child process killed by signal 6

https://jenkins.qa.ubuntu.com/view/Trusty/view/AutoPkgTest/job/trusty-adt-glib-networking/29/ARCH=i386,label=adt/console

After looking at the upload timings of glib, this seems to have begun with the upload of glib 2.39.2. bug #721921 is for some glib test flakiness that happened around the same time. Ryan blamed that one on the test itself.
Comment 1 Dan Winship 2014-01-16 14:03:15 UTC
what version of glib-networking exactly?
Comment 2 Iain Lane 2014-01-16 14:06:42 UTC
Oops, I'm sorry - I forgot to say. It's 2.39.3.
Comment 3 Dan Winship 2014-01-18 20:34:54 UTC
Created attachment 266631 [details] [review]
tls/tests: try to fix a flaky test

----

could you try applying this patch to your package to see if it makes
the failures go away?
Comment 4 Iain Lane 2014-01-20 10:45:15 UTC
Cheers, I've upload it to Ubuntu. If you like, keep an eye on

  https://jenkins.qa.ubuntu.com/view/Trusty/view/AutoPkgTest/job/trusty-adt-glib-networking/

which should get new test results shortly. If the first result passes, I'll
then retry it a few times to see if it's solid.

(Even if not, this patch looks sensible to me)
Comment 5 Iain Lane 2014-01-20 11:57:56 UTC
Oops, it happens for write-during-handshake (as well?). I've applied the same fix there; testing that now.
Comment 6 Iain Lane 2014-01-20 12:56:27 UTC
Review of attachment 266631 [details] [review]:

I've run it three times with the fix to write_during_handshake too and it's been fine every time. Thanks.
Comment 7 Dan Winship 2014-01-20 17:15:37 UTC
pushed with the fix for write_during_handshake as well. thanks
Comment 8 Dan Winship 2014-01-20 17:17:48 UTC
*** Bug 719727 has been marked as a duplicate of this bug. ***