1342155 - disabled too many osx-debug tests, ones that are not run in e10s mode

Status: RESOLVED FIXED

(Infrastructure & Operations Graveyard :: CIDuty, task)

Description

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

we have xpcshell running, but we need to turn back on these non-e10s tests:
gtest
cppunit
jittest
mochitest-jetpack (needed until deprecated)

Comment 1

[Alin Selagea [:aselagea]]

So we only want to remove the non-e10s tests that *do* have an e10s equivalent?
That would mean:
    - the four tests listed in this bug's description
    - mochitest-a11y, mochitest-chrome, xpcshell (which were explicitly kept enabled in bug 1339185)
    - media-tests
    - media-youtube-tests

Do we want to keep 'media-tests' and 'media-youtube-tests' as well? (since they don't have an e10s version)

Comment 2

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

yes, we should keep those as well.

:RyanVM, can you ensure there is a bug on file and in the right hands for making the media*tests work in e10s mode?

Comment 3

[Alin Selagea [:aselagea]]

Attachment: bug_1342155.patch

This should enable the missing tests back and make sure no non-e10s is dropped until the corresponding e10s version is present.

Comment 4

[Alin Selagea [:aselagea]]

Attachment: diff.txt

Tests added.

Comment 5

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

Comment on attachment 8840825 [details] [diff] [review]
bug_1342155.patch

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

this looks good, but we have so many edge cases, can you show me a builder diff?

Comment 6

[Alin Selagea [:aselagea]]

(In reply to Joel Maher ( :jmaher) from comment #5)
> Comment on attachment 8840825 [details] [diff] [review]
> bug_1342155.patch
> 
> Review of attachment 8840825 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> this looks good, but we have so many edge cases, can you show me a builder
> diff?

This doesn't affect the builders, since we do the changes on mozilla-tests/config.py. The difference in tests can be seen in 'diff.txt' (attached to this bug).

Comment 7

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

that diff looks good!  thanks

Comment 8

[Alin Selagea [:aselagea]]

Merged the change in production: https://hg.mozilla.org/build/buildbot-configs/rev/da14345b72d4

Comment 9

[Ryan VanderMeulen [:RyanVM]]

(In reply to Joel Maher ( :jmaher) from comment #2)
> :RyanVM, can you ensure there is a bug on file and in the right hands for
> making the media*tests work in e10s mode?

I have no idea why you're asking me to do this. I don't even know who owns them. I suggest pinging Maire and going from there.

Comment 10

[Alin Selagea [:aselagea]]

I think the work here is done, marking the bug as fixed.