Open Bug 1397839 Opened 3 years ago Updated 2 years ago

change docker-image task labels from build-docker-image-<imageName> to build-docker-image:<imageName>

Categories

(Firefox Build System :: Task Configuration, task)

task
Not set
normal

Tracking

(Not tracked)

People

(Reporter: jonasfj, Unassigned)

Details

This is really simple:
  build-docker-image-<imageName>   ->   build-docker-image:<imageName>

Or maybe something better, but using a colon instead of a dash would help a lot in some of these cases. (yes, I know this is simple)

Thoughts? Any technical reason not to? any non-technical reasons not to? :)
Flags: needinfo?(dustin)
All of the labels are dash-separated so far.  I'm not sure this wouldn't run afoul of something that's handling labels.  Why does the colon help?
Flags: needinfo?(dustin)
I as looking at labels in |mach try fuzzy --full| and it's really hard when things are called:
  build-qemu-image-docker-image-builder


a separator would make these labels more readable :) but I want to keep things consistent..


Anyways, this is just a nit suggestion..
I'm of two minds.  Mind 1: let's wait until stuff calms down around in-tree configuration, rather than potentially introducing breakage to an already turbulent area.  Mind 2: let's get some `:` characters in labels now to flush out any issues with colons.

I'm leaning toward Mind 2, really.

Note that changing labels has the potential to break things, too.  Labels for "the same task" are expected to be stable over the long term, to support analysis and things like SETA.  I don't think much depends on that right now, and certainly not for the image tasks, but it does mean that wholesale label rewriting may introduce some temporary breakage.
> Note that changing labels has the potential to break things, too.  Labels for "the same task" are expected to be stable over the long term, to support analysis and things like SETA.

Interesting note, thanks.


Maybe we wait a week or two, there is really no rush to do this.
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.