Closed
Bug 1505514
Opened 7 years ago
Closed 7 years ago
Maven upload issues on fennec 63.0.2
Categories
(Release Engineering :: Release Automation, defect)
Release Engineering
Release Automation
Tracking
(geckoview62 wontfix, geckoview63 wontfix, firefox63 fixed, firefox64 fixed, firefox65 fixed)
People
(Reporter: Callek, Assigned: jlorenzo)
References
Details
Attachments
(1 file)
For the ship graph of fennec 63.0.2 (https://tools.taskcluster.net/groups/fzkRkMt9SpOBW3yxqZK-aw) we hit 3 beetmover task failures for maven...
This did not block the Fennec 63.0.2 release (google play pushing succeeded)
Those tasks were:
https://tools.taskcluster.net/groups/fzkRkMt9SpOBW3yxqZK-aw/tasks/Ta2RKEBXSEeaSDA-OK6heA/details
https://tools.taskcluster.net/groups/fzkRkMt9SpOBW3yxqZK-aw/tasks/c678QILXTEeLMLHsEN5jGA/details
and https://tools.taskcluster.net/groups/fzkRkMt9SpOBW3yxqZK-aw/tasks/RejAzZSPRp2gZ_hRvVkJ3g/details
From the log of one of those:
2018-11-07 18:07:59,282 - beetmoverscript.zip - INFO - Archive "/builds/scriptworker/work/cot/LwarfUeeRSud4fZYiCOWfA/public/build/target.maven.zip" contains files with legitimate sizes.
2018-11-07 18:07:59,282 - scriptworker.client - ERROR - Failed to run async_main
Traceback (most recent call last):
File "/builds/scriptworker/lib/python3.6/site-packages/scriptworker/client.py", line 203, in _handle_asyncio_loop
await async_main(context)
File "/builds/scriptworker/lib/python3.6/site-packages/beetmoverscript/script.py", line 287, in async_main
await action_map[context.action](context)
File "/builds/scriptworker/lib/python3.6/site-packages/beetmoverscript/script.py", line 172, in push_to_maven
context.config.get('zip_max_file_size_in_mb', DEFAULT_ZIP_MAX_FILE_SIZE_IN_MB)
File "/builds/scriptworker/lib/python3.6/site-packages/beetmoverscript/script.py", line 184, in _extract_and_check_maven_artifacts_to_beetmove
artifacts, expected_files, zip_max_file_size_in_mb
File "/builds/scriptworker/lib/python3.6/site-packages/beetmoverscript/zip.py", line 61, in check_and_extract_zip_archives
task_id, expected_files_per_archive, zip_max_size_in_mb
File "/builds/scriptworker/lib/python3.6/site-packages/beetmoverscript/zip.py", line 75, in _check_and_extract_zip_archives_for_given_task
archive_path, expected_files, zip_max_size_in_mb
File "/builds/scriptworker/lib/python3.6/site-packages/beetmoverscript/zip.py", line 95, in _check_extract_and_delete_zip_archive
_ensure_all_expected_files_are_present_in_archive(zip_path, relative_paths_in_archive, expected_files)
File "/builds/scriptworker/lib/python3.6/site-packages/beetmoverscript/zip.py", line 186, in _ensure_all_expected_files_are_present_in_archive
file_, zip_path, unique_expected_files
scriptworker.exceptions.TaskVerificationError: File "org/mozilla/geckoview/geckoview-release-armeabi-v7a/63.0.20181105152108/geckoview-release-armeabi-v7a-63.0.20181105152108.aar.md5" present in archive "/builds/scriptworker/work/cot/LwarfUeeRSud4fZYiCOWfA/public/build/target.maven.zip" is not expected. Expected: {'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108.pom.sha1', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108.pom.md5', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108-sources.jar.sha1', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108-javadoc.jar.md5', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108-javadoc.jar.sha1', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108-sources.jar.md5', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108.aar.sha1', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108-javadoc.jar', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108.aar', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108.pom', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108-sources.jar', 'org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108.aar.md5'}
Flags: needinfo?(mtabara)
Flags: needinfo?(jlorenzo)
Comment 1•7 years ago
|
||
At first glance, it seems like there's and extra "release" within it in the extra in the file naming.
We have "org/mozilla/geckoview/geckoview-release-armeabi-v7a/63.0.20181105152108/geckoview-release-armeabi-v7a-63.0.20181105152108.aar.md5" when we should be having "org/mozilla/geckoview/geckoview-armeabi-v7a/63.0.20181105152108/geckoview-armeabi-v7a-63.0.20181105152108.aar.md5" (trimming the both occurrence of "release").
Might be related to the changes that were happening in bug 1500897 but it's weird as those patches (mainly https://phabricator.services.mozilla.com/D9551) were never uplifted. So it's either expected to fail or there's something else going on.
Keeping the NI to investigate more.
See Also: → 1500897
| Assignee | ||
Comment 2•7 years ago
|
||
I'm sorry, I forgot to tell you over IRC, Mihai. I'm on the fix. I have something ready on my machine.
Assignee: nobody → jlorenzo
Flags: needinfo?(mtabara)
Flags: needinfo?(jlorenzo)
Comment 3•7 years ago
|
||
(In reply to Johan Lorenzo [:jlorenzo] from comment #2)
> I'm sorry, I forgot to tell you over IRC, Mihai. I'm on the fix. I have
> something ready on my machine.
No worries, thanks for working on it. Looking forward for the fix!
| Assignee | ||
Comment 4•7 years ago
|
||
| Assignee | ||
Updated•7 years ago
|
status-firefox64:
--- → affected
status-firefox65:
--- → affected
status-geckoview62:
--- → wontfix
status-geckoview63:
--- → affected
Pushed by jlorenzo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c38af115dcd9
geckoview artifacts don't specify branch when mozilla-release r=snorp
Comment 6•7 years ago
|
||
| bugherder | ||
| Comment hidden (Intermittent Failures Robot) |
| Assignee | ||
Comment 8•7 years ago
|
||
| Assignee | ||
Comment 9•7 years ago
|
||
Comment 8 was backed out[1] because it caused "gradle-dependencies"[2] to fail:
> [task 2018-11-12T09:40:03.017Z] 1:11.04 7: Task failed with an exception.
> [task 2018-11-12T09:40:03.017Z] 1:11.04 -----------
> [task 2018-11-12T09:40:03.017Z] 1:11.04 * What went wrong:
> [task 2018-11-12T09:40:03.017Z] 1:11.04 Could not resolve all files for configuration ':app:officialWithoutGeckoBinariesNoMinApiPhotonDebugAndroidTestRuntimeClasspath'.
> [task 2018-11-12T09:40:03.017Z] 1:11.04 > Could not find play-services-basement.aar (com.google.android.gms:play-services-basement:15.0.1).
> [task 2018-11-12T09:40:03.017Z] 1:11.04 Searched in the following locations:
> [task 2018-11-12T09:40:03.017Z] 1:11.04 http://localhost:8081/nexus/content/repositories/jcenter/com/google/android/gms/play-services-basement/15.0.1/play-services-basement-15.0.1.aar
> [task 2018-11-12T09:40:03.017Z] 1:11.04 * Try:
> [task 2018-11-12T09:40:03.017Z] 1:11.04 Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.
One interesting thing about this job is that it only whenever a change is made on the gradle-side. If the current changes are already cached, then no job is spawned. So I manually retriggered "gradle-dependencies" on top of the backout[3] and the problem remained. Therefore, this patch is not the root cause of the breakage. I believe bug 1476271 is. Petru, what do you think?
[1] https://hg.mozilla.org/releases/mozilla-beta/rev/4fe80a60a6a04458add149d68b2dd2f7224f65f9
[2] https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&selectedJob=211160959&revision=ec9a256c6703565eafd888030b9502f55092de38
[3] https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&revision=a4175191b29b274d59ed117567ec03df9d6b8cf5&selectedJob=211224139
Comment 10•7 years ago
|
||
My patch from bug 1476271 was for for another dependency.
Regarding Play Services Basement, it seems that that aar is missing atm from jcenter - https://jcenter.bintray.com/com/google/android/gms/play-services-basement/15.0.1/play-services-basement-15.0.1.aar
But it can be found on found on Google's Maven [1][2] and based on bug 1501351 that is where it should look first.
[1] https://dl.google.com/dl/android/maven2/com/google/android/gms/group-index.xml
[2] https://dl.google.com/dl/android/maven2/com/google/android/gms/play-services-basement/15.0.1/play-services-basement-15.0.1.aar
Flags: needinfo?(petru.lingurar)
| Assignee | ||
Comment 11•7 years ago
|
||
Thank you for the super quick and informative answer, Petru! It seems I need bug 1501351 on beta, then!
Comment 12•7 years ago
|
||
Probably. If a try build with it works, then it's an easy fix.
| Assignee | ||
Comment 13•7 years ago
|
||
| Assignee | ||
Comment 14•7 years ago
|
||
| Assignee | ||
Comment 15•7 years ago
|
||
gradle-dependencies is green on beta: https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&revision=17a9ab8d9200549682861bde9c01e27e4e22c308&selectedJob=211421531
Updated•7 years ago
|
Comment 16•7 years ago
|
||
| bugherder uplift | ||
status-firefox63:
--- → fixed
Updated•1 year ago
|
Component: Release Automation: Uploading → Release Automation
You need to log in
before you can comment on or make changes to this bug.
Description
•