Moblab setup page could not save |
||||||||||||
Issue description
I just tried the top of the tree. The moblab steup page save failed with the following error:
ValueError: too many values to unpack
Traceback (most recent call last):
File "/usr/local/autotest/frontend/afe/json_rpc/serviceHandler.py", line 114, in dispatchRequest
results['result'] = self.invokeServiceEndpoint(meth, args)
File "/usr/local/autotest/frontend/afe/json_rpc/serviceHandler.py", line 154, in invokeServiceEndpoint
return meth(*args)
File "/usr/local/autotest/frontend/afe/rpc_handler.py", line 125, in new_fn
return f(*args, **keyword_args)
File "/usr/local/autotest/frontend/afe/site_rpc_interface.py", line 303, in verify
return func(*args, **kwargs)
File "/usr/local/autotest/frontend/afe/site_rpc_interface.py", line 405, in update_config_handler
for key, value in config_value_list:
ValueError: too many values to unpack
To reproduce, just modify any value and try to save.
,
May 14 2016
,
May 16 2016
,
May 16 2016
Please specify OSs affected by this bug. Thank you.
,
May 16 2016
,
May 16 2016
Only apply to chromeos on moblab device (guado-moblab)
,
May 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e5653cf6ce7fa70458eb2e5ddf4bd784a0240983 commit e5653cf6ce7fa70458eb2e5ddf4bd784a0240983 Author: Michael Tang <ntang@google.com> Date: Sat May 14 04:29:00 2016 Fixed the moblab setup save. BUG= chromium:611939 TEST=manual tested Change-Id: If0f8d93463a1968f2d40e77d1990c61191c3bb43 Reviewed-on: https://chromium-review.googlesource.com/344638 Commit-Ready: Michael Tang <ntang@chromium.org> Tested-by: Michael Tang <ntang@chromium.org> Reviewed-by: Stephen Lin <stephenlin@google.com> Reviewed-by: Michael Tang <ntang@chromium.org> [modify] https://crrev.com/e5653cf6ce7fa70458eb2e5ddf4bd784a0240983/frontend/client/src/autotest/moblab/ConfigSettingsView.java
,
May 16 2016
,
May 16 2016
,
May 16 2016
,
May 17 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/1745de6a1789fbca8851a6344a5209a49eea0c04 commit 1745de6a1789fbca8851a6344a5209a49eea0c04 Author: Michael Tang <ntang@google.com> Date: Sat May 14 04:29:00 2016 Fixed the moblab setup save. BUG= chromium:611939 TEST=manual tested Change-Id: If0f8d93463a1968f2d40e77d1990c61191c3bb43 Reviewed-on: https://chromium-review.googlesource.com/344638 Commit-Ready: Michael Tang <ntang@chromium.org> Tested-by: Michael Tang <ntang@chromium.org> Reviewed-by: Stephen Lin <stephenlin@google.com> Reviewed-by: Michael Tang <ntang@chromium.org> (cherry picked from commit e5653cf6ce7fa70458eb2e5ddf4bd784a0240983) Reviewed-on: https://chromium-review.googlesource.com/344806 Reviewed-by: Krk Mohan <krk@google.com> Commit-Queue: Michael Tang <ntang@chromium.org> Trybot-Ready: Michael Tang <ntang@chromium.org> [modify] https://crrev.com/1745de6a1789fbca8851a6344a5209a49eea0c04/frontend/client/src/autotest/moblab/ConfigSettingsView.java
,
May 20 2016
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
,
May 20 2016
,
Aug 29 2016
,
Aug 31 2016
|
||||||||||||
►
Sign in to add a comment |
||||||||||||
Comment 1 by ntang@google.com
, May 14 2016