[ignore] Modify payload for Node1Details and Node2Details if object i…

…s being updated or created for ndo_virtual_port_channel_interface module.
CiscoDevNet · Dec 19, 2024 · 94aba9f · 94aba9f
1 parent bf9e238
commit 94aba9f
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 16 deletions.
diff --git a/plugins/modules/ndo_virtual_port_channel_interface.py b/plugins/modules/ndo_virtual_port_channel_interface.py
@@ -385,26 +385,35 @@ def main():
             fabric_policy_template.validate_template("fabricPolicy")
             interface_policy_group_uuid = fabric_policy_template.get_interface_policy_group_uuid(interface_policy_group.get("name"))
 
-        mso_values = dict(
-            name=name,
-            description=description,
-            node1Details=dict(
-                node=node_1,
-                memberInterfaces=interfaces_node_1,
-            ),
-            node2Details=dict(
-                node=node_2,
-                memberInterfaces=interfaces_node_2,
-            ),
-            policy=interface_policy_group_uuid,
-            interfaceDescriptions=format_interface_descriptions(mso, interface_descriptions),
-        )
-
         if mso.existing:
+            mso_values = {
+                "name": name,
+                "description": description,
+                ("node1Details", "node"): node_1,
+                ("node1Details", "memberInterfaces"): interfaces_node_1,
+                ("node2Details", "node"): node_2,
+                ("node2Details", "memberInterfaces"): interfaces_node_2,
+                "policy": interface_policy_group_uuid,
+                "interfaceDescriptions": format_interface_descriptions(mso, interface_descriptions),
+            }
             append_update_ops_data(ops, match.details, virtual_port_channel_attrs_path, mso_values)
             mso.sanitize(match.details, collate=True)
 
         else:
+            mso_values = dict(
+                name=name,
+                description=description,
+                node1Details=dict(
+                    node=node_1,
+                    memberInterfaces=interfaces_node_1,
+                ),
+                node2Details=dict(
+                    node=node_2,
+                    memberInterfaces=interfaces_node_2,
+                ),
+                policy=interface_policy_group_uuid,
+                interfaceDescriptions=format_interface_descriptions(mso, interface_descriptions),
+            )
             mso.sanitize(mso_values)
             ops.append(dict(op="add", path="/fabricResourceTemplate/template/virtualPortChannels/-", value=mso.sent))
 

diff --git a/tests/integration/targets/ndo_virtual_port_channel_interface/tasks/main.yml b/tests/integration/targets/ndo_virtual_port_channel_interface/tasks/main.yml
@@ -385,7 +385,7 @@
       register: query_one
 
     - name: Query a virtual port channel with template_name and UUID
-      cisco.mso.ndo_tenant_custom_qos_policy: &query_virtual_port_channel_uuid
+      cisco.mso.ndo_virtual_port_channel_interface: &query_virtual_port_channel_uuid
         <<: *create_virtual_port_channel_interface_2
         uuid: '{{ query_one.current.uuid }}'
         name: '{{ fakevar | default(omit)}}'