[autotest] Make ssh verbosity flag as an argument of SiteHost
We are appending ssh verbosity option to site_host.GLOBAL_SSH_COMMAND_OPTIONS
in autoserv. This was a hacky workaround because make_ssh_command
didn't take a host object as an argument when the code was written.
make_ssh_command has recently been refactored as a method in SiteHost. Now we
can pass ssh verbosity option as an arg to the _initialize method of
SiteHost and remove the hacky global variable.
The verbosity option is threaded through autoserv, server_job, hosts.factory
to SiteHost as what we do for passing ssh_user/ssh_pass.
BUG=chromium:276628
TEST=Run test_that with --ssh_verbosity 1, confirmed detailed debugging
info in output; Verify/Repair/Reset jobs work; Dummy tests work.
Change-Id: Ic51a65c16e061a4d5eb940da6fdfec8c40b51d98
Reviewed-on: https://chromium-review.googlesource.com/167415
Reviewed-by: Alex Miller <milleral@chromium.org>
Tested-by: Fang Deng <fdeng@chromium.org>
Commit-Queue: Fang Deng <fdeng@chromium.org>
4 files changed