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

Issue 902935 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 16
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

build_image failing on pi-arc PFQ

Project Member Reported by cmtm@google.com, Nov 7

Issue description

It started failing yesterday afternoon.
This isn't related to infra, so if this is the wrong component, please change it.
 
Cc: cmtm@chromium.org
Error from build_image:

ARC Packages Cache builder. Failed to parse.
  org.chromium.arc.cachebuilder.InvalidStructureException: Could not parse prop file line: setprop
  	at org.chromium.arc.cachebuilder.Utils.readSystemProperties(Utils.java:164)
  	at org.chromium.arc.cachebuilder.CacheGenerator.process(CacheGenerator.java:778)
  	at org.chromium.arc.cachebuilder.CacheGenerator.main(CacheGenerator.java:877)
Owner: cmtm@chromium.org
Breakage caused by
ag/5411839

"setprop" shouldn't be in the PRODCUT_DEFAULT_PROPERTY_OVERRIDES.
Fix is simple.
Fix up for review:
ag/5464522
Cc: hidehiko@chromium.org uekawa@chromium.org kinaba@chromium.org
Labels: Merge-Request-71
The above CL has merged. Request merge request to M71 for the above CL (ag/5464522). The CL introduced the bug is being merged to M71.
Components: -Infra>Client>ChromeOS>CI Platform>Apps>ARC
Labels: ArcConstable
Project Member

Comment 8 by sheriffbot@chromium.org, Nov 8

Labels: -Merge-Request-71 Hotlist-Merge-Review Merge-Review-71
This bug requires manual review: M71 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: benmason@(Android), kariahda@(iOS), kbleicher@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: OS-Chrome
Labels: -Merge-Review-71 Merge-Approved-71
Approved for ChromeOS M71
Project Member

Comment 11 by sheriffbot@chromium.org, Nov 12

Cc: geo...@google.com yawano@chromium.org
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: Merge-Merged
ag/5477537 is the merge CL. Adding Merge-Merged label manually for recording.
Status: Fixed (was: Assigned)
pi-pfq is green, I assume this is fixed. Please reopen if it's not the case.

Project Member

Comment 14 by sheriffbot@chromium.org, Nov 16

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-71

Sign in to add a comment