Closed
Bug 1453275
Opened 6 years ago
Closed 6 years ago
taskgraph: Make treescript tasks support esr60
Categories
(Release Engineering :: Release Automation: Other, enhancement)
Release Engineering
Release Automation: Other
Tracking
(firefox-esr60 fixed, firefox61 fixed)
RESOLVED
FIXED
People
(Reporter: jlorenzo, Assigned: jlorenzo)
References
Details
Attachments
(1 file)
Generating a taskgraph with esr60 configs[1] leads to some discrepancies. The things I've noticed are: * release-version-bump-firefox * Bad version in tags. This might be caused bug This might be caused by bug 1453261 comment 1 * worker type is fine. There are probably other issues I didn't see. [1] https://bugzilla.mozilla.org/show_bug.cgi?id=1434889#c4
Assignee | ||
Comment 1•6 years ago
|
||
Hello Callek! 60.0esr goes to build on April 30th. Would you feel comfortable looking into this bug by then?
Flags: needinfo?(bugspam.Callek)
Comment 2•6 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] - On PTO - Back on April 16th from comment #0) > Generating a taskgraph with esr60 configs[1] leads to some discrepancies. > The things I've noticed are: > > * release-version-bump-firefox > * Bad version in tags. This might be caused bug This might be caused by > bug 1453261 comment 1 > * worker type is fine. > > There are probably other issues I didn't see. > > [1] https://bugzilla.mozilla.org/show_bug.cgi?id=1434889#c4 I *think* we want version to be 60.0 but version_display to be 60.0esr I'm not 100% sure here... Pretty sure the main version file needs to be 60.0/60.1 etc but we want version_display to read esr due to https://bugzilla.mozilla.org/show_bug.cgi?id=1432737
Flags: needinfo?(bugspam.Callek)
See Also: → 1432737
Comment 3•6 years ago
|
||
So looking closer at this, I *think* we want to use version_display for tags at https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/transforms/task.py#1135 This certainly relates to a need to set 'esr' to the version_display value ala Bug 1432737 (at initial uplift time). Ship it, as today, would continue to just set 'version': '60.1' and so on. We then also need to do https://github.com/mozilla-releng/treescript/issues/16 otherwise treescript will fail to update the version_display. n-i to Sylvestre to be aware of it.
Flags: needinfo?(sledru)
Comment 4•6 years ago
|
||
Looking even closer we don't need any .py taskgraph changes for this, due to this call stack: https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/util/scriptworker.py#435 https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/parameters.py#70 https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/parameters.py#36 https://dxr.mozilla.org/mozilla-central/source/taskcluster/taskgraph/__init__.py#12
Comment 5•6 years ago
|
||
Sounds right, do you think you could finish that in time for the 60.0esr release? merci!
Flags: needinfo?(sledru) → needinfo?(bugspam.Callek)
Assignee | ||
Comment 6•6 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] from comment #0) > * release-version-bump-firefox > * Bad version in tags. This might be caused bug This might be caused by > bug 1453261 comment 1 With the correct data discussed in bug 1453261 comment 4, the tags in payload now look sane "FIREFOX_60_0esr_BUILD1", "FIREFOX_60_0esr_RELEASE". Please note I didn't have to update version_display.txt. For more information, please refer to the first paragraph of bug 1453261 comment 5. Callek pointed the file list was also wrong. Here's a fix below.
Comment hidden (mozreview-request) |
Comment 8•6 years ago
|
||
mozreview-review |
Comment on attachment 8968181 [details] Bug 1453275 - Fix file list to bump in esr60 https://reviewboard.mozilla.org/r/236852/#review242640 r+ but since this code rides trains and won't be used on esr52 anyway, might as well just s/esr52/esr60/ in here?
Attachment #8968181 -
Flags: review?(bugspam.Callek) → review+
Pushed by Callek@gmail.com: https://hg.mozilla.org/integration/autoland/rev/b3a530c29fbe Fix file list to bump in esr60 r=Callek
Updated•6 years ago
|
Flags: needinfo?(bugspam.Callek)
Comment 10•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b3a530c29fbe
Updated•6 years ago
|
Assignee: nobody → jlorenzo
Comment 11•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-esr60/rev/f9b431913a04
status-firefox-esr60:
--- → fixed
Comment 12•6 years ago
|
||
Pushed by mozilla@hocat.ca: https://hg.mozilla.org/integration/mozilla-inbound/rev/f05c85e615c8 Fix file list to bump in esr60 r=Callek
Comment 13•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f05c85e615c8
You need to log in
before you can comment on or make changes to this bug.
Description
•