Skip to content

Commit ec2104a

Browse files
sajaganalhercot
authored andcommitted
[ignore] Fixed documentation and test file changes on aci_endpoint_ip_tag, aci_endpoint_mac_tag modules
1 parent 98d1311 commit ec2104a

File tree

4 files changed

+673
-795
lines changed

4 files changed

+673
-795
lines changed

plugins/modules/aci_endpoint_ip_tag.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
description:
3030
- The IPv4 or IPv6 address of the Endpoint IP Tag.
3131
type: str
32-
aliases: [ ip_addr ]
32+
aliases: [ ip_addr, ip ]
3333
vrf:
3434
description:
3535
- The name of the VRF.
@@ -51,7 +51,12 @@
5151
- cisco.aci.annotation
5252
- cisco.aci.owner
5353
54+
notes:
55+
- The O(tenant) and O(vrf) used must exist before using this module in your playbook.
56+
- The M(cisco.aci.aci_tenant) and M(cisco.aci.aci_vrf) modules can be used for this.
5457
seealso:
58+
- module: cisco.aci.aci_tenant
59+
- module: cisco.aci.aci_vrf
5560
- name: APIC Management Information Model reference
5661
description: More information about the internal APIC class B(fv:EpIpTag).
5762
link: https://developer.cisco.com/docs/apic-mim-ref/
@@ -229,7 +234,7 @@ def main():
229234
argument_spec.update(
230235
tenant=dict(type="str"),
231236
name=dict(type="str"),
232-
endpoint_ip_address=dict(type="str", aliases=["ip_addr"]),
237+
endpoint_ip_address=dict(type="str", aliases=["ip_addr", "ip"]),
233238
vrf=dict(type="str", aliases=["vrf_name"]),
234239
name_alias=dict(type="str"),
235240
state=dict(type="str", default="present", choices=["absent", "present", "query"]),

plugins/modules/aci_endpoint_mac_tag.py

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
description:
3030
- The MAC address of the Endpoint MAC Tag.
3131
type: str
32-
aliases: [ mac_addr ]
32+
aliases: [ mac_addr, mac ]
3333
bd:
3434
description:
3535
- The name of the Bridge Domain.
@@ -51,9 +51,14 @@
5151
- cisco.aci.annotation
5252
- cisco.aci.owner
5353
54+
notes:
55+
- The C(tenant) and C(bd) used must exist before using this module in your playbook.
56+
- The M(cisco.aci.aci_tenant) and M(cisco.aci.aci_bd) modules can be used for this.
5457
seealso:
58+
- module: cisco.aci.aci_tenant
59+
- module: cisco.aci.aci_bd
5560
- name: APIC Management Information Model reference
56-
description: More information about the internal APIC class B(fabric:LePortPGrp, fabric:SpPortPGrp).
61+
description: More information about the internal APIC class B(fv:EpMacTag).
5762
link: https://developer.cisco.com/docs/apic-mim-ref/
5863
author:
5964
- Sabari Jaganathan (@sajagana)
@@ -240,7 +245,7 @@ def main():
240245
argument_spec.update(
241246
tenant=dict(type="str"),
242247
name=dict(type="str"),
243-
endpoint_mac_address=dict(type="str", aliases=["mac_addr"]),
248+
endpoint_mac_address=dict(type="str", aliases=["mac_addr", "mac"]),
244249
bd=dict(type="str", aliases=["bd_name"]),
245250
name_alias=dict(type="str"),
246251
state=dict(type="str", default="present", choices=["absent", "present", "query"]),
@@ -250,8 +255,8 @@ def main():
250255
argument_spec=argument_spec,
251256
supports_check_mode=True,
252257
required_if=[
253-
["state", "absent", ["bd", "endpoint_mac_address"]],
254-
["state", "present", ["bd", "endpoint_mac_address"]],
258+
["state", "absent", ["tenant", "bd", "endpoint_mac_address"]],
259+
["state", "present", ["tenant", "bd", "endpoint_mac_address"]],
255260
],
256261
)
257262

0 commit comments

Comments
 (0)