Only run endurance tests on nodes with the 'endurance' label. Fixes issue #173 #200
Conversation
| @@ -65,6 +65,9 @@ def main(): | |||
| continue | |||
|
|
|||
| for locale in locales: | |||
| if script in ['endurance']: | |||
| node_labels.append(script) | |||
|
|
|||
There was a problem hiding this comment.
I don't think we want to do that for each locale right? Otherwise we would end-up with [windows, xp, 32bit, endurance, endurance...].
There was a problem hiding this comment.
D'oh! Thanks for spotting that.
|
Looks fine otherwise. |
|
Updated. Thanks @whimboo |
|
Looks good to me now. Have you tested it locally? If it works fine we can merge and also push it to staging for further testing. |
|
It works locally for triggering ondemand but I haven't tested with pulse... On 4 February 2013 14:49, Henrik Skupin notifications@github.com wrote:
Dave Hunt |
|
Ok, so lets get this to staging then and test there. Can you cover that work? |
|
Landed in commit e3f81ee |
This pull adds script/testrun specific node allocation for Mozmill CI, initially only endurance takes advantage of this, however it could easily be expanded in the future.
I have done limited testing for this, and would propose that we look to test this thoroughly on staging before we push to production.