openstack/ansible-config_template
Code Issues Proposed changes

Python fix to the connection issue

Browse Source 
Instead of having a conditional, we should directly use python .get()
internals, that allow to have a default.

Change-Id: I9908e83e3faa2fcc22c891c0fd6caa1fd8e92cc4
Related-Bug: 1695944
This commit is contained in:
Jean-Philippe Evrard 2017-06-12 09:24:07 +00:00
parent 1c7cb99fcc
commit a556e38164
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
connection/ssh.py
View File

@ -64,9 +64,8 @@ class Connection(SSH.Connection):
def set_host_overrides(self, host, hostvars=None): def set_host_overrides(self, host, hostvars=None):
if self._container_check() or self._chroot_check(): if self._container_check() or self._chroot_check():
physical_host_addr = hostvars.get('physical_host_addr') physical_host_addr = hostvars.get('physical_host_addr',
if not physical_host_addr: self.physical_host)
physical_host_addr = self.physical_host
self.host = self._play_context.remote_addr = physical_host_addr self.host = self._play_context.remote_addr = physical_host_addr
def exec_command(self, cmd, in_data=None, sudoable=True): def exec_command(self, cmd, in_data=None, sudoable=True):