Closed Bug 1277881 Opened 8 years ago Closed 8 years ago

Remove all gecko.v1 routes

Categories

(Firefox Build System :: Task Configuration, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla52

People

(Reporter: mshal, Assigned: mshal)

References

Details

Attachments

(1 file)

I don't believe any clients actually use this, so we can stop publishing tasks to these routes. If anyone is aware of its use, please let me know.
Assignee: nobody → mshal
Comment on attachment 8793511 [details]
Bug 1277881 - Remove all gecko.v1 routes;

https://reviewboard.mozilla.org/r/80224/#review79056

::: taskcluster/taskgraph/transforms/job/mulet.py
(Diff revision 1)
>  
>      docker_worker_add_public_artifacts(config, job, taskdesc)
>      docker_worker_add_gecko_vcs_env_vars(config, job, taskdesc)
>  
> -    taskdesc.setdefault('routes', []).extend([
> -        'index.gecko.v1.{project}.latest.simulator.opt'.format(**config.params),

This one is the odd man out.. let's see if we can find out whether this is used or not..
Flags: needinfo?(lissyx+mozillians)
Comment on attachment 8793511 [details]
Bug 1277881 - Remove all gecko.v1 routes;

https://reviewboard.mozilla.org/r/80224/#review79060
Attachment #8793511 - Flags: review?(dustin) → review+
I remember we talked about that yesterday with :mshal about mozilla-download npm package (https://github.com/mozilla-b2g/mozilla-download) because it was using the old buildbot indexes.

I guess this change might invalidate some URLs we have?
Flags: needinfo?(lissyx+mozillians)
Blocks: 1304707
Ah, if it's using the buildbot routes, then this patch will not affect it (it only affects gecko.v1 routes).
No longer blocks: 1304707
Depends on: 1304707
Pushed by mshal@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/bce3390cf544
Remove all gecko.v1 routes; r=dustin
https://hg.mozilla.org/mozilla-central/rev/bce3390cf544
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
Product: TaskCluster → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: