1769110 - adjust mochitest harness to run with conditioned profiles when specified

Status: RESOLVED FIXED

(Testing :: General, task)

Description

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

working from the work in bug 1769098 to get condprof tools available, we need to have the ability within mochitest to specify we are using a conditioned profile and use it for the tests.

Comment 1

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

Attachment: Bug 1769110 - Add support to mochitest harness for conditioned profiles to be used while testing. r=gbrown!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6a5777839117
Add support to mochitest harness for conditioned profiles to be used while testing. r=gbrown,asuth

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7769fe500298
Fix py black lint failure in testing/mochitest/runtests.py CLOSED TREE

Comment 4

[Noemi Erli[:noemi_erli]]

Backed out 2 changesets (Bug 1769110) for causing mochitest failures CLOSED TREE

Push with failures: https://treeherder.mozilla.org/jobs?repo=autoland&group_state=expanded&collapsedPushes=842382&selectedTaskRun=AL_dJy_gQO2Yn3QfSVcxlA.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel&revision=6a57778391178a749f9f318064755eca661caf57

Log: https://treeherder.mozilla.org/logviewer?job_id=379823953&repo=autoland&lineNumber=1656

Backout: https://hg.mozilla.org/integration/autoland/rev/b19e04d1066e72e3f90e9a33209e57536ddfed27

Comment 5

[Andrew Halberstadt [:ahal]]

Looks like jmaher has needinfos blocked, which I guess is why I was needinfo'ed here.

But I'm heading on PTO, so redirecting to gbrown.

Comment 6

[Geoff Brown [:gbrown]]

Failures are osx-only; caused by bug 1769655 which intentionally excluded condprof on osx. Backout 1769655 and find a better way to handle that bug, or import condprof conditionally in runtests.py?

Comment 7

[Geoff Brown [:gbrown]]

I think the easiest way forward, assuming condprof is not required on osx, is to simply move the condprof imports into downloadConditionedProfile():

https://treeherder.mozilla.org/jobs?repo=try&revision=928ae4796943661e2b7ec9510afdef6730e6a85d

Comment 8

[Geoff Brown [:gbrown]]

Comment 7 has a simple fix, but probably best to wait for jmaher to return, unless this is blocking someone.

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7f2bad2e7344
Add support to mochitest harness for conditioned profiles to be used while testing. r=gbrown,asuth

Comment 10

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/7f2bad2e7344