1196393 - Install psutil into the virtualenv created by desktop_unittest.py

Status: RESOLVED FIXED

(Release Engineering :: Applications: MozharnessCore, defect)

Description

[Chris Manchester (limited bugmail, email directly)]

Work is underway to make it available for all machines running Talos, I'd like to use it for something in Mochitests -- I think the only thing that remains is figuring out how to build a .whl that will work on our OSX 10.6 testers.

Comment 1

[Chris Manchester (limited bugmail, email directly)]

Confirming OSX 10.6 is the only problem area: https://treeherder.mozilla.org/#/jobs?repo=try&revision=5dfc5b338bd8

Comment 2

[Chris Manchester (limited bugmail, email directly)]

Attachment: psutil-3.1.1-cp27-none-macosx_10_4_x86_64.whl

This is the .whl I generated from a 10.6 tester for upload to internal pypi

Comment 3

[Chris Manchester (limited bugmail, email directly)]

(In reply to Chris Manchester [:chmanchester] from comment #1)
> Confirming OSX 10.6 is the only problem area:
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=5dfc5b338bd8

With comment 2 uploaded to pypi, re-triggers here are succeeding.

Comment 4

[Chris Manchester (limited bugmail, email directly)]

Attachment: MozReview Request: Bug 1196393 - Install psutil 3.1.1 in the desktop_unittests vitualenv so it will be available to desktop unittests. r=jlund

Bug 1196393 - Install psutil 3.1.1 in the desktop_unittests vitualenv so it will be available to desktop unittests. r=jlund

Comment 5

[Jordan Lund (:jlund)]

Comment on attachment 8652568 [details]
MozReview Request: Bug 1196393 - Install psutil 3.1.1 in the desktop_unittests vitualenv so it will be available to desktop unittests. r=jlund

https://reviewboard.mozilla.org/r/17191/#review15411

I see win32 and win64 whl's were recently uploaded to pypi too.

::: testing/mozharness/scripts/desktop_unittest.py:259
(Diff revision 1)
> +        self.register_virtualenv_module(name='pip>=1.5')

why is this needed? I would have thought we would have 'pip' available already?

Comment 6

[Julien Pagès (:parkouss)]

It is required because pip only prefer wheel format starting with version >= 1.5 (https://packaging.python.org/en/latest/installing.html#installing-cached-wheels).

Same kind of thing in bug 1194074 and I had some issues without that - if I remember well some win32 test machines are using an older pip by default.

Comment 7

[Jordan Lund (:jlund)]

(In reply to Julien Pagès from comment #6)
> It is required because pip only prefer wheel format starting with version >=
> 1.5
> (https://packaging.python.org/en/latest/installing.html#installing-cached-
> wheels).
> 
> Same kind of thing in bug 1194074 and I had some issues without that - if I
> remember well some win32 test machines are using an older pip by default.

thanks for the explanation.

Comment 8

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cd9a769c5ac2

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/cd9a769c5ac2