Fix dependencies broken atest list output format change |
|
Issue descriptionA bunch of bash scripts that use the output `atest server list` broke when the format of the output changed. Examples found so far: - https://crbug.com/888039 , - https://crbug.com/890085. The scope of this bug is to list and fix all such examples.
,
Oct 2
Fix for #c2: https://crrev.com/c/1257570
,
Oct 10
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/3e2ab26dceb14ff2fc3b89e5c3fca08e6732d320 commit 3e2ab26dceb14ff2fc3b89e5c3fca08e6732d320 Author: Alex Zamorzaev <zamorzaev@chromium.org> Date: Wed Oct 10 04:14:13 2018 [autotest] deploy_puppet: Fix atest output parsing deploy_puppet gets a host list by parsing the output of `atest server list`. The format of the output changed and broke the current code path. BUG=chromium:891111 TEST=ran the modified line locally with various inputs Change-Id: I92d2aef14a9f75bc7b3cbf02f2f2137aa54fd2de Reviewed-on: https://chromium-review.googlesource.com/1257570 Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com> Tested-by: Alex Zamorzaev <zamorzaev@chromium.org> Reviewed-by: Richard Barnette <jrbarnette@google.com> [modify] https://crrev.com/3e2ab26dceb14ff2fc3b89e5c3fca08e6732d320/contrib/deploy_puppet.sh |
|
►
Sign in to add a comment |
|
Comment 1 by zamorzaev@chromium.org
, Oct 2