If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Default timeout for wait_for_port is 3000s (50 minutes)

RESOLVED FIXED in mozilla26

Status

Testing
Marionette
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jgraham, Assigned: Debloper)

Tracking

unspecified
mozilla26
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][lang=python][mentor=mdas])

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
The default timeout of 3000s seems excessively long. Possibly it is supposed to be 3s? In that case it would make sense to retry more than once per second.

(It is also not clear why there is a 5s wait after the port has been opened).
I think 30seconds is perfect for wait for port, since it accommodates the case where we restart b2g.

As a first bug:
To make this change, you'll need to look at (mozilla-central source_testing/marionette/client/marionette/marionette.py and modify the default value for the wait_for_port function and how it uses the default value (ie: what it passes to the timedelta function)
Whiteboard: [good first bug][lang=python][mentor=mdas]
(Assignee)

Updated

4 years ago
Assignee: nobody → debloper
OS: Linux → All
Hardware: x86_64 → All
(Assignee)

Comment 2

4 years ago
Created attachment 802621 [details] [diff] [review]
v1.patch
Attachment #802621 - Flags: review?(mdas)
Comment on attachment 802621 [details] [diff] [review]
v1.patch

Review of attachment 802621 [details] [diff] [review]:
-----------------------------------------------------------------

Great, thanks! I've tested this locally against the unit test suite and it works as expected. Will land in m-i/m-c shortly.
Attachment #802621 - Flags: review?(mdas) → review+
I've pushed it to m-i:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e0da7141fa04


by the way, Debloper, I had to add your username to the patch so you'd be recognized as the one who wrote the change. You should add your username to your hg configuration so your username will show up in future patches automatically: https://developer.mozilla.org/en-US/docs/Installing_Mercurial#Basic_configuration
https://hg.mozilla.org/mozilla-central/rev/e0da7141fa04
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla26
(Assignee)

Comment 6

4 years ago
(In reply to Malini Das [:mdas] from comment #4)
> I've pushed it to m-i:
> https://hg.mozilla.org/integration/mozilla-inbound/rev/e0da7141fa04
Awesome, thanks!

> by the way, Debloper, I had to add your username to the patch so you'd be
> recognized as the one who wrote the change.
/me checks the hgrc config... *seems alright*
/me checks the patch again... *no author info*
/me isn't sure what went wrong, but sure as hell it's one such case!
Thanks, bytheway!
You need to log in before you can comment on or make changes to this bug.