From ea3ac5f195cbd3fb352a914d46a374571973d93a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Alexandre=20D=C3=ADaz?= Date: Sun, 2 Feb 2025 17:27:28 +0100 Subject: [PATCH] fix: docker_compose_v2_run: don't need sanitize labels (#1034) --- changelogs/fragments/1034-do-not-sanitize-labels.yaml | 2 ++ plugins/modules/docker_compose_v2_run.py | 3 --- 2 files changed, 2 insertions(+), 3 deletions(-) create mode 100644 changelogs/fragments/1034-do-not-sanitize-labels.yaml diff --git a/changelogs/fragments/1034-do-not-sanitize-labels.yaml b/changelogs/fragments/1034-do-not-sanitize-labels.yaml new file mode 100644 index 00000000..c723afe3 --- /dev/null +++ b/changelogs/fragments/1034-do-not-sanitize-labels.yaml @@ -0,0 +1,2 @@ +bugfixes: + - docker_compose_v2_run - the module has a conflict between the type of parameter it expects and the one it tries to sanitize. Fix removes the label sanitization step because they are already validated by the parameter definition (https://github.com/ansible-collections/community.docker/pull/1034). diff --git a/plugins/modules/docker_compose_v2_run.py b/plugins/modules/docker_compose_v2_run.py index 760f6e3c..3d07bfb2 100644 --- a/plugins/modules/docker_compose_v2_run.py +++ b/plugins/modules/docker_compose_v2_run.py @@ -253,8 +253,6 @@ from ansible_collections.community.docker.plugins.module_utils.compose_v2 import common_compose_argspec_ex, ) -from ansible_collections.community.docker.plugins.module_utils.util import sanitize_labels - class ExecManager(BaseComposeManager): def __init__(self, client): @@ -424,7 +422,6 @@ def main(): needs_api_version=False, **argspec_ex ) - sanitize_labels(client.module.params['labels'], 'labels', client) try: manager = ExecManager(client)