Closed Bug 1158845 Opened 9 years ago Closed 9 years ago

disable 10.10 talos on mozilla-release

Categories

(Testing :: Talos, defect)

defect
Not set
normal

Tracking

(firefox40 affected)

RESOLVED FIXED
Tracking Status
firefox40 --- affected

People

(Reporter: jmaher, Assigned: jmaher)

Details

Attachments

(1 file)

we don't track talos on mozilla-release- 10.10 could save some cycles!
and the braindump list builder differences script:
2015-04-27 11:56:18,009 - Loaded 11 masters in 27.64s
\n
Builders removed
- Rev5 MacOSX Yosemite 10.10 mozilla-release talos chromez
- Rev5 MacOSX Yosemite 10.10 mozilla-release talos dromaeojs
- Rev5 MacOSX Yosemite 10.10 mozilla-release talos g1
- Rev5 MacOSX Yosemite 10.10 mozilla-release talos other_nol64
- Rev5 MacOSX Yosemite 10.10 mozilla-release talos svgr
- Rev5 MacOSX Yosemite 10.10 mozilla-release talos tp5o

Here's the file that contains your changes: /home/jmaher/mozilla/braindump/community/differences.txt
jmaher@jmaher-ThinkPad-X230:~/mozilla/braindump/community$
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8598057 - Flags: review?(bugspam.Callek)
Comment on attachment 8598057 [details] [diff] [review]
disable 10.10 talos tests on mozilla-release

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

::: mozilla-tests/config.py
@@ +2252,5 @@
>          continue
>      BRANCHES[branch]['platforms']['macosx64']['mountainlion']['opt_unittest_suites'] = []
>      BRANCHES[branch]['platforms']['macosx64']['mountainlion']['debug_unittest_suites'] = []
>      #disable talos on branches that have 10.10 enabled excluding b2g-inbound 
>      #which idn't have talos tests before

While here can you correct typo on comment (also #<space>text to account for pep8?)

@@ +2253,5 @@
>      BRANCHES[branch]['platforms']['macosx64']['mountainlion']['opt_unittest_suites'] = []
>      BRANCHES[branch]['platforms']['macosx64']['mountainlion']['debug_unittest_suites'] = []
>      #disable talos on branches that have 10.10 enabled excluding b2g-inbound 
>      #which idn't have talos tests before
> +    if branch in ['b2g-inbound', 'mozilla-release']:

I'd love a comment along the lines of "We don't run talos in -release" per our IRC convo
Attachment #8598057 - Flags: review?(bugspam.Callek) → review+
Thanks Callek!  Fixed up some comments and committed:
https://hg.mozilla.org/build/buildbot-configs/rev/a11a4fce0b5a
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: