Compare commits

...

3 commits

Author SHA1 Message Date
Sloane Hertel
e5bd61d56b
Merge f854cca17d into 2ed8bdc8e8 2023-05-31 11:28:04 -05:00
s-hertel
f854cca17d add a changelog 2023-03-03 15:02:53 -05:00
s-hertel
2e304ee260 remove deprecated internal argument from synchronize
the new_stdin argument is no longer used to instantiate a connection plugin
2023-03-03 13:58:57 -05:00
2 changed files with 7 additions and 4 deletions

View file

@ -0,0 +1,2 @@
trivial:
- synchronize - instantiate the connection plugin without the ``new_stdin`` argument, which is deprecated in ansible-core 2.15 (https://github.com/ansible-collections/ansible.posix/pull/421).

View file

@ -284,9 +284,6 @@ class ActionModule(ActionBase):
# told (via delegate_to) that a different host is the source of the
# rsync
if not use_delegate and remote_transport:
# Create a connection to localhost to run rsync on
new_stdin = self._connection._new_stdin
# Unlike port, there can be only one shell
localhost_shell = None
for host in C.LOCALHOST:
@ -315,7 +312,11 @@ class ActionModule(ActionBase):
localhost_executable = C.DEFAULT_EXECUTABLE
self._play_context.executable = localhost_executable
new_connection = connection_loader.get('local', self._play_context, new_stdin)
try:
new_connection = connection_loader.get('local', self._play_context)
except TypeError:
# Needed for ansible-core < 2.15
new_connection = connection_loader.get('local', self._play_context, self._connection._new_stdin)
self._connection = new_connection
# Override _remote_is_local as an instance attribute specifically for the synchronize use case
# ensuring we set local tmpdir correctly