From a556e381641a81f727a496449c72db0837e0608e Mon Sep 17 00:00:00 2001 From: Jean-Philippe Evrard Date: Mon, 12 Jun 2017 09:24:07 +0000 Subject: [PATCH] Python fix to the connection issue Instead of having a conditional, we should directly use python .get() internals, that allow to have a default. Change-Id: I9908e83e3faa2fcc22c891c0fd6caa1fd8e92cc4 Related-Bug: 1695944 --- connection/ssh.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/connection/ssh.py b/connection/ssh.py index e0b016a..5ad0ac7 100644 --- a/connection/ssh.py +++ b/connection/ssh.py @@ -64,9 +64,8 @@ class Connection(SSH.Connection): def set_host_overrides(self, host, hostvars=None): if self._container_check() or self._chroot_check(): - physical_host_addr = hostvars.get('physical_host_addr') - if not physical_host_addr: - physical_host_addr = self.physical_host + physical_host_addr = hostvars.get('physical_host_addr', + self.physical_host) self.host = self._play_context.remote_addr = physical_host_addr def exec_command(self, cmd, in_data=None, sudoable=True):