From 8f55e817d2e9ad6dd99426f449bb5b73df275e90 Mon Sep 17 00:00:00 2001 From: UNR Information Security <40803249+unrinfosec@users.noreply.github.com> Date: Fri, 4 Jan 2019 02:16:16 -0800 Subject: [PATCH] Correct Errors in Documentation (#50507) * Correct Errors in Documentation According to the module, "compose" is not required, but the docs said it was - removed the "required" tag. Additionally, the example for removing a stack did not include the required "name" option (this method is why "compose" is not required. +label: docsite_pr * Removed change to description Corrected issue to my "correction" on the description. * Removed required false * Removed all instances of required: false. --- lib/ansible/modules/cloud/docker/docker_stack.py | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/lib/ansible/modules/cloud/docker/docker_stack.py b/lib/ansible/modules/cloud/docker/docker_stack.py index 951291ef9a1..787fb8b4f4b 100644 --- a/lib/ansible/modules/cloud/docker/docker_stack.py +++ b/lib/ansible/modules/cloud/docker/docker_stack.py @@ -35,14 +35,12 @@ options: - present - absent compose: - required: true default: [] description: - List of compose definitions. Any element may be a string referring to the path of the compose file on the target host or the YAML contents of a compose file nested as dictionary. prune: - required: false default: false description: - If true will add the C(--prune) option to the C(docker stack deploy) command. @@ -50,21 +48,18 @@ options: current stack definition. type: bool with_registry_auth: - required: false default: false description: - If true will add the C(--with-registry-auth) option to the C(docker stack deploy) command. This will have docker send registry authentication details to Swarm agents. type: bool resolve_image: - required: false choices: ["always", "changed", "never"] description: - If set will add the C(--resolve-image) option to the C(docker stack deploy) command. This will have docker query the registry to resolve image digest and supported platforms. If not set, docker use "always" by default. absent_retries: - required: false default: 0 description: - If C(>0) and C(state==absent) the module will retry up to @@ -73,7 +68,6 @@ options: If the last try still reports the stack as not completely removed the module will fail. absent_retries_interval: - required: false default: 1 description: - Interval in seconds between C(absent_retries) @@ -119,6 +113,7 @@ EXAMPLES = ''' - name: deprovision 'stack1' docker_stack: + name: stack1 state: absent '''