1362396 - Make CI tests for DevEdition visible on Treeherder

Status: RESOLVED FIXED

(Firefox Build System :: Task Configuration, task)

Description

[Sebastian Pacurar [:spacurar]]

Please make the tests for DevEdition platforms visible on treeherder.

Comment 1

[Sebastian Pacurar [:spacurar]]

Attachment: bug_1362396_linux64-talos.patch

I added the devedition variant to mozilla-inbound. Is this the correct place or should it be added to mozilla-beta/jamun?

Comment 2

[Ed Morley [:emorley]]

Ah if this bug is for adding support for devedition tests, then the Taskcluster component is more appropriate.

From the current bug summary it seemed as though this was requesting that the sheriffs change the visibility of an existing job via the "visibility profiles" feature.

Comment 3

[Kim Moir [:kmoir] ET]

Comment on attachment 8870068 [details] [diff] [review]
bug_1362396_linux64-talos.patch

We should test it first on jamun before we land the patches on inbound.

On jamun you will need something like this add talos tests
diff --git a/taskcluster/ci/test/test-platforms.yml b/taskcluster/ci/test/test-platforms.yml
--- a/taskcluster/ci/test/test-platforms.yml
+++ b/taskcluster/ci/test/test-platforms.yml
@@ -55,16 +55,17 @@ linux64/opt:
 linux64-devedition/opt:
     build-platform: linux64-devedition-nightly/opt
     test-sets:
         - common-tests
         - web-platform-tests
         - opt-only-tests
         - desktop-screenshot-capture
         - awsy
+        - talos


The current patch doesn't apply cleanly to jamun.

Also, if you look at taskcluster/ci/test/tests.yml  the talos tests are all specify  run-on-projects and jamun is not liated as a branch, so this will have to be updated.

Comment 4

[Sebastian Pacurar [:spacurar]]

Attachment: bug_1362396_linux64-talos-jamun.patch

Enabled jamun for talos tests in taskcluster/ci/test/tests.yml, added talos for linux64-devedition/opt test builder, added the variant for devedition.

Comment 5

[Sebastian Pacurar [:spacurar]]

Attachment: bug_1362396_v2_linux64-talos-jamun.patch

Sorry for the previous patch, it had a flaw in the code. I fixed it now.

Enabled jamun for talos tests in taskcluster/ci/test/tests.yml, added talos for linux64-devedition/opt test builder, added the variant for devedition.

Comment 6

[Kim Moir [:kmoir] ET]

Comment on attachment 8870792 [details] [diff] [review]
bug_1362396_v2_linux64-talos-jamun.patch

Looks good.  We can hold off landing this until we have the corresponding buildbot jobs enabled, otherwise the jobs will just fail with a payload exception.

Comment 7

[Sebastian Pacurar [:spacurar]]

Attachment: diff for bug_1362396_v2_linux64-talos-jamun.diff

This is the diff after enabling linux64 talos on jamun.

Comment 8

[Sebastian Pacurar [:spacurar]]

I believe this can be marked as fixed now.