When calling output_perf_value a list of performance values the resulting output format is wrong. |
||||||||||||||
Issue description
If you call test.output_perf_value with a list for the value the output
will look like:
"charts": {
"ch006_mode11G_wep_PSon": {
"udp_rx": {
"units": "Mbps",
"type": "scalar",
"value": [
27.93,
29.14,
28.77
],
"improvement_direction": "up"
}
}
expected results
"charts": {
"ch006_mode11G_wep_PSon": {
"udp_rx": {
"units": "Mbps",
"type": "list_of_scalar_values",
"values": [
27.93,
29.14,
28.77
],
"improvement_direction": "up"
}
}
,
Mar 16 2016
,
Mar 16 2016
,
Mar 17 2016
Your change meets the bar and is auto-approved for M50 (branch: 2661)
,
Mar 17 2016
[Automated comment] Request affecting a post-stable build (M49), manual review required.
,
Mar 17 2016
[Automated comment] Request affecting a post-stable build (M49), manual review required.
,
Mar 17 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/980828e6c05c9f5e9cc6929e66029f659192adb2 commit 980828e6c05c9f5e9cc6929e66029f659192adb2 Author: Keith Haddow <haddowk@chromium.org> Date: Wed Mar 16 00:16:03 2016 Fix issue where logging a list of performance values was incorrect. BUG= chromium:595169 TEST=hand execution of test_unittest.py Change-Id: I945714b7790df1bca06beba9b4f09ef6c725a3c5 Previous-Reviewed-on: https://chromium-review.googlesource.com/332890 (cherry picked from commit 89a9d0e88a89ba049e76aad224db4e094176a06c) Reviewed-on: https://chromium-review.googlesource.com/333162 Reviewed-by: Keith Haddow <haddowk@chromium.org> Commit-Queue: Keith Haddow <haddowk@chromium.org> Tested-by: Keith Haddow <haddowk@chromium.org> [modify] https://crrev.com/980828e6c05c9f5e9cc6929e66029f659192adb2/client/common_lib/test_unittest.py [modify] https://crrev.com/980828e6c05c9f5e9cc6929e66029f659192adb2/client/common_lib/test.py
,
Apr 1 2016
,
Apr 5 2016
,
Apr 11 2016
,
Apr 27 2016
,
Apr 29 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
,
Apr 29 2016
,
May 23 2016
Bulk verified
,
Jul 31 2016
|
||||||||||||||
►
Sign in to add a comment |
||||||||||||||
Comment 1 by bugdroid1@chromium.org
, Mar 16 2016