New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 851535 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: ----



Sign in to add a comment

supersize archive (ChromeModernPublic.apk) failing on multiple builders

Project Member Reported by sheriff-...@appspot.gserviceaccount.com, Jun 11 2018

Issue description

Filed by sheriff-o-matic@appspot.gserviceaccount.com on behalf of charliea@chromium.org

supersize archive (ChromeModernPublic.apk) failing on multiple builders

Builders failed on: 
- Android Builder Perf: 
  https://ci.chromium.org/buildbot/chromium.perf/Android%20Builder%20Perf
- Android arm64 Builder Perf: 
  https://ci.chromium.org/buildbot/chromium.perf/Android%20arm64%20Builder%20Perf

Andrew, Ned said that you're probably the person that knows the most about what this is. Could you look into this failure?


 
Components: Speed>Benchmarks>Waterfall
Owner: agrieve@chromium.org
Status: Assigned (was: Available)
Will have a look!
Cc: wnwen@chromium.org
Note: Error is:
I  87416 Pak Compression Ratio: 0.330000 Actual: 0.327983 Diff: 7150
Successfully loaded maps from: ChromeModernPublic.apk.mapping
Traceback (most recent call last):
  File "/b/build/slave/Android_Builder_Perf/build/src/tools/binary_size/libsupersize/main.py", line 114, in <module>
    main()
  File "/b/build/slave/Android_Builder_Perf/build/src/tools/binary_size/libsupersize/main.py", line 110, in main
    args.func(args, parser)
  File "/b/build/slave/Android_Builder_Perf/build/src/tools/binary_size/libsupersize/archive.py", line 1360, in Run
    pak_files=args.pak_file, pak_info_file=args.pak_info_file)
  File "/b/build/slave/Android_Builder_Perf/build/src/tools/binary_size/libsupersize/archive.py", line 1100, in CreateSectionSizesAndSymbols
    _ParseDexSymbols(section_sizes, apk_path, output_directory))
  File "/b/build/slave/Android_Builder_Perf/build/src/tools/binary_size/libsupersize/archive.py", line 942, in _ParseDexSymbols
    symbols = apkanalyzer.CreateDexSymbols(apk_path, output_directory)
  File "/b/build/slave/Android_Builder_Perf/build/src/tools/binary_size/libsupersize/apkanalyzer.py", line 132, in CreateDexSymbols
    'Node size too large, check for node processing errors.')
AssertionError: Node size too large, check for node processing errors.
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 12 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e33e2f9ffd3022f07adc99dbf644abe60079a551

commit e33e2f9ffd3022f07adc99dbf644abe60079a551
Author: Andrew Grieve <agrieve@chromium.org>
Date: Tue Jun 12 17:08:19 2018

supersize: Change failing dex size assert to a log

TBR=agrieve

Bug:  851535 
Change-Id: I907235a16d390a6f0c4710089e9f160b2e9c0818
Reviewed-on: https://chromium-review.googlesource.com/1097467
Commit-Queue: agrieve <agrieve@chromium.org>
Reviewed-by: agrieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#566473}
[modify] https://crrev.com/e33e2f9ffd3022f07adc99dbf644abe60079a551/tools/binary_size/libsupersize/apkanalyzer.py

Status: Fixed (was: Assigned)
Bot should go green. Filed a follow-up  bug 852039  for a proper fix.

Sign in to add a comment