[autotest] client: use json for serialization instead of cPickle [chromiumos/third_party/autotest : release-R78-12499.B]

5 views
Skip to first unread message

Commit Bot (Gerrit)

unread,
Apr 8, 2020, 11:30:19 AM4/8/20
to Mike Frysinger, Keith Haddow

Commit Bot submitted this change.

View Change

Approvals: Keith Haddow: Looks good to me, approved; Commit; Verified
[autotest] client: use json for serialization instead of cPickle

Since cPickle only exists under Python 2, rewrite this code to use
the json module instead. The serialization logic is simple enough
(just simple lists/dicts of strings) that we don't need the more
powerful pickle logic here.

BUG=chromium:990593
TEST=CQ passes

Change-Id: I0fc65170ea77af08c341cf29c0254c8f93a7f8b6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/1925417
Reviewed-by: Allen Li <aya...@chromium.org>
Tested-by: Mike Frysinger <vap...@chromium.org>
Commit-Queue: Mike Frysinger <vap...@chromium.org>
(cherry picked from commit f86b2fd88f43987a7c8b33d350698b17c475e9b2)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/autotest/+/2142167
Reviewed-by: Keith Haddow <had...@chromium.org>
Commit-Queue: Keith Haddow <had...@chromium.org>
Tested-by: Keith Haddow <had...@chromium.org>
---
M client/common_lib/hosts/base_classes.py
1 file changed, 11 insertions(+), 11 deletions(-)


To view, visit change 2142167. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: chromiumos/third_party/autotest
Gerrit-Branch: release-R78-12499.B
Gerrit-Change-Id: I0fc65170ea77af08c341cf29c0254c8f93a7f8b6
Gerrit-Change-Number: 2142167
Gerrit-PatchSet: 2
Gerrit-Owner: Keith Haddow <had...@chromium.org>
Gerrit-Reviewer: Commit Bot <commi...@chromium.org>
Gerrit-Reviewer: Keith Haddow <had...@chromium.org>
Gerrit-MessageType: merged
Reply all
Reply to author
Forward
0 new messages