Closed
Bug 1079924
Opened 10 years ago
Closed 10 years ago
Schedule Linux64 CodeCoverage builds on Ash
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: ahal, Assigned: ahal)
References
Details
Attachments
(1 file)
1.17 KB,
patch
|
jlund
:
review+
|
Details | Diff | Splinter Review |
I need to test some changes from bug 1059951 on ash-mozharness. Unfortunately that requires code coverage builds to be scheduled there as well..
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #8501796 -
Flags: review?(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,
Attachment #8501796 -
Flags: review?(jlund) → review+
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.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•