ansible/community.general
1
0
Fork 0
mirror of https://github.com/ansible-collections/community.general.git synced 2024-09-14 20:13:21 +02:00
Code Releases Activity

Fix argument_spec for modules that are using the old version (#47693)

Browse source 
* Use correct argument spec
This commit is contained in:
Chris Archibald 2018-11-05 08:56:59 -08:00 committed by John R Barker
parent 6f8ffeb6bf
commit 0cf88809ef
5 changed files with 18 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
lib/ansible/modules/storage/netapp/na_ontap_cluster_peer.py
View file

@ -15,7 +15,7 @@ author: NetApp Ansible Team (ng-ansibleteam@netapp.com)
description: description:
- Create/Delete cluster peer relations on ONTAP - Create/Delete cluster peer relations on ONTAP
extends_documentation_fragment: extends_documentation_fragment:
- netapp.ontap - netapp.na_ontap
module: na_ontap_cluster_peer module: na_ontap_cluster_peer
options: options:
state: state:
@ -57,7 +57,6 @@ version_added: "2.7"
''' '''
EXAMPLES = """ EXAMPLES = """
- name: Create cluster peer - name: Create cluster peer
na_ontap_cluster_peer: na_ontap_cluster_peer:
state: present state: present
@ -68,7 +67,6 @@ EXAMPLES = """
username: "{{ netapp_username }}" username: "{{ netapp_username }}"
password: "{{ netapp_password }}" password: "{{ netapp_password }}"
dest_hostname: "{{ dest_netapp_hostname }}" dest_hostname: "{{ dest_netapp_hostname }}"
- name: Delete cluster peer - name: Delete cluster peer
na_ontap_cluster_peer: na_ontap_cluster_peer:
state: absent state: absent
@ -99,7 +97,7 @@ class NetAppONTAPClusterPeer(object):
def __init__(self): def __init__(self):
self.argument_spec = netapp_utils.ontap_sf_host_argument_spec() self.argument_spec = netapp_utils.na_ontap_host_argument_spec()
self.argument_spec.update(dict( self.argument_spec.update(dict(
state=dict(required=False, type='str', choices=['present', 'absent'], default='present'), state=dict(required=False, type='str', choices=['present', 'absent'], default='present'),
source_intercluster_lif=dict(required=False, type='str'), source_intercluster_lif=dict(required=False, type='str'),
@ -125,14 +123,14 @@ class NetAppONTAPClusterPeer(object):
if HAS_NETAPP_LIB is False: if HAS_NETAPP_LIB is False:
self.module.fail_json(msg="the python NetApp-Lib module is required") self.module.fail_json(msg="the python NetApp-Lib module is required")
else: else:
self.server = netapp_utils.setup_ontap_zapi(module=self.module) self.server = netapp_utils.setup_na_ontap_zapi(module=self.module)
# set destination server connection # set destination server connection
self.module.params['hostname'] = self.parameters['dest_hostname'] self.module.params['hostname'] = self.parameters['dest_hostname']
if self.parameters.get('dest_username'): if self.parameters.get('dest_username'):
self.module.params['username'] = self.parameters['dest_username'] self.module.params['username'] = self.parameters['dest_username']
if self.parameters.get('dest_password'): if self.parameters.get('dest_password'):
self.module.params['password'] = self.parameters['dest_password'] self.module.params['password'] = self.parameters['dest_password']
self.dest_server = netapp_utils.setup_ontap_zapi(module=self.module) self.dest_server = netapp_utils.setup_na_ontap_zapi(module=self.module)
def cluster_peer_get_iter(self, cluster): def cluster_peer_get_iter(self, cluster):
""" """

8
lib/ansible/modules/storage/netapp/na_ontap_net_vlan.py
View file

@ -14,7 +14,7 @@ DOCUMENTATION = '''
module: na_ontap_net_vlan module: na_ontap_net_vlan
short_description: NetApp ONTAP network VLAN short_description: NetApp ONTAP network VLAN
extends_documentation_fragment: extends_documentation_fragment:
- netapp.ontap - netapp.na_ontap
version_added: '2.6' version_added: '2.6'
author: NetApp Ansible Team (ng-ansibleteam@netapp.com) author: NetApp Ansible Team (ng-ansibleteam@netapp.com)
description: description:
@ -71,7 +71,7 @@ class NetAppOntapVlan(object):
""" """
Initializes the NetAppOntapVlan function Initializes the NetAppOntapVlan function
""" """
self.argument_spec = netapp_utils.ontap_sf_host_argument_spec() self.argument_spec = netapp_utils.na_ontap_host_argument_spec()
self.argument_spec.update(dict( self.argument_spec.update(dict(
state=dict(required=False, choices=['present', 'absent'], default='present'), state=dict(required=False, choices=['present', 'absent'], default='present'),
parent_interface=dict(required=True, type='str'), parent_interface=dict(required=True, type='str'),
@ -95,7 +95,7 @@ class NetAppOntapVlan(object):
if HAS_NETAPP_LIB is False: if HAS_NETAPP_LIB is False:
self.module.fail_json(msg="the python NetApp-Lib module is required") self.module.fail_json(msg="the python NetApp-Lib module is required")
else: else:
self.server = netapp_utils.setup_ontap_zapi(module=self.module) self.server = netapp_utils.setup_na_ontap_zapi(module=self.module)
return return
def create_vlan(self): def create_vlan(self):
@ -154,7 +154,7 @@ class NetAppOntapVlan(object):
changed = False changed = False
result = None result = None
results = netapp_utils.get_cserver(self.server) results = netapp_utils.get_cserver(self.server)
cserver = netapp_utils.setup_ontap_zapi(module=self.module, vserver=results) cserver = netapp_utils.setup_na_ontap_zapi(module=self.module, vserver=results)
netapp_utils.ems_log_event("na_ontap_net_vlan", cserver) netapp_utils.ems_log_event("na_ontap_net_vlan", cserver)
existing_vlan = self.does_vlan_exist() existing_vlan = self.does_vlan_exist()
if existing_vlan: if existing_vlan:

6
lib/ansible/modules/storage/netapp/na_ontap_snapmirror.py
View file

@ -16,7 +16,7 @@ description:
- Create/Delete/Initialize SnapMirror volume/vserver relationships - Create/Delete/Initialize SnapMirror volume/vserver relationships
- Modify schedule for a SnapMirror relationship - Modify schedule for a SnapMirror relationship
extends_documentation_fragment: extends_documentation_fragment:
- netapp.ontap - netapp.na_ontap
module: na_ontap_snapmirror module: na_ontap_snapmirror
options: options:
state: state:
@ -126,7 +126,7 @@ class NetAppONTAPSnapmirror(object):
def __init__(self): def __init__(self):
self.argument_spec = netapp_utils.ontap_sf_host_argument_spec() self.argument_spec = netapp_utils.na_ontap_host_argument_spec()
self.argument_spec.update(dict( self.argument_spec.update(dict(
state=dict(required=False, type='str', choices=['present', 'absent'], default='present'), state=dict(required=False, type='str', choices=['present', 'absent'], default='present'),
source_vserver=dict(required=False, type='str'), source_vserver=dict(required=False, type='str'),
@ -161,7 +161,7 @@ class NetAppONTAPSnapmirror(object):
if HAS_NETAPP_LIB is False: if HAS_NETAPP_LIB is False:
self.module.fail_json(msg="the python NetApp-Lib module is required") self.module.fail_json(msg="the python NetApp-Lib module is required")
else: else:
self.server = netapp_utils.setup_ontap_zapi(module=self.module) self.server = netapp_utils.setup_na_ontap_zapi(module=self.module)
def snapmirror_get_iter(self): def snapmirror_get_iter(self):
""" """

6
lib/ansible/modules/storage/netapp/na_ontap_software_update.py
View file

@ -15,7 +15,7 @@ author: NetApp Ansible Team (ng-ansibleteam@netapp.com)
description: description:
- Update ONTAP software - Update ONTAP software
extends_documentation_fragment: extends_documentation_fragment:
- netapp.ontap - netapp.na_ontap
module: na_ontap_software_update module: na_ontap_software_update
options: options:
state: state:
@ -77,7 +77,7 @@ class NetAppONTAPSoftwareUpdate(object):
def __init__(self): def __init__(self):
self.argument_spec = netapp_utils.ontap_sf_host_argument_spec() self.argument_spec = netapp_utils.na_ontap_host_argument_spec()
self.argument_spec.update(dict( self.argument_spec.update(dict(
state=dict(required=False, type='str', choices=['present', 'absent'], default='present'), state=dict(required=False, type='str', choices=['present', 'absent'], default='present'),
node=dict(required=False, type='list'), node=dict(required=False, type='list'),
@ -97,7 +97,7 @@ class NetAppONTAPSoftwareUpdate(object):
if HAS_NETAPP_LIB is False: if HAS_NETAPP_LIB is False:
self.module.fail_json(msg="the python NetApp-Lib module is required") self.module.fail_json(msg="the python NetApp-Lib module is required")
else: else:
self.server = netapp_utils.setup_ontap_zapi(module=self.module) self.server = netapp_utils.setup_na_ontap_zapi(module=self.module)
def cluster_image_get_iter(self): def cluster_image_get_iter(self):
""" """

10
lib/ansible/modules/storage/netapp/na_ontap_vserver_peer.py
View file

@ -15,7 +15,7 @@ author: NetApp Ansible Team (ng-ansibleteam@netapp.com)
description: description:
- Create/Delete vserver peer - Create/Delete vserver peer
extends_documentation_fragment: extends_documentation_fragment:
- netapp.ontap - netapp.na_ontap
module: na_ontap_vserver_peer module: na_ontap_vserver_peer
options: options:
state: state:
@ -54,7 +54,6 @@ version_added: "2.7"
''' '''
EXAMPLES = """ EXAMPLES = """
- name: Source vserver peer create - name: Source vserver peer create
na_ontap_vserver_peer: na_ontap_vserver_peer:
state: present state: present
@ -66,7 +65,6 @@ EXAMPLES = """
username: "{{ netapp_username }}" username: "{{ netapp_username }}"
password: "{{ netapp_password }}" password: "{{ netapp_password }}"
dest_hostname: "{{ netapp_dest_hostname }}" dest_hostname: "{{ netapp_dest_hostname }}"
- name: vserver peer delete - name: vserver peer delete
na_ontap_vserver_peer: na_ontap_vserver_peer:
state: absent state: absent
@ -96,7 +94,7 @@ class NetAppONTAPVserverPeer(object):
def __init__(self): def __init__(self):
self.argument_spec = netapp_utils.ontap_sf_host_argument_spec() self.argument_spec = netapp_utils.na_ontap_host_argument_spec()
self.argument_spec.update(dict( self.argument_spec.update(dict(
state=dict(required=False, type='str', choices=['present', 'absent'], default='present'), state=dict(required=False, type='str', choices=['present', 'absent'], default='present'),
vserver=dict(required=True, type='str'), vserver=dict(required=True, type='str'),
@ -122,14 +120,14 @@ class NetAppONTAPVserverPeer(object):
if HAS_NETAPP_LIB is False: if HAS_NETAPP_LIB is False:
self.module.fail_json(msg="the python NetApp-Lib module is required") self.module.fail_json(msg="the python NetApp-Lib module is required")
else: else:
self.server = netapp_utils.setup_ontap_zapi(module=self.module) self.server = netapp_utils.setup_na_ontap_zapi(module=self.module)
if self.parameters.get('dest_hostname'): if self.parameters.get('dest_hostname'):
self.module.params['hostname'] = self.parameters['dest_hostname'] self.module.params['hostname'] = self.parameters['dest_hostname']
if self.parameters.get('dest_username'): if self.parameters.get('dest_username'):
self.module.params['username'] = self.parameters['dest_username'] self.module.params['username'] = self.parameters['dest_username']
if self.parameters.get('dest_password'): if self.parameters.get('dest_password'):
self.module.params['password'] = self.parameters['dest_password'] self.module.params['password'] = self.parameters['dest_password']
self.dest_server = netapp_utils.setup_ontap_zapi(module=self.module) self.dest_server = netapp_utils.setup_na_ontap_zapi(module=self.module)
def vserver_peer_get_iter(self): def vserver_peer_get_iter(self):
""" """