Skip to content

Commit cc57256

Browse files
committed
Removed need for specifying fabric_name
1 parent 12b9a17 commit cc57256

File tree

2 files changed

+29
-12
lines changed

2 files changed

+29
-12
lines changed

roles/ndfc_rest_vpc_create/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ Create vpc peering ``vpc_name`` in fabric ``fabric_name``
66

77
Variable | Type | Description
88
----------------|-------|----------------------------------------
9-
fabric_name | str() | The fabric in which ``vpc_name`` resides
109
vpc_name | str() | The name of the vpc peering to create
1110

1211
``fabric_name`` and ``vpc_name`` are defined in the following file:

roles/ndfc_rest_vpc_create/tasks/main.yml

Lines changed: 29 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -6,17 +6,26 @@
66
# - else
77
# skip
88

9+
- ansible.builtin.include_role:
10+
name: ndfc_vpc_config_get
11+
12+
- ansible.builtin.include_role:
13+
name: ndfc_device_config_get
14+
vars:
15+
device_name: "{{ vpc_config.peer_1 }}"
16+
17+
# TODO: Use ndfc_device_info_get here (see ndfc_vpc_interface_config_merged)
918
# set_fact for p1_serial, p1_vpc, p1_db_id
10-
- name: main query FABRIC {{ fabric_name }} VPC {{ vpc_name }} vpc_peer_1 {{ ip }}
19+
- name: main query FABRIC {{ device_config.switch_fabric }} VPC {{ vpc_name }} vpc_peer_1 {{ device_config.ip }}
1120
cisco.dcnm.dcnm_inventory:
12-
fabric: "{{ fabric_name }}"
21+
fabric: "{{ device_config.switch_fabric }}"
1322
state: query
1423
config:
15-
- seed_ip: "{{ ip }}"
24+
- seed_ip: "{{ device_config.ip }}"
1625
register: mr1
1726
vars:
18-
mq1: "[?fabric == '{{ fabric_name }}'] | [?vpc_name == '{{ vpc_name }}'].peer_1_ip | [0]"
19-
ip: "{{ vpc_peers | json_query(mq1)}}"
27+
ansible_connection: httpapi
28+
2029
- name: main set_fact p1_serial, p1_vpc, p1_db_id
2130
set_fact:
2231
p1_serial: "{{ mr1.response | json_query(mq2) }}"
@@ -27,17 +36,25 @@
2736
mq3: "[0].isVpcConfigured"
2837
mq4: "[0].switchDbID"
2938

39+
- ansible.builtin.include_role:
40+
name: ndfc_device_config_get
41+
vars:
42+
device_name: "{{ vpc_config.peer_2 }}"
43+
44+
# TODO: Use ndfc_device_info_get here (see ndfc_vpc_interface_config_merged)
3045
# set_fact for p2_serial, p2_vpc, p2_db_id
31-
- name: main query FABRIC {{ fabric_name }} VPC {{ vpc_name }} vpc_peer_2
46+
- name: main query FABRIC {{ device_config.switch_fabric }} VPC {{ vpc_name }} vpc_peer_2
3247
cisco.dcnm.dcnm_inventory:
33-
fabric: "{{ fabric_name }}"
48+
fabric: "{{ device_config.switch_fabric }}"
3449
state: query
3550
config:
36-
- seed_ip: "{{ ip }}"
51+
- seed_ip: "{{ device_config.ip }}"
3752
register: mr2
3853
vars:
39-
mq5: "[?fabric == '{{ fabric_name }}'] | [?vpc_name == '{{ vpc_name }}'].peer_2_ip | [0]"
40-
ip: "{{ vpc_peers | json_query(mq5)}}"
54+
ansible_connection: httpapi
55+
# mq5: "[?fabric == '{{ device_config.switch_fabric }}'] | [?vpc_name == '{{ vpc_name }}'].peer_2_ip | [0]"
56+
# ip: "{{ vpc_peers | json_query(mq5)}}"
57+
4158
- name: main set_fact p2_serial, p2_vpc, p2_db_id
4259
set_fact:
4360
p2_serial: "{{ mr2.response | json_query(mq6) }}"
@@ -49,12 +66,13 @@
4966
mq8: "[0].switchDbID"
5067

5168
# Conditionally create vpc peering
52-
- name: main REST create VPC Peers fabric_name {{ fabric_name }} vpc_name {{ vpc_name }} configured {{ p1_vpc }}/{{ p2_vpc }} serial_number {{ p1_serial }}/{{ p2_serial }}"
69+
- name: main REST create VPC Peers fabric_name {{ device_config.switch_fabric }} vpc_name {{ vpc_name }} configured {{ p1_vpc }}/{{ p2_vpc }} serial_number {{ p1_serial }}/{{ p2_serial }}"
5370
cisco.dcnm.dcnm_rest:
5471
method: POST
5572
path: "/appcenter/cisco/ndfc/api/v1/lan-fabric/rest/vpcpair"
5673
json_data: "{{ vars_dict | to_json }}"
5774
vars:
75+
ansible_connection: httpapi
5876
vars_dict:
5977
peerOneId: "{{ p1_serial }}"
6078
peerTwoId: "{{ p2_serial }}"

0 commit comments

Comments
 (0)