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 769674 - some GIO tests' arbitrary timeouts are too short
some GIO tests' arbitrary timeouts are too short
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: gio
2.48.x
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks:
 
 
Reported: 2016-08-09 15:36 UTC by Simon McVittie
Modified: 2017-10-11 12:34 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
gio/tests/file: increase an arbitrary timeout (822 bytes, patch)
2016-08-09 15:37 UTC, Simon McVittie
committed Details | Review
gio/tests/gdbus-proxy: sleep longer when testing that we time out (2.60 KB, patch)
2016-08-09 15:37 UTC, Simon McVittie
committed Details | Review

Description Simon McVittie 2016-08-09 15:36:33 UTC
On a heavily loaded and not particularly fast ARM autobuilder (multiple packages building in parallel), gio/tests/file and gio/tests/gdbus-proxy sometimes fail. See the individual patches for more details.

Not building packages in parallel on this particular machine is not really an option: we're trying to balance throughput with latency, and this is the least-bad configuration.
Comment 1 Simon McVittie 2016-08-09 15:37:24 UTC
Created attachment 333001 [details] [review]
gio/tests/file: increase an arbitrary timeout

On slow ARM machines doing parallel builds, there's no guarantee that
we'll get through this in 5 seconds.
Comment 2 Simon McVittie 2016-08-09 15:37:56 UTC
Created attachment 333002 [details] [review]
gio/tests/gdbus-proxy: sleep longer when testing that we time out

On slow ARM machines doing parallel builds, there's no guarantee that
we'll get scheduled in a window between (100ms|250ms) and 500ms.
Comment 3 Hyungwon Hwang 2016-08-11 06:47:51 UTC
I also have experienced similar situations too.

Looks reasonable to fix it.
Comment 4 Philip Withnall 2017-10-11 12:06:04 UTC
Review of attachment 333002 [details] [review]:

Looks good to me, assuming you still need this for your builds?
Comment 5 Philip Withnall 2017-10-11 12:06:09 UTC
Review of attachment 333001 [details] [review]:

Looks good to me, assuming you still need this for your builds?
Comment 6 Simon McVittie 2017-10-11 12:34:04 UTC
Comment on attachment 333001 [details] [review]
gio/tests/file: increase an arbitrary timeout

7d5c738d
Comment 7 Simon McVittie 2017-10-11 12:34:22 UTC
Comment on attachment 333002 [details] [review]
gio/tests/gdbus-proxy: sleep longer when testing that we time out

331f73c2
Comment 8 Simon McVittie 2017-10-11 12:34:57 UTC
Fixed in git for 2.55.0, thanks for reviewing!