Skip to content

Commit 8d43e7d

Browse files
committed
Removed unused variables.
1 parent fe66546 commit 8d43e7d

1 file changed

Lines changed: 13 additions & 28 deletions

File tree

Infrastructure_as_Code/Ansible/Volume_Management/delete_volume_and_share.yaml

Lines changed: 13 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# Title: delete_volume_and_share.yaml
22

33
---
4-
- name: Playbook to delete a volume and a CIFS share that points to it on an FSx for ONTAP file system.
4+
- name: Playbook to delete a volume and the CIFS share that points to it.
55
hosts: localhost
66
collections:
77
- netapp.ontap
@@ -19,26 +19,12 @@
1919
when: vars[item] is undefined
2020
loop:
2121
- volume_name
22+
- volume_size
2223
- vserver
2324
- secret_name
2425
- fsxn_hostname
2526
#
2627
# Give default values to optional variables if they are not defined
27-
- name: Set security_style to ntfs if not provide.
28-
set_fact:
29-
security_style: "ntfs"
30-
when: security_style is not defined
31-
32-
- name: Set aggr to 'aggr1' if not provided.
33-
set_fact:
34-
aggr: "aggr1"
35-
when: aggr is not defined
36-
37-
- name: Set volume_type to "rw" if not provided.
38-
set_fact:
39-
volume_type: "rw"
40-
when: volume_type is not defined
41-
4228
- name: Set use_lambda to true if lambda_function_name is provided.
4329
set_fact:
4430
use_lambda: true
@@ -48,17 +34,7 @@
4834
set_fact:
4935
aws_profile: "default"
5036
when: aws_profile is not defined
51-
52-
- name: Set junction_path to "/<volume_name>" if not provided.
53-
set_fact:
54-
junction_path: "/{{ volume_name }}"
55-
when: junction_path is not defined
56-
57-
- name: Set share_name to "<volume_name>" if not provided.
58-
set_fact:
59-
share_name: "{{ volume_name }}"
60-
when: share_name is not defined
61-
37+
6238
- name: Ensure that aws_region has been provided if use_lambda is true.
6339
fail:
6440
msg: "aws_region must be defined when use_lambda is true."
@@ -80,6 +56,16 @@
8056
password: "{{ lookup('amazon.aws.aws_secret', '{{ secret_name }}.password', nested=true) }}"
8157
no_log: true
8258

59+
- name: Set junction_path to "/<volume_name>" if not provided.
60+
set_fact:
61+
junction_path: "/{{ volume_name }}"
62+
when: junction_path is not defined
63+
64+
- name: Set share_name to "<volume_name>" if not provided.
65+
set_fact:
66+
share_name: "{{ volume_name }}"
67+
when: share_name is not defined
68+
8369
- name: Delete CIFS Share
8470
netapp.ontap.na_ontap_cifs:
8571
state: absent
@@ -101,7 +87,6 @@
10187
state: absent
10288
name: "{{ volume_name }}"
10389
vserver: "{{ vserver }}"
104-
aggregate_name: "{{ aggr }}"
10590
use_lambda: "{{ use_lambda }}"
10691
lambda_config:
10792
aws_profile: "{{ aws_profile }}"

0 commit comments

Comments
 (0)