Closed
Bug 1245725
Opened 8 years ago
Closed 8 years ago
[funsize] Partial MAR file names should be configurable
Categories
(Release Engineering :: Release Automation: Other, defect)
Release Engineering
Release Automation: Other
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: rail, Assigned: rail)
References
Details
Attachments
(4 files)
47 bytes,
text/x-github-pull-request
|
jlund
:
review+
rail
:
checked-in+
|
Details | Review |
42 bytes,
text/x-github-pull-request
|
jlund
:
review+
rail
:
checked-in+
|
Details | Review |
42 bytes,
text/x-github-pull-request
|
jlund
:
review+
rail
:
checked-in+
|
Details | Review |
48 bytes,
text/x-github-pull-request
|
rail
:
review+
rail
:
checked-in+
|
Details | Review |
ATM they use not that predictable file names (Firefox-$buildid1-$buildid2-...). It'd be great to use file names similar to what we have now. Since funsize generates more than 1 file, we'll have to use a bit longer file names, containing locale and platform.
Assignee | ||
Comment 1•8 years ago
|
||
Attachment #8715612 -
Flags: review?(jlund)
Assignee | ||
Comment 2•8 years ago
|
||
This doesn't require immediate deployment, because release promotion uses :latest images.
Attachment #8715616 -
Flags: review?(jlund)
Updated•8 years ago
|
Attachment #8715612 -
Flags: review?(jlund) → review+
Comment 3•8 years ago
|
||
Comment on attachment 8715616 [details] [review] funsize changes looks good but one comment about template default filename differing from releasetasks passed env. should they be the same? I'll r+ if you convince me otherwise :)
Attachment #8715616 -
Flags: review?(jlund) → review-
Comment 4•8 years ago
|
||
Comment on attachment 8715616 [details] [review] funsize changes see PR
Attachment #8715616 -
Flags: review- → review+
Assignee | ||
Updated•8 years ago
|
Attachment #8715612 -
Flags: checked-in+
Assignee | ||
Updated•8 years ago
|
Attachment #8715616 -
Flags: checked-in+
Assignee | ||
Comment 5•8 years ago
|
||
Attachment #8716387 -
Flags: review?(jlund)
Comment 6•8 years ago
|
||
Comment on attachment 8716387 [details] [review] fix bad quoting this has been merged. see PR
Attachment #8716387 -
Flags: review?(jlund) → review+
Assignee | ||
Updated•8 years ago
|
Attachment #8716387 -
Flags: checked-in+
Assignee | ||
Comment 7•8 years ago
|
||
Attachment #8716942 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 8•8 years ago
|
||
Comment on attachment 8716942 [details] [review] use {{version}} r+ed in the PR
Attachment #8716942 -
Flags: review?(bugspam.Callek)
Attachment #8716942 -
Flags: review+
Attachment #8716942 -
Flags: checked-in+
Assignee | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•