902124 - Write mozharness script for gaia integration tests

Status: RESOLVED FIXED

(Release Engineering :: Applications: MozharnessCore, defect)

Description

[Jonathan Griffin (:jgriffin)]

We're getting close to the point where we can start writing a mozharness script for invoking gaia-integration-tests in buildbot.

Comment 1

[Gareth Aye [:gaye] (back from PTO)]

Attachment: Patch for bug 902124

Hi Aki!

Thanks for all of your help. This patch adds a new mozharness script to run |MOCHA_REPORTER=mocha-tbpl-reporter| in the gaia directory and then publish the results to TBPL. Would you mind taking a look? I'd also like to know how I can push this to a staging-type environment to make sure that it works like I expect for it to. Do you all usually run these locally? There weren't any tests to exercise the old gaia_unit.py, so I'm a bit confused.

Comment 2

[Gareth Aye [:gaye] (back from PTO)]

Attachment: Pull request to Gaia

Hi James,

In addition to the patch I sent to Aki, I needed a tiny bit of plumbing to help the Makefile propagate choosing the mocha-tbpl-reporter over the dot reporter for our marionette tests.

Comment 3

[James Lal [:lightsofapollo]]

Comment on attachment 809372 [details] [review]
Pull request to Gaia

wait for the build to go green then land :D

Comment 4

[Aki Sasaki (not active)]

Comment on attachment 809361 [details] [diff] [review]
Patch for bug 902124

I think this is overall good.

However, I don't think we should be importing anything from scripts/.
Can you move scripts/gaia.py to mozharness/mozilla/testing/gaia.py ?
Then the +sys.path.insert(1, os.path.dirname(sys.path[0])) line will need to change (three layers of os.path.dirname()) and the |+from scripts.gaia import GaiaTest| line will need to change to |from mozharness.mozilla.testing.gaia import GaiaTest|.

Other than that this looks good.  Thanks!

Comment 5

[Gareth Aye [:gaye] (back from PTO)]

Sure I'll make those changes... thanks Aki!

Comment 6

[Gareth Aye [:gaye] (back from PTO)]

Attachment: Updated patch to mozharness

Aki! Please check this in for me!

Comment 7

[Aki Sasaki (not active)]

Comment on attachment 815148 [details] [diff] [review]
Updated patch to mozharness

Checked in, minus the .hgtags changes.
https://hg.mozilla.org/build/mozharness/rev/e2e19076b0d0

Comment 8

[John Hopkins (:jhopkins)]

in production