From a7c7adce2f54a59ebebd8bb3032ddf6f130e8b3c Mon Sep 17 00:00:00 2001 From: Felix Fontein Date: Sat, 24 Feb 2024 20:45:13 +0100 Subject: [PATCH] Add docker_container_exec note on env variables; remove superfluous notes (#806) * Remove unnecessary notes. * Add note for evaluating environment variables. --- plugins/modules/docker_container_exec.py | 3 ++- plugins/modules/docker_image_load.py | 3 --- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/plugins/modules/docker_container_exec.py b/plugins/modules/docker_container_exec.py index 6a28cd3e..0d92dad9 100644 --- a/plugins/modules/docker_container_exec.py +++ b/plugins/modules/docker_container_exec.py @@ -94,9 +94,10 @@ options: version_added: 2.1.0 notes: - - Does not support C(check_mode). - Does B(not work with TCP TLS sockets) when using O(stdin). This is caused by the inability to send C(close_notify) without closing the connection with Python's C(SSLSocket)s. See U(https://github.com/ansible-collections/community.docker/issues/605) for more information. + - If you need to evaluate environment variables of the container in O(command) or O(argv), you need to pass the command through a shell, + like O(command=/bin/sh -c "echo $ENV_VARIABLE"). author: - "Felix Fontein (@felixfontein)" diff --git a/plugins/modules/docker_image_load.py b/plugins/modules/docker_image_load.py index 6d264350..ec628d1b 100644 --- a/plugins/modules/docker_image_load.py +++ b/plugins/modules/docker_image_load.py @@ -39,9 +39,6 @@ options: type: path required: true -notes: - - Does not support C(check_mode). - requirements: - "Docker API >= 1.25"