1190265 - update talos to use latest mozbase dependencies

Status: RESOLVED FIXED

(Testing :: Talos, defect)

Description

[Julien Pagès (:parkouss)]

We should use newer mozprocess into talos. It should not change anything - but some important changes in process output handling were made into mozprocess 0.22, so eventually this could lead to regressions, or bug fixes (relative to the "5 seconds required sleep after process" for example).

Comment 1

[Julien Pagès (:parkouss)]

Now that we resolved the logging stuff, I thing we should try to update to every latest dependency in one go. Else we are going into troubles because of mozlog 3.0 compatibility issue with older versions.

Comment 2

[Julien Pagès (:parkouss)]

pushed to try!

https://treeherder.mozilla.org/#/jobs?repo=try&revision=6ae1b7f2fa27

Comment 3

[Julien Pagès (:parkouss)]

Attachment: update_mozbase_deps.patch

Oh, and here is the patch. This updates every mozbase dependencies to latests - available on internal pypi, really similar to public pypi.

Also this do not enforce versions. Since we are going to use in tree stuff, this is not a bad idea I think.

Comment 4

[Julien Pagès (:parkouss)]

PGO build:

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a41acd9131ad

Comment 5

[Julien Pagès (:parkouss)]

All green! --10 times now, but I think we are good.

Comment 7

[Julien Pagès (:parkouss)]

Yay, everything is just green! We should land that right now. :)

Comment 8

[Joel Maher ( :jmaher ) (UTC -8)]

I am concerned about the windows pgo stuff, that is where we had problems before, although I suspect it won't be as much of a problem.  I say we get this in and land on inbound asap, then we can backout if needed.

Comment 9

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8644457 [details] [diff] [review]
update_mozbase_deps.patch

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

much cleaner.

Comment 10

[Julien Pagès (:parkouss)]

Hmm, on PGO build, dromaeo win7 is failing every time.

Comment 11

[Joel Maher ( :jmaher ) (UTC -8)]

yeah, we disabled dromaeo_dom in general because it fails in win7 e10s pgo all the time.  That test is just problematic and if it is a single test causing problems in a variety of environments, the problem is the test :)

This is the test which caused problems before.

Comment 12

[Julien Pagès (:parkouss)]

Thanks for the explanation!

Landed in https://hg.mozilla.org/build/talos/rev/6e62b920e8d8.