Remove / update "host" entry in global_config.ini. |
||||
Issue descriptionThe autotest global config contains a hardcoded "host" entry to the chromeos-server25.mtv IP address. We should remove this entry, and replace it with something that can be managed via puppet.
,
Feb 22 2018
pprabhu@ thinks the default should be "localhost", which seems reasonable.
,
Feb 22 2018
,
Feb 22 2018
After https://chrome-internal-review.googlesource.com/c/chromeos/chromeos-admin/+/575505 we do control this via shadow_config.ini on master. The only AI here is to change global_config.ini to localhost: https://chromium-review.googlesource.com/#/c/chromiumos/third_party/autotest/+/932741 That CL can't possible break anybody because before that CL, the reference is to a server that is no longer running anyway.
,
Feb 24 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/11731c926058e25950003766aeef6db0e04f18c1 commit 11731c926058e25950003766aeef6db0e04f18c1 Author: Prathmesh Prabhu <pprabhu@chromium.org> Date: Sat Feb 24 05:07:26 2018 global_config: Drop references to non-existent server. These entries are controlled via shadow_config in all realistic environments. BUG= chromium:814862 TEST=None Change-Id: I74cdc0aa2fc947557d0096372fe81fbb671139d9 Reviewed-on: https://chromium-review.googlesource.com/932741 Commit-Ready: Jeffy Chen <jeffy.chen@rock-chips.com> Tested-by: Prathmesh Prabhu <pprabhu@chromium.org> Reviewed-by: Don Garrett <dgarrett@chromium.org> [modify] https://crrev.com/11731c926058e25950003766aeef6db0e04f18c1/global_config.ini
,
Feb 26 2018
|
||||
►
Sign in to add a comment |
||||
Comment 1 by dgarr...@chromium.org
, Feb 22 2018