Paygen Broken, check_update_payload does not recognize minor version 6 |
||||||
Issue descriptionPaygen appears broken on release builders "Error: Unsupported minor version: 6" https://luci-logdog.appspot.com/logs/chromeos/buildbucket/cr-buildbucket.appspot.com/8924725587176919152/+/steps/PaygenBuildCanary/0/stdout M73 dev release was abandoned.
,
Jan 10
Its the recent uprev. I need to add code to update_payload. Please don't revert crrev.com/c/1399303
,
Jan 10
,
Jan 10
Sent crrev.com/c/1406049 for review. This should fix it.
,
Jan 10
+norvez@ just an FYI @sheriffs: In case this became an emergency and we needed to get green canary ASAP, feel free to chump the aforementioned CL after it passes the precq. The reason being is that none of the stuff in update_payload goes through CQ testing, etc.
,
Jan 11
The following revision refers to this bug: https://chromium.googlesource.com/aosp/platform/system/update_engine/+/ef1af2773e238de982d98490d8542668f6499ac5 commit ef1af2773e238de982d98490d8542668f6499ac5 Author: Amin Hassani <ahassani@chromium.org> Date: Thu Jan 10 22:13:46 2019 update_payload: uprev to minor version 6 BUG= chromium:920699 TEST=unittest TEST=check_update_payload --check # with a delta payload Change-Id: I40bc1e535ed241b74c6d889757a8dcf046bac0de [modify] https://crrev.com/ef1af2773e238de982d98490d8542668f6499ac5/scripts/update_payload/applier.py [modify] https://crrev.com/ef1af2773e238de982d98490d8542668f6499ac5/scripts/update_payload/checker.py
,
Jan 11
,
Jan 14
Issue 921639 has been merged into this issue. |
||||||
►
Sign in to add a comment |
||||||
Comment 1 by bleung@google.com
, Jan 10