Closed
Bug 1473356
Opened 7 years ago
Closed 7 years ago
Script the creation of a WNP blob (schema 9)
Categories
(Release Engineering :: Release Automation, defect)
Release Engineering
Release Automation
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jlorenzo, Assigned: jlorenzo)
References
Details
Attachments
(2 files, 1 obsolete file)
56 bytes,
text/x-github-pull-request
|
jlorenzo
:
review+
jlorenzo
:
checked-in+
|
Details | Review |
61 bytes,
text/x-github-pull-request
|
jlund
:
review+
jlorenzo
:
checked-in+
|
Details | Review |
Manual steps are documented at: https://github.com/mozilla-releng/releasewarrior-2.0/blob/master/docs/release-promotion/desktop/howto-rc.md#wnp
Callek had scripted the old way of doing WNP. Let's update these scripts to let it do it for us.
Assignee | ||
Comment 1•7 years ago
|
||
I'm updating the script, and I'd like to make the script default to the previous release. To do so, mozilla-version should extract versions from a release name like "Firefox-60.0-build3". What do you think, Tom?
Assignee: nobody → jlorenzo
Attachment #8989941 -
Flags: review?(mozilla)
Comment 2•7 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] from comment #1)
> To do so, mozilla-version should extract versions from a
> release name like "Firefox-60.0-build3". What do you think, Tom?
I think mozilla-version should be able to extract a name from a release identifier like that *if* you ask to, but not when you just ask it to parse a version number.
Assignee | ||
Comment 3•7 years ago
|
||
This probably requires some changes after we changed the API of mozilla-version, but the algorithm should be about the same.
I'm also feeling less comfortable having this live in braindump because of the lack of automated testing. I only manually tested it :jlund, do you think this can live in releasewarrior-2.0?
Flags: needinfo?(jlund)
Attachment #8990272 -
Flags: review?(bugspam.Callek)
Comment 4•7 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] from comment #3)
> I'm also feeling less comfortable having this live in braindump because of
> the lack of automated testing. I only manually tested it :jlund, do you
> think this can live in releasewarrior-2.0?
Please, yes let's use rw/scripts/. I imagine it was only in braindump initially as callek was using it as a personal tool and wanted to share it. Much like all braindump things. But now it's probably right for it to graduate :)
Flags: needinfo?(jlund)
Comment 5•7 years ago
|
||
Comment on attachment 8990272 [details] [diff] [review]
[build/braindump] Update create_wnp_blob.py to support new balrog schema
Review of attachment 8990272 [details] [diff] [review]:
-----------------------------------------------------------------
I did write the original stuff here, but I know little about the Schema 9, and I haven't yet investigated mozilla-version all that well, so I'm tossing this over to tom who's already flagged in this bug (and is releaseduty).
Attachment #8990272 -
Flags: review?(bugspam.Callek) → review?(mozilla)
Assignee | ||
Comment 6•7 years ago
|
||
Comment on attachment 8990272 [details] [diff] [review]
[build/braindump] Update create_wnp_blob.py to support new balrog schema
There was a WNP request today. I used that script to generate it. I seize this occasion to move the script to releasewarrior.
Attachment #8990272 -
Attachment is obsolete: true
Attachment #8990272 -
Flags: review?(mozilla)
Assignee | ||
Comment 7•7 years ago
|
||
Comment on attachment 8989941 [details] [review]
[mozilla-version] PR: Add support for balrog release names
Was r+'d by Tom at https://github.com/mozilla-releng/mozilla-version/pull/5#pullrequestreview-145046422
Released as part of mozilla-version 0.2.0
Attachment #8989941 -
Flags: checked-in+
Assignee | ||
Updated•7 years ago
|
Attachment #8989941 -
Flags: review?(mozilla) → review+
Assignee | ||
Comment 8•7 years ago
|
||
This PR automates the creation of the blog locally. Yesterday, Mihai, Tom, and I thought of a way of implementing this in-tree. I'll file a follow up bug for this.
In the meantime, I suggest we use this script. An early revision of it helped me publish the 61.0.2 blob.
Attachment #8999141 -
Flags: review?(mozilla)
Attachment #8999141 -
Flags: review?(jlund)
Updated•7 years ago
|
Attachment #8999141 -
Flags: review?(mozilla)
Attachment #8999141 -
Flags: review?(jlund)
Attachment #8999141 -
Flags: review+
Assignee | ||
Comment 9•7 years ago
|
||
Comment on attachment 8999141 [details] [review]
[releasewarrior-2.0] Script the creation of a WNP blob (schema 9)
https://github.com/mozilla-releng/releasewarrior-2.0/commit/6dda01c729647b5b80fa085ced87a6273307d236
Attachment #8999141 -
Flags: checked-in+
Assignee | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•3 months ago
|
Component: Release Automation: Updates → Release Automation
You need to log in
before you can comment on or make changes to this bug.
Description
•