Closed
Bug 1457010
Opened 6 years ago
Closed 6 years ago
partner followup - gpg signing
Categories
(Release Engineering :: Release Automation: Other, enhancement)
Release Engineering
Release Automation: Other
Tracking
(firefox61 fixed)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: nthomas, Assigned: nthomas)
References
Details
Attachments
(1 file)
We gpg sign "installers" on all platforms in the buildbot partner repack infra, and want to maintain that in taskcluster.
Comment hidden (mozreview-request) |
Assignee | ||
Comment 2•6 years ago
|
||
There's a staging release at https://tools.taskcluster.net/groups/Y-bKtkRhTNq1RxNxl85wgQ.
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8971078 [details] Bug 1457010 - gpg sign partner repacks, https://reviewboard.mozilla.org/r/239850/#review245556 Good job! I'm not 100% sure about the transform, but our 61.0b9 build3 has gpg sigs, so lgtm! ::: taskcluster/taskgraph/transforms/repackage_signing_partner.py:70 (Diff revision 1) > build_platform=attributes.get('build_platform'), > build_type=attributes.get('build_type') > ) > ) > > + if 'linux' in dep_job.label: `if 'linux' in dep_job.attributes['build_platform']:` is probably preferred, as long as that works. ::: taskcluster/taskgraph/transforms/repackage_signing_partner.py:73 (Diff revision 1) > ) > > + if 'linux' in dep_job.label: > + # we want the repack job, via the dependencies for the the chunking-dummy dep_job > + for dep in dep_job.dependencies: > + if dep.startswith('release-partner-repack'): Is `dep` a dict or a string here? I'm never really sure.
Attachment #8971078 -
Flags: review?(aki) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 6•6 years ago
|
||
mozreview-review-reply |
Comment on attachment 8971078 [details] Bug 1457010 - gpg sign partner repacks, https://reviewboard.mozilla.org/r/239850/#review245556 > `if 'linux' in dep_job.attributes['build_platform']:` is probably preferred, as long as that works. Fixed. > Is `dep` a dict or a string here? I'm never really sure. In this case it's a dict of the form {a: a}, so I switched to explicitly using .values().
Pushed by nthomas@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b59f651f46ec gpg sign partner repacks, r=aki
Assignee | ||
Comment 8•6 years ago
|
||
https://hg.mozilla.org/projects/maple/rev/c42185866273efd0e9f0b0823b8da5bec3b2b661
Comment 9•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b59f651f46ec
You need to log in
before you can comment on or make changes to this bug.
Description
•