Closed Bug 1527108 Opened 5 years ago Closed 5 years ago

mozmill timeout in wait_and_create_network too short. "Sorry, cannot connect to jsbridge extension"

Categories

(Thunderbird :: Testing Infrastructure, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 67.0

People

(Reporter: KaiE, Assigned: KaiE)

Details

Attachments

(2 files)

We spent a long time trying to figure out why mozmill isn't working for me.

I didn't assume my laptop is too slow, but apparently nobody else is using a 2.8 ghz laptops with a debug build to run mozmill?

mail/test/resources/jsbridge/jsbridge/init.py contains a 60 second timeout. I need 90 seconds to make it work.

BTW, after updating that file and running ./mach build, the copy at
objdir/_virtualenvs/init/local/lib/python2.7/site-packages/jsbridge/init.py
isn't updated, which further slowed down the analysis of the cause.

Attached patch 1527108-v1.patchSplinter Review
Assignee: nobody → kaie
Attachment #9043115 - Flags: review?(acelists)
Comment on attachment 9043115 [details] [diff] [review]
1527108-v1.patch

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

Thanks.
Attachment #9043115 - Flags: review?(acelists) → review+
Attached patch 1527108-mz.patchSplinter Review

Increase the wait also on mozmill side.

Attachment #9043119 - Flags: review?(jorgk)
Comment on attachment 9043119 [details] [diff] [review]
1527108-mz.patch

OK, I'll land this now.
Attachment #9043119 - Flags: review?(jorgk) → review+

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/21352b679dc6
Increase JS Bridge timeout to 90 seconds. r=aceman
https://hg.mozilla.org/comm-central/rev/b71df5ca4cb2
Increase MozMill timeout to wait for JS Bridge startup. r=jorgk

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 67.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: