mirror of
https://github.com/ansible-collections/community.docker.git
synced 2025-12-16 11:58:43 +00:00
Fix crashes due to wrong names. (#1161)
This commit is contained in:
parent
892e9d9cbd
commit
33c8a49191
4
changelogs/fragments/1161-fix.yml
Normal file
4
changelogs/fragments/1161-fix.yml
Normal file
@ -0,0 +1,4 @@
|
|||||||
|
bugfixes:
|
||||||
|
- "docker connection plugin - fix crash instead of warning if Docker version does not support ``remote_user`` (https://github.com/ansible-collections/community.docker/pull/1161)."
|
||||||
|
- "docker_compose_v2_exec - fix crash instead of reporting error if ``detach=true`` and ``stdin`` is provided (https://github.com/ansible-collections/community.docker/pull/1161)."
|
||||||
|
- "docker_compose_v2_run - fix crash instead of reporting error if ``detach=true`` and ``stdin`` is provided (https://github.com/ansible-collections/community.docker/pull/1161)."
|
||||||
@ -353,7 +353,7 @@ class Connection(ConnectionBase):
|
|||||||
actual_user = self._get_docker_remote_user()
|
actual_user = self._get_docker_remote_user()
|
||||||
if actual_user != self.get_option("remote_user"):
|
if actual_user != self.get_option("remote_user"):
|
||||||
display.warning(
|
display.warning(
|
||||||
f'docker {self.docker_version} does not support remote_user, using container default: {self.actual_user or "?"}'
|
f'docker {self.docker_version} does not support remote_user, using container default: {actual_user or "?"}'
|
||||||
)
|
)
|
||||||
return actual_user
|
return actual_user
|
||||||
elif self._display.verbosity > 2:
|
elif self._display.verbosity > 2:
|
||||||
|
|||||||
@ -200,7 +200,7 @@ class ExecManager(BaseComposeManager):
|
|||||||
self.argv = shlex.split(parameters["command"])
|
self.argv = shlex.split(parameters["command"])
|
||||||
|
|
||||||
if self.detach and self.stdin is not None:
|
if self.detach and self.stdin is not None:
|
||||||
self.mail("If detach=true, stdin cannot be provided.")
|
self.fail("If detach=true, stdin cannot be provided.")
|
||||||
|
|
||||||
if self.stdin is not None and parameters["stdin_add_newline"]:
|
if self.stdin is not None and parameters["stdin_add_newline"]:
|
||||||
self.stdin += "\n"
|
self.stdin += "\n"
|
||||||
|
|||||||
@ -286,7 +286,7 @@ class ExecManager(BaseComposeManager):
|
|||||||
self.argv = shlex.split(parameters["command"])
|
self.argv = shlex.split(parameters["command"])
|
||||||
|
|
||||||
if self.detach and self.stdin is not None:
|
if self.detach and self.stdin is not None:
|
||||||
self.mail("If detach=true, stdin cannot be provided.")
|
self.fail("If detach=true, stdin cannot be provided.")
|
||||||
|
|
||||||
if self.stdin is not None and parameters["stdin_add_newline"]:
|
if self.stdin is not None and parameters["stdin_add_newline"]:
|
||||||
self.stdin += "\n"
|
self.stdin += "\n"
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user