[ignore] Added new attributes to aci_l4l7_policy_based_redirect_destination

Author: shrsr

plugins/modules/aci_l4l7_policy_based_redirect_destination.py

@@ -28,6 +28,10 @@
     - The name of an existing Policy Based Redirect Policy.
     type: str
     aliases: [ policy_name ]
+  description:
+    description:
+    - The description for redirection.
+    type: str
   ip:
     description:
     - The destination IP for redirection.
@@ -77,13 +81,19 @@
   pod_id:
     description:
     - The Pod ID to deploy Policy Based Redirect destination on.
-    - The APIC defaults to C(1) when unset during creation.
+    - The APIC defaults to 1 when unset during creation.
     type: int
   health_group:
     description:
     - The Health Group to bind the Policy Based Redirection Destination to.
     - To remove an existing binding from a Health Group, submit a request with I(state=present) and I(health_group="") value.
     type: str
+  weight:
+    description:
+    - The weight of the fault in calculating the health score of an object.
+    - The APIC defaults to 1 when unset during creation.
+    - Permitted values are in the range of [1, 10].
+    type: int
   state:
     description:
     - Use C(present) or C(absent) for adding or removing.
@@ -286,6 +296,8 @@ def main():
         health_group=dict(type="str"),
         state=dict(type="str", default="present", choices=["absent", "present", "query"]),
         pod_id=dict(type="int"),
+        weight=dict(type="int"),
+        description=dict(type="str"),
     )
 
     module = AnsibleModule(
@@ -313,6 +325,8 @@ def main():
     health_group = module.params.get("health_group")
     state = module.params.get("state")
     pod_id = module.params.get("pod_id")
+    weight = module.params.get("weight")
+    description = module.params.get("description")
 
     if destination_type == "l1/l2":
         aci_class = "vnsL1L2RedirectDest"
@@ -396,6 +410,8 @@ def main():
                 destName=destination_name,
                 podId=pod_id,
                 ip2=additional_ip,
+                weight=weight,
+                descr=description,
             ),
             child_configs=child_configs,
         )

plugins/modules/aci_l4l7_redirect_health_group.py

@@ -27,6 +27,11 @@
     description:
     - The name of the Health Group.
     type: str
+    aliases: [ name ]
+  description:
+    description:
+    - The description of the Health Group.
+    type: str
   state:
     description:
     - Use C(present) or C(absent) for adding or removing.
@@ -210,7 +215,8 @@ def main():
     argument_spec.update(
         tenant=dict(type="str", aliases=["tenant_name"]),
         state=dict(type="str", default="present", choices=["absent", "present", "query"]),
-        health_group=dict(type="str"),
+        health_group=dict(type="str", aliases=["name"]),
+        description=dict(type="str"),
     )
 
     module = AnsibleModule(
@@ -225,6 +231,7 @@ def main():
     tenant = module.params.get("tenant")
     state = module.params.get("state")
     health_group = module.params.get("health_group")
+    description = module.params.get("description")
 
     aci = ACIModule(module)
 
@@ -247,7 +254,10 @@ def main():
     if state == "present":
         aci.payload(
             aci_class="vnsRedirectHealthGroup",
-            class_config=dict(name=health_group),
+            class_config=dict(
+                name=health_group,
+                descr=description,
+            ),
         )
         aci.get_diff(aci_class="vnsRedirectHealthGroup")
 

tests/integration/targets/aci_l4l7_policy_based_redirect_destination/tasks/main.yml

@@ -21,6 +21,13 @@
       use_proxy: '{{ aci_use_proxy | default(true) }}'
       output_level: '{{ aci_output_level | default("info") }}'
 
+- name: Query system information
+  cisco.aci.aci_system:
+    <<: *aci_info
+    id: 1
+    state: query
+  register: version
+
 # CLEAN ENVIRONMENT
 - name: Remove ansible_tenant if it already exists
   cisco.aci.aci_tenant:
@@ -424,6 +431,31 @@
     - update_pbr_l1l2_dest.current.0.vnsL1L2RedirectDest.children.0.vnsRsToCIf.attributes.tDn == "uni/tn-ansible_tenant/lDevVip-ansible_device/cDev-ansible_concrete_device/cIf-[ansible_concrete_interface]"
     - update_pbr_l1l2_dest.current.0.vnsL1L2RedirectDest.children | length == 1
 
+- name: Execute tasks only for ACI v6+
+  when:
+  - version.current.0.topSystem.attributes.version is version('6', '>=')
+  block:
+  - name: Create another L4-L7 Policy Based Redirect Destination to check weight
+    cisco.aci.aci_l4l7_policy_based_redirect_destination:
+      <<: *aci_info
+      tenant: ansible_tenant
+      policy: ansible_pbr_policy
+      redirect_ip: 192.168.10.5
+      additional_ip: 192.168.50.5
+      redirect_mac: AB:CD:EF:12:34:61
+      dest_name: redirect_dest
+      health_group: ansible_health_group
+      pod_id: 1
+      weight: 5
+      state: present
+    register: add_weight
+
+  - name: Verify another L4-L7 Policy Based Redirect Destination to check weight
+    ansible.builtin.assert:
+      that:
+      - add_weight is changed
+      - add_weight.current.0.vnsRedirectDest.attributes.weight == "5"
+
 # QUERY L4-L7 PBR DESTINATION
 - name: Query L4-L7 Policy Based Redirect Destination
   cisco.aci.aci_l4l7_policy_based_redirect_destination: