1145176 - Use consistent runner task numerical prefix

Status: RESOLVED INCOMPLETE

(Release Engineering :: General, defect)

Description

[Rail Aliiev [:rail]]

Found this:

2015-03-19 08:00:39,158 - DEBUG - tasks: ['0-checkout_tools', '00-instance_metadata', '1-cleanslate', '1-cleanup', '1-mig_agent', '1-purge_builds', '3-config_mockbuild', '3-populate_shared_repos', '3-update_shared_repos', '4-buildbot.py', '99-post_flight']


Sounds like 00-instance_metadata is supposed to run first, but it wasn't.

We can use moar prefix digits, let's say 2 like initscripts do?

Comment 1

[Morgan Phillips [:mrrrgn]]

Yes, I agree, this is an issue. I'll make the ordering algorithm prefer digits.

Comment 2

[Morgan Phillips [:mrrrgn]]

Attachment: https://github.com/mozilla/build-runner/pull/26/files

This will make our tasks sort the way people expect. It also handles comparisons like: 0-task > 00-task (it considers the task with extra zeros to be a "lower" ranking).

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 8580804 [details]
https://github.com/mozilla/build-runner/pull/26/files

Comments are in the PR

Comment 4

[Rail Aliiev [:rail]]

(In reply to Rail Aliiev [:rail] from comment #0)
> Sounds like 00-instance_metadata is supposed to run first, but it wasn't.

Just to clarify this statement. 
`ls` lists 00-instance_metadata first, this is why I said "supposed to".