Addressed review comments, given as part of other reviews (#4904)
This commit is contained in:
parent
8886afd145
commit
7362b8d08f
4 changed files with 14 additions and 14 deletions
|
@ -180,21 +180,21 @@ updates:
|
|||
|
||||
responses:
|
||||
description: The set of responses from issuing the commands on the device
|
||||
retured: when not check_mode
|
||||
returned: when not check_mode
|
||||
type: list
|
||||
sample: ['...', '...']
|
||||
|
||||
saved:
|
||||
description: Returns whether the configuration is saved to the startup
|
||||
description: Returns whether the configuration is saved to the startup
|
||||
configuration or not.
|
||||
retured: when not check_mode
|
||||
returned: when not check_mode
|
||||
type: bool
|
||||
sample: True
|
||||
|
||||
"""
|
||||
from ansible.module_utils.netcfg import NetworkConfig, dumps
|
||||
from ansible.module_utils.network import NetworkModule
|
||||
from ansible.module_utils.dellos10 import get_config, get_sublevel_config
|
||||
from ansible.module_utils.dellos10 import get_config, get_sublevel_config
|
||||
|
||||
def get_candidate(module):
|
||||
candidate = NetworkConfig(indent=1)
|
||||
|
@ -220,11 +220,10 @@ def main():
|
|||
match=dict(default='line',
|
||||
choices=['line', 'strict', 'exact', 'none']),
|
||||
replace=dict(default='line', choices=['line', 'block']),
|
||||
|
||||
update=dict(choices=['merge', 'check'], default='merge'),
|
||||
save=dict(type='bool', default=False),
|
||||
config=dict(),
|
||||
backup =dict(type='bool', default=False)
|
||||
backup=dict(type='bool', default=False)
|
||||
)
|
||||
|
||||
mutually_exclusive = [('lines', 'src')]
|
||||
|
|
|
@ -194,7 +194,7 @@ saved:
|
|||
"""
|
||||
from ansible.module_utils.netcfg import dumps
|
||||
from ansible.module_utils.network import NetworkModule
|
||||
from ansible.module_utils.dnos6 import get_config, get_sublevel_config, Dellos6NetworkConfig
|
||||
from ansible.module_utils.dellos6 import get_config, get_sublevel_config, Dellos6NetworkConfig
|
||||
|
||||
|
||||
def get_candidate(module):
|
||||
|
|
|
@ -24,6 +24,7 @@ DOCUMENTATION = """
|
|||
---
|
||||
module: dellos9_command
|
||||
version_added: "2.2"
|
||||
author: "Dhivya P (@dhivyap)"
|
||||
short_description: Run commands on remote devices running Dell OS9
|
||||
description:
|
||||
- Sends arbitrary commands to a Dell OS9 node and returns the results
|
||||
|
@ -83,22 +84,22 @@ tasks:
|
|||
- name: run show version on remote devices
|
||||
dellos9_command:
|
||||
commands: show version
|
||||
provider "{{ cli }}"
|
||||
provider: "{{ cli }}"
|
||||
|
||||
- name: run show version and check to see if output contains OS9
|
||||
dellos9_command:
|
||||
commands: show version
|
||||
wait_for: result[0] contains OS9
|
||||
provider "{{ cli }}"
|
||||
provider: "{{ cli }}"
|
||||
|
||||
- name: run multiple commands on remote nodes
|
||||
dellos9_command:
|
||||
commands:
|
||||
- show version
|
||||
- show interfaces
|
||||
provider "{{ cli }}"
|
||||
provider: "{{ cli }}"
|
||||
|
||||
- name: run multiple commands and evalute the output
|
||||
- name: run multiple commands and evaluate the output
|
||||
dellos9_command:
|
||||
commands:
|
||||
- show version
|
||||
|
@ -106,7 +107,7 @@ tasks:
|
|||
wait_for:
|
||||
- result[0] contains OS9
|
||||
- result[1] contains Loopback
|
||||
provider "{{ cli }}"
|
||||
provider: "{{ cli }}"
|
||||
"""
|
||||
|
||||
RETURN = """
|
||||
|
|
|
@ -180,14 +180,14 @@ updates:
|
|||
|
||||
responses:
|
||||
description: The set of responses from issuing the commands on the device
|
||||
retured: when not check_mode
|
||||
returned: when not check_mode
|
||||
type: list
|
||||
sample: ['...', '...']
|
||||
|
||||
saved:
|
||||
description: Returns whether the configuration is saved to the startup
|
||||
configuration or not.
|
||||
retured: when not check_mode
|
||||
returned: when not check_mode
|
||||
type: bool
|
||||
sample: True
|
||||
|
||||
|
|
Loading…
Reference in a new issue