openstack
/
openstack-ansible-plugins
Code Issues Proposed changes

Retrieve container name and physical host via get_options 

This was done in the constructor and also via get_options,
this patch simplifies the constructor and relies on get_options
to populate these variables.

Change-Id: I3f5896d4f4a6286ad8d587a745f24a4f6dd226f0
This commit is contained in:
Jonathan Rosser 2023-10-24 14:13:17 +01:00
parent 362f98ca93
commit faf4b76ea5
1 changed files with 3 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
plugins/connection/ssh.py
View File

@ -339,22 +339,15 @@ class Connection(SSH.Connection):
super(Connection, self).__init__(*args, **kwargs)
self.args = args
self.kwargs = kwargs
if hasattr(self._play_context, 'container_name'):
self.container_name = self._play_context.container_name
else:
self.container_name = None
if hasattr(self._play_context, 'physical_host'):
self.physical_host = self._play_context.physical_host
else:
self.physical_host = None
self.container_name = None
self.physical_host = None
if not hasattr(self._play_context, 'retries'):
self._play_context.retries = 3
# Store the container pid for multi-use
self.container_pid = None
self.is_container = self._container_check()
self.is_container = None
def set_options(self, task_keys=None, var_options=None, direct=None):