docker_swarm_service: rename return variable to swarm_service (#53229)
* Rename return variable to swarm_service. * Add changelog. * Add that old name will stay in Ansible 2.7.x.
This commit is contained in:
parent
70f6cdb814
commit
61abbfc269
3 changed files with 13 additions and 4 deletions
|
@ -0,0 +1,5 @@
|
||||||
|
bugfixes:
|
||||||
|
- "docker_swarm_service - the return value was documented as ``ansible_swarm_service``, but the
|
||||||
|
module actually returned ``ansible_docker_service``. Documentation and code have been updated
|
||||||
|
so that the variable is now called ``swarm_service``. In Ansible 2.7.x, the old name
|
||||||
|
``ansible_docker_service`` can still be used to access the result."
|
|
@ -666,13 +666,17 @@ notes:
|
||||||
'''
|
'''
|
||||||
|
|
||||||
RETURN = '''
|
RETURN = '''
|
||||||
ansible_swarm_service:
|
swarm_service:
|
||||||
returned: always
|
returned: always
|
||||||
type: dict
|
type: dict
|
||||||
description:
|
description:
|
||||||
- Dictionary of variables representing the current state of the service.
|
- Dictionary of variables representing the current state of the service.
|
||||||
Matches the module parameters format.
|
Matches the module parameters format.
|
||||||
- Note that facts are not part of registered vars but accessible directly.
|
- Note that facts are not part of registered vars but accessible directly.
|
||||||
|
- Note that before Ansible 2.7.9, the return variable was documented as C(ansible_swarm_service),
|
||||||
|
while the module actually returned a variable called C(ansible_docker_service). The variable
|
||||||
|
was renamed to C(swarm_service) in both code and documentation for Ansible 2.7.9 and Ansible 2.8.0.
|
||||||
|
In Ansible 2.7.x, the old name C(ansible_docker_service) can still be used.
|
||||||
sample: '{
|
sample: '{
|
||||||
"args": [
|
"args": [
|
||||||
"sleep",
|
"sleep",
|
||||||
|
@ -2278,7 +2282,7 @@ def main():
|
||||||
changed=changed,
|
changed=changed,
|
||||||
rebuilt=rebuilt,
|
rebuilt=rebuilt,
|
||||||
changes=changes,
|
changes=changes,
|
||||||
ansible_docker_service=facts,
|
swarm_service=facts,
|
||||||
)
|
)
|
||||||
if client.module._diff:
|
if client.module._diff:
|
||||||
before, after = dsm.diff_tracker.get_before_after()
|
before, after = dsm.diff_tracker.get_before_after()
|
||||||
|
|
|
@ -50,7 +50,7 @@
|
||||||
- name: assert service args are correct
|
- name: assert service args are correct
|
||||||
assert:
|
assert:
|
||||||
that:
|
that:
|
||||||
- output.ansible_docker_service.args == ['sleep', '1800']
|
- output.swarm_service.args == ['sleep', '1800']
|
||||||
|
|
||||||
- name: set service mode to global
|
- name: set service mode to global
|
||||||
register: output
|
register: output
|
||||||
|
@ -88,7 +88,7 @@
|
||||||
|
|
||||||
- name: fake image key as it is not predictable
|
- name: fake image key as it is not predictable
|
||||||
set_fact:
|
set_fact:
|
||||||
ansible_docker_service_output: "{{ output.ansible_docker_service|combine({'image': 'busybox'}) }}"
|
ansible_docker_service_output: "{{ output.swarm_service|combine({'image': 'busybox'}) }}"
|
||||||
|
|
||||||
- name: assert service matches expectations
|
- name: assert service matches expectations
|
||||||
assert:
|
assert:
|
||||||
|
|
Loading…
Reference in a new issue