Closed Bug 1449350 Opened 3 years ago Closed 3 years ago

distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('six')


(Release Engineering :: General, defect)

Not set


(firefox61 fixed)

Tracking Status
firefox61 --- fixed


(Reporter: nthomas, Assigned: nthomas)




(2 files, 1 obsolete file)

Particularly on esr52, in the first fifteen minutes of the hour. eg:
Comment on attachment 8962908 [details] [diff] [review]
[gecko] Move six earlier in websocketprocessbridge_requirements.txt

We could even move this to the top, above twisted; six doesn't require twisted, but an unpinned twisted dep may require six at some point.
Attachment #8962908 - Flags: review?(aki) → review+
Updated patch, carrying r+
Attachment #8962908 - Attachment is obsolete: true
Attachment #8962910 - Flags: review+
Pushed by
distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('six'), r=aki
The backstory here is
* we saw a lot of red windows tests jobs on
* these all run on older hardware, t-w732, with python not able to talk TLSv1.2
* if we use pip to install a package containing a which specifies requirements, then setuptools doesn't respect the args we've given to pip to install from our pypi servers, so it talks to
* is migrating to a new CDN provider, and having brownouts on TLSv1.0 and v1.1 for the first 15 minutes of the hour

So jobs fail when the starts align. The (attempted) fix is get pip to grab six before txWS needs it, so we setuptools doesn't need to reach out for anything. I presume we don't have test failures on newer branches because they can still talk to external pypi, but they shouldn't be doing so.
We're still hitting errors with attachment 8962910 [details] [diff] [review], eg

Here's your suggestion to install six immediately after setting up the virtualenv.
Attachment #8962970 - Flags: review?(aki)
Attachment #8962970 - Flags: review?(aki) → review+
Comment on attachment 8962970 [details] [diff] [review]
[gecko] Add six to initial virtualenv setup

Backout earlier patch:

Land this one with version pinned:

Will follow up on inbound/beta once we have some data about effectiveness.
Closed: 3 years ago
Resolution: --- → FIXED
Resolution: FIXED → ---
Note to self - come back here and figure out what we want to do with syncing branches.
Flags: needinfo?(nthomas)
Component: General Automation → General
See Also: → 1459391
Blocks: 1460684
(In reply to Nick Thomas [:nthomas] (UTC+12) from comment #8)
> Backout earlier patch:
> ebb75ac4798c5e17be6233ad82d2ad8e64ed4585

Relanded as because something broke between dc927ef3e774 on May 26 and restoring buidlbot jobs today (bug 1465288). Possibly 1463592.
I think we're done here.
Closed: 3 years ago3 years ago
Flags: needinfo?(nthomas)
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.