Closed
Bug 1380605
Opened 7 years ago
Closed 7 years ago
Provide OpenH264 v1.7.1 build
Categories
(Release Engineering :: Release Requests, enhancement, P1)
Release Engineering
Release Requests
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: nthomas, Assigned: nthomas)
References
Details
Attachments
(7 files)
(deleted),
text/x-github-pull-request
|
bhearsum
:
review+
|
Details |
(deleted),
text/x-github-pull-request
|
bhearsum
:
review+
|
Details |
(deleted),
text/plain
|
Details | |
(deleted),
application/zip
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
text/x-review-board-request
|
catlee
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
catlee
:
review+
|
Details |
Time for another OpenH264 release, last time was bug 1286533.
Assignee | ||
Comment 1•7 years ago
|
||
Notes from investigating
* bug 1284285 comment #1 the download domain is going to change from http://ciscobinary.openh264.org to https://c7ab4fbd1155b1257820-fefb1450afc680271ea365edabf976ea.ssl.cf1.rackcdn.com
--> TODO we'll need to adjust the Balrog whitelist
* we last did this about a year ago, bug 1286533 comment #6 has the steps Callek used
* 52.0 was the last release cycle that support 32-bit mac, only need 64-bit if this is going to ride the trains with 55/56
* we have an AArch64 for android now
* our dump_syms executables for mac/linux64/windows are from Dec 2015, and there's been a couple of syncs with upstream since then, so we should refresh them (possibly by just grabbing from an appropriate build and put them in tooltool as public artifacts)
* expecting to have to iterate to find which parts of main tooltool manifests are necessary these days, eg tooltool manifests for desktop don't include gcc/clang/visual studio/rustc
* cross-compiling on mac might be fun, for arbitrarily small values of fun
Assignee | ||
Comment 2•7 years ago
|
||
Status
- first updates of openh264 script and config at try 0619db083bef (based on attachment 8770575 [details])
- using that & openh264 rev 6c2e7008f8bbe2ff90100972f97071eb87ec37cb (tip of v1.7.1-Firefox39)
- linux32 and linux64 compile & test without error
- android builds fail with an ndk issue (I've updated to the r11c ndk to match desktop), will debug this next
Assignee | ||
Comment 3•7 years ago
|
||
Attachment #8892697 -
Flags: review?(bhearsum)
Comment 4•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla/balrog
https://github.com/mozilla/balrog/commit/2e0517ac2d0fce1c8449527eec4f15cc653ef43b
Bug 1380605 - add new OpenH264 download host to whitelists (#363). r=bhearsum
Comment 5•7 years ago
|
||
(In reply to Nick Thomas [:nthomas] from comment #3)
> Created attachment 8892697 [details]
> PR to add new download host
Do you know what the timeline for this release is? We normally avoid doing code pushes during merge & release weeks. If we want or need to ship prior to August 16th please let me know, and I'll ensure we get a push done.
Flags: needinfo?(nthomas)
Updated•7 years ago
|
Attachment #8892697 -
Flags: review?(bhearsum) → review+
Hi Nick,
Our PE Mo (mzanaty@cisco.com) just pointed out the new domain name (https://c7ab4fbd1155b1257820-fefb1450afc680271ea365edabf976ea.ssl.cf1.rackcdn.com) to host the binaries is the internal container name of the CDN, which should not be used by the users directly. There would be problem if the CDN provider changes in the future.
We are trying to fix it right now. Please hold on your release until we fix this issue. Thanks.
Flags: needinfo?(nthomas)
Flags: needinfo?(mzanaty)
Assignee | ||
Comment 9•7 years ago
|
||
Attachment #8893584 -
Flags: review?(bhearsum)
Comment 10•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla/balrog
https://github.com/mozilla/balrog/commit/04959f759188eb4fe9da30e83831e391a543b5f6
Revert "Bug 1380605 - add new OpenH264 download host to whitelists (#363). r=bhearsum" (#365)
Updated•7 years ago
|
Attachment #8893584 -
Flags: review?(bhearsum) → review+
Assignee | ||
Updated•7 years ago
|
Comment 11•7 years ago
|
||
Hi Nick,
We managed to keep the previous CDN domain name (http://ciscobinary.openh264.org/) and finished the deployment. You can go ahead with your release and send the plugin binaries to me when ready. Thanks.
Flags: needinfo?(mzanaty)
Assignee | ||
Comment 12•7 years ago
|
||
Status update - the builds are nearly ready, I'll sign them tomorrow and pass them over to Cisco.
Assignee | ||
Comment 13•7 years ago
|
||
Builds are generated and emailed to Hank at Cisco
Docs attached are here. Our code reference is try a8125131fc56cb50ff1ede4e800dd3dcdac0c8f3, tracing back to mozilla-central 933a04a91ce3bd44b230937083a835cb60637084. I'll put the patch up for review so we can get it into the tree.
Assignee | ||
Comment 14•7 years ago
|
||
Comment 15•7 years ago
|
||
Hi Nick,
All the files are uploaded into CDN now. Here are the download links, please check out.
http://ciscobinary.openh264.org/SHA512SUMS
http://ciscobinary.openh264.org/SHA512SUMS.asc
http://ciscobinary.openh264.org/openh264-android-aarch64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-android-aarch64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
http://ciscobinary.openh264.org/openh264-android-arm-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-android-arm-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
http://ciscobinary.openh264.org/openh264-android-x86-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-android-x86-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
http://ciscobinary.openh264.org/openh264-linux32-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-linux32-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
http://ciscobinary.openh264.org/openh264-linux64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-linux64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
http://ciscobinary.openh264.org/openh264-macosx64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-macosx64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
http://ciscobinary.openh264.org/openh264-win32-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-win32-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
http://ciscobinary.openh264.org/openh264-win64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip
http://ciscobinary.openh264.org/openh264-win64-6c2e7008f8bbe2ff90100972f97071eb87ec37cb.zip.asc
Flags: needinfo?(nthomas)
Assignee | ||
Comment 16•7 years ago
|
||
Thanks. I've set up our update server on the nightlytest channel to serve 1.7.1. I'm not 100% sure sure I have the build target correct for aarch64 (bug 1393312).
Note to self - once we ship to nightly we should update https://dxr.mozilla.org/mozilla-central/source/toolkit/content/gmp-sources/openh264.json
Flags: needinfo?(nthomas)
Assignee | ||
Comment 17•7 years ago
|
||
The json for the balrog release was created by hand (could do with a script for that), so this is a quick hack to verify that all the urls work, as well as the file size and hash matching.
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Assignee | ||
Comment 20•7 years ago
|
||
Comment on attachment 8902942 [details]
Bug 1380605 - modifications to build openh264 v1.7.1
Blurk, my plans to have one review request failed miserably. Just let me know if you'd rather have one roll-up patch instead.
Attachment #8902942 -
Flags: review?(catlee)
Comment 21•7 years ago
|
||
mozreview-review |
Comment on attachment 8902940 [details]
Bug 1380605 - Rebased patch from OpenH264 v1.6 (bug 1286533), p=Callek
https://reviewboard.mozilla.org/r/174678/#review182596
Attachment #8902940 -
Flags: review?(catlee) → review+
Comment 22•7 years ago
|
||
mozreview-review |
Comment on attachment 8902942 [details]
Bug 1380605 - modifications to build openh264 v1.7.1
https://reviewboard.mozilla.org/r/174680/#review182598
Attachment #8902942 -
Flags: review?(catlee) → review+
Comment 23•7 years ago
|
||
Pushed by nthomas@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/252f32919ce2
Rebased patch from OpenH264 v1.6 (bug 1286533), r=catlee
https://hg.mozilla.org/integration/mozilla-inbound/rev/25413c572c6e
modifications to build openh264 v1.7.1, r=catlee
Assignee | ||
Updated•7 years ago
|
Keywords: leave-open
Comment 24•7 years ago
|
||
bugherder |
Comment 25•7 years ago
|
||
When this bug landed, these improvements have been noticed for about 3 days.
I'm commenting here for easier perf tracking.
== Change summary for alert #9342 (as of September 08 2017 04:53 UTC) ==
Improvements:
11% remote-blank summary android-4-4-armv7-api16 opt 1,110.85 -> 993.52
For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=9342
Assignee | ||
Comment 26•7 years ago
|
||
(In reply to Ionuț Goldan [:igoldan], Performance Sheriffing from comment #25)
I think it must have been something else, the code here doesn't affect Fennec builds at all.
Assignee | ||
Comment 27•7 years ago
|
||
Tracking deployment in bug 1284285.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Comment 28•7 years ago
|
||
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
Updated•3 years ago
|
Component: Custom Release Requests → Release Requests
You need to log in
before you can comment on or make changes to this bug.
Description
•