-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Bug 1648785 - Let taskcluster upload APKs to Github releases #12907
Conversation
upcoming_release_notes.md
Outdated
@@ -0,0 +1,3 @@ | |||
Thanks for trying out Firefox Daylight! In {version}, we have lots of new improvements: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO for myself before landing: Tell the people in charge of releases about this file and how to update it.
TODO as a followup: Let bump task clobber this file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For latter I've added a comment in https://bugzilla.mozilla.org/show_bug.cgi?id=1635488#c9 to track it when we add that support.
if abi not in only_abis: | ||
continue | ||
apk_path = apk_metadata["name"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit for this PR: Rename name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll handle this in a followup PR. I don't remember what I did back in July so making this change in this PR adds some risk. I'd prefer to focus on getting this task up and running in production rather than fixing this kind of names.
group-by: build-type | ||
|
||
only-for-build-types: | ||
- fennec-beta |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO change build types once #12827 lands.
Codecov Report
@@ Coverage Diff @@
## master #12907 +/- ##
============================================
+ Coverage 29.22% 29.23% +0.01%
Complexity 1161 1161
============================================
Files 454 454
Lines 18541 18540 -1
Branches 2393 2393
============================================
+ Hits 5419 5421 +2
+ Misses 12745 12742 -3
Partials 377 377
Continue to review full report at Codecov.
|
upcoming_release_notes.md
Outdated
@@ -0,0 +1,3 @@ | |||
Thanks for trying out Firefox Daylight! In {version}, we have lots of new improvements: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For latter I've added a comment in https://bugzilla.mozilla.org/show_bug.cgi?id=1635488#c9 to track it when we add that support.
taskcluster/ci/build/kind.yml
Outdated
@@ -20,6 +20,7 @@ job-defaults: | |||
# `name` and `path`. | |||
apk-artifact-template: | |||
type: file | |||
github-name: "firefox_preview_v{version}_{abi}.apk" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: did we want to update the name to s/preview/daylight
?
fennec-beta: true | ||
fennec-release: false | ||
default: true | ||
release-name: Firefox Preview {version} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: did we want to update the name to s/Sreview/Daylight
?
|
||
def _get_release_notes(config): | ||
release_notes_path = os.path.join(config.graph_config.vcs_root, "upcoming_release_notes.md") | ||
with codecs.open(release_notes_path, encoding="utf-8") as f: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
|
||
yield task | ||
|
||
def _build_artifact_map(task): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 for the consistency with beetmover schemas, even though they are not related in practice, but only conceptually.
374aaa4
to
627a458
Compare
@JohanLorenzo is this still something that is able to be landed? I'm not sure what happened here, but would love to have it, especially if the work is done! looks like it got approved, but then CI broke? |
can't wait! however why is CI still running after 9 hours :( |
Thanks for your interest in this @liuche 😃 We have an explanation about the CI timeouts. We need some new workers to be online. Releng depends on CloudOps to do so, and we face that problem https://bugzilla.mozilla.org/show_bug.cgi?id=1658298#c2. So, we can't merge this PR just yet. I'll keep you tuned. |
627a458
to
e5f51c6
Compare
d81f293
to
4d954e4
Compare
It now works and it's been tested end-to-end: https://firefox-ci-tc.services.mozilla.com/tasks/bGKyHzrGR-y0mOBzVx79pQ/runs/4/logs/https%3A%2F%2Ffirefox-ci-tc.services.mozilla.com%2Fapi%2Fqueue%2Fv1%2Ftask%2FbGKyHzrGR-y0mOBzVx79pQ%2Fruns%2F4%2Fartifacts%2Fpublic%2Flogs%2Flive_backing.log#L58 Staging will be a dummy task just like this one https://firefox-ci-tc.services.mozilla.com/tasks/Cku_lREtSVKlNF2mYcBVPQ/runs/1/logs/https%3A%2F%2Ffirefox-ci-tc.services.mozilla.com%2Fapi%2Fqueue%2Fv1%2Ftask%2FCku_lREtSVKlNF2mYcBVPQ%2Fruns%2F1%2Fartifacts%2Fpublic%2Flogs%2Flive_backing.log This is ready to land and to be tested with the actual production environment. |
Pull Request checklist
After merge
To download an APK when reviewing a PR: