Andrew Halberstadt [:ahal] Assignee
	Description • 10 years ago

I need to test some changes from bug 1059951 on ash-mozharness. Unfortunately that requires code coverage builds to be scheduled there as well..

	Jordan Lund (:jlund)
	Comment 2 • 10 years ago

Comment on attachment 8501796 [details] [diff] [review]
Schedule linux64-cc on Ash

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

*sigh* you think this patch would work but we are being bitten by our multiple points of config mutation. A downside to not writing out exhaustively long explicit configs.

It looks like we have overwritten ['ash']['platforms'] elsewhere and so linux64-cc was never even there in this loop to delete before this patch.

r+ if you add the following to your patch:

diff --git a/mozilla/project_branches.py b/mozilla/project_branches.py
index 239f9bb..7d301ef 100644
--- a/mozilla/project_branches.py
+++ b/mozilla/project_branches.py
@@ -92,30 +92,31 @@ PROJECT_BRANCHES = {
     #},
     'ash': {
         'enable_perproduct_builds': False,
         'desktop_mozharness_repacks_enabled': True,
         'enable_nightly': True,
         'mozharness_repo_path': 'build/ash-mozharness',
         'mozharness_repo': 'https://hg.mozilla.org/build/ash-mozharness',
         'mozharness_tag': 'default',
         'lock_platforms': True,
         'talos_suites': {
             'xperf': 1,
         },
         'platforms': {
             'linux': {},
             'linux64': {},
+            'linux64-cc': {},
             'win32': {},
             'macosx64': {},
             'linux-debug': {},
             'linux64-debug': {},
             'linux64-br-haz': {},
             'linux64-sh-haz': {},
             'macosx64-debug': {},
             'win32-debug': {},
         },
         'mobile_platforms': {
             'android': {
                 'slave_platforms': ['panda_android', 'ubuntu64_vm_large'],
             },
             'android-x86': {
                 'enable_opt_unittests': True,

	Andrew Halberstadt [:ahal] Assignee
	Comment 3 • 10 years ago

With Armen's recent landing of mozharness-try, there's no longer any reason to have this running on Ash.