Closed
Bug 627182
Opened 14 years ago
Closed 13 years ago
Automate signing and publishing of XULRunner builds
Categories
(Release Engineering :: Release Automation, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: rail)
References
Details
(Whiteboard: [release][automation])
Attachments
(4 files, 5 obsolete files)
10.76 KB,
patch
|
bhearsum
:
review+
bhearsum
:
feedback+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
21.51 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
2.57 KB,
patch
|
jhford
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
1.83 KB,
patch
|
bhearsum
:
review+
rail
:
checked-in+
|
Details | Diff | Splinter Review |
We do these steps manually
https://wiki.mozilla.org/Releases/Firefox_3.6.13/BuildNotes#XULRunner_Signing
to create a XULRunner release to match each Firefox release. We should automate it too.
Reporter | ||
Comment 1•14 years ago
|
||
With this, bug 565882, and a push to mirrors builder, we would pretty much have automated XULRunner builds.
Comment 2•13 years ago
|
||
The signing portion of this is fixed by bug 509158, I think?
Updated•13 years ago
|
Blocks: hg-automation
Assignee | ||
Updated•13 years ago
|
Assignee: nobody → rail
Priority: P5 → P3
Assignee | ||
Updated•13 years ago
|
Priority: P3 → P2
Assignee | ||
Comment 3•13 years ago
|
||
* generate checksums: allow override product, ssh user name/key
* push to mirrors: allow override product, ssh user name/key; shouldn't exclude zip files for xulrunner
to be tested
Assignee | ||
Updated•13 years ago
|
Component: Release Engineering → Release Engineering: Releases
QA Contact: release → bhearsum
Assignee | ||
Comment 4•13 years ago
|
||
* Uploads files to win32 instead of unsigned/win32 (post_upload.py --signed ...)
* Generates MD5/SHA1/SHA512SUMS files automatically
* Pushes to the internal mirrors just after checksums generation
Assignee | ||
Updated•13 years ago
|
Component: Release Engineering: Releases → Release Engineering: Automation
QA Contact: bhearsum → catlee
Assignee | ||
Comment 6•13 years ago
|
||
Comments incoming
Attachment #602901 -
Attachment is obsolete: true
Attachment #605931 -
Flags: review?(bhearsum)
Assignee | ||
Comment 7•13 years ago
|
||
Comment on attachment 605931 [details] [diff] [review]
tools
Review of attachment 605931 [details] [diff] [review]:
-----------------------------------------------------------------
::: scripts/release/generate-sums.py
@@ +96,5 @@
> + if productName == 'firefox':
> + cmd = 'mkdir -v -m 2775 %s/contrib %s/contrib-localized' % \
> + (candidatesDir, candidatesDir)
> + run_remote_cmd(cmd, server=stageServer, username=stageUsername,
> + sshKey=stageSshKey)
candidates are applicable to firefox only.
::: scripts/release/push-to-mirrors.py
@@ -29,5 @@
> DEFAULT_RSYNC_EXCLUDES = ['--exclude=*tests*',
> '--exclude=*crashreporter*',
> '--exclude=*.log',
> '--exclude=*.txt',
> - '--exclude=*.zip',
Xulrunner needs *.zip files (SDK)
Assignee | ||
Comment 8•13 years ago
|
||
This enables signing on demand for Xulrunner. The patch shouldn't be landed unless we are sure that there is no 11.0 chemspill or we need to land the build system patches to m-r.
Attachment #602910 -
Attachment is obsolete: true
Attachment #605942 -
Flags: review?(bhearsum)
Comment 9•13 years ago
|
||
Comment on attachment 605931 [details] [diff] [review]
tools
Review of attachment 605931 [details] [diff] [review]:
-----------------------------------------------------------------
::: scripts/release/generate-sums.py
@@ +96,5 @@
> + if productName == 'firefox':
> + cmd = 'mkdir -v -m 2775 %s/contrib %s/contrib-localized' % \
> + (candidatesDir, candidatesDir)
> + run_remote_cmd(cmd, server=stageServer, username=stageUsername,
> + sshKey=stageSshKey)
Can we find a better way of doing this than special casing the product, please? I think it's better to pass an explicit --create-contrib option (or maybe --no-contrib) from release.py since we're already forced into special casing things there.
::: scripts/release/push-to-mirrors.py
@@ +165,5 @@
> + stageSshKey = options.ssh_key or branchConfig["stage_ssh_key"]
> + stageSshKey = path.join(os.path.expanduser("~"), ".ssh", stageSshKey)
> +
> + if productName != 'xulrunner':
> + DEFAULT_RSYNC_EXCLUDES.append('--exclude=*.zip')
Similar thing here. I'm not sure if it's better to have an --exclude-zip option or --additional-exclude=*.zip though.
Attachment #605931 -
Flags: review?(bhearsum) → review-
Updated•13 years ago
|
Attachment #605942 -
Flags: review?(bhearsum)
Assignee | ||
Comment 10•13 years ago
|
||
To be tested
Attachment #605931 -
Attachment is obsolete: true
Attachment #607148 -
Flags: feedback?(bhearsum)
Assignee | ||
Comment 11•13 years ago
|
||
Attachment #605942 -
Attachment is obsolete: true
Updated•13 years ago
|
Attachment #607148 -
Flags: feedback?(bhearsum) → feedback+
Assignee | ||
Comment 12•13 years ago
|
||
Attachment #607149 -
Attachment is obsolete: true
Attachment #608726 -
Flags: review?(bhearsum)
Assignee | ||
Comment 13•13 years ago
|
||
Comment on attachment 607148 [details] [diff] [review]
tools
worked fine in staging
Attachment #607148 -
Flags: review?(bhearsum)
Updated•13 years ago
|
Attachment #608726 -
Flags: review?(bhearsum) → review+
Updated•13 years ago
|
Attachment #607148 -
Flags: review?(bhearsum) → review+
Assignee | ||
Comment 14•13 years ago
|
||
Comment on attachment 607148 [details] [diff] [review]
tools
http://hg.mozilla.org/build/tools/rev/a540057c0257
Attachment #607148 -
Flags: checked-in+
Assignee | ||
Comment 15•13 years ago
|
||
Comment on attachment 608726 [details] [diff] [review]
builbotcustom
http://hg.mozilla.org/build/buildbotcustom/rev/19db1875ce2e
Attachment #608726 -
Flags: checked-in+
Assignee | ||
Updated•13 years ago
|
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 16•13 years ago
|
||
bustage found, thanks to jhford:
AssertionError: <buildbotcustom.scheduler.AggregatingScheduler instance at 0x13eb2320> uses unknown builder release-mozilla-1.9.2-xulrunner_checksums
patch incoming
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 17•13 years ago
|
||
Added extra checks. git diff -b (ignore blanks) looks like https://gist.github.com/2211305
Attachment #609564 -
Flags: review?(jhford)
Comment 18•13 years ago
|
||
Comment on attachment 609564 [details] [diff] [review]
extra checks
looks good to me
Attachment #609564 -
Flags: review?(jhford) → review+
Assignee | ||
Comment 19•13 years ago
|
||
Comment on attachment 609564 [details] [diff] [review]
extra checks
http://hg.mozilla.org/build/buildbotcustom/rev/1e0277f9acb8
Attachment #609564 -
Flags: checked-in+
Comment 20•13 years ago
|
||
Bulk move of bugs to Release Automation component.
Component: Release Engineering: Automation (General) → Release Engineering: Automation (Release Automation)
Flags: checked-in+
Updated•13 years ago
|
No longer blocks: hg-automation
Updated•13 years ago
|
No longer blocks: hg-automation
Comment 22•13 years ago
|
||
this was deployed during a reconfig today
Assignee | ||
Comment 23•13 years ago
|
||
Somehow, the attached hunk hasn't been applied to the final push. :/
win32 build went to unsigned in 12.0b3
Attachment #610132 -
Flags: review?(bhearsum)
Comment 24•13 years ago
|
||
Comment on attachment 610132 [details] [diff] [review]
missing post_upload param
Review of attachment 610132 [details] [diff] [review]:
-----------------------------------------------------------------
d'oh!
Attachment #610132 -
Flags: review?(bhearsum) → review+
Assignee | ||
Updated•13 years ago
|
Attachment #610132 -
Flags: checked-in+
Assignee | ||
Updated•13 years ago
|
Attachment #607148 -
Flags: checked-in+
Assignee | ||
Updated•13 years ago
|
Attachment #609564 -
Flags: checked-in+
Assignee | ||
Updated•13 years ago
|
Attachment #608726 -
Flags: checked-in+
Assignee | ||
Comment 25•13 years ago
|
||
(In reply to Rail Aliiev [:rail] from comment #23)
> Created attachment 610132 [details] [diff] [review]
> missing post_upload param
>
> Somehow, the attached hunk hasn't been applied to the final push. :/
> win32 build went to unsigned in 12.0b3
Actually it was removed by another patch http://hg.mozilla.org/build/buildbotcustom/rev/8aff61059e92#l3.563
Comment 26•13 years ago
|
||
These changes went live around 11:55AM PDT.
Assignee | ||
Updated•13 years ago
|
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•