660363 - Enable all m-c tests in projects/electrolysis repo

Status: VERIFIED FIXED

(Release Engineering :: General, defect, P2)

Description

[:Felipe Gomes (needinfo for replies!)]

Talos tests are currently disabled in the electrolysis project branch. We're going to merge to/from mozilla-central and this branch now, so we'll need test parity between the two.

On a first glance it looks like that the hidden Jetpack tests are also not present in electrolysis tbpl. Don't know how to verify that there are no other tests missing

Comment 1

[Armen [:armenzg]]

Attachment: [configs] add talos to electrolysis

Comment 2

[Armen [:armenzg]]

(In reply to comment #0)
> Talos tests are currently disabled in the electrolysis project branch. We're
> going to merge to/from mozilla-central and this branch now, so we'll need
> test parity between the two.
> 
> On a first glance it looks like that the hidden Jetpack tests are also not
> present in electrolysis tbpl. Don't know how to verify that there are no
> other tests missing
The patch adds the talos coverage that you are missing.
You already have the set of default unit tests that get added for project branches.
For now we are holding on enabling few suites like jetpack to just m-c and try since we are overloaded on our testing infrastructure and they needed to stabilize first.
These will be added in another bug.

Comment 3

[:Felipe Gomes (needinfo for replies!)]

(In reply to comment #2)
> The patch adds the talos coverage that you are missing.
> You already have the set of default unit tests that get added for project
> branches.
> For now we are holding on enabling few suites like jetpack to just m-c and
> try since we are overloaded on our testing infrastructure and they needed to
> stabilize first.
> These will be added in another bug.

sounds good, thanks for the info

Comment 4

[Armen [:armenzg]]

Comment on attachment 536093 [details] [diff] [review]
[configs] add talos to electrolysis

checked-in but not yet deployed:
http://hg.mozilla.org/build/buildbot-configs/rev/1b1dac6b9235

Comment 5

[Armen [:armenzg]]

It has been deployed now.

felipe could you please verify once you guys push or merge into m-c if you get talos coverage?

Comment 6

[:Felipe Gomes (needinfo for replies!)]

latest push to electrolysis got talos coverage