Fix command deletion for QoS interfaces module (#473)

* Fix command deletion for QoS interfaces module * Correct PR #
ansible-collections · Nov 6, 2024 · cd4757e · cd4757e
1 parent 7b4b6f8
commit cd4757e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/changelogs/fragments/473-qos-interfaces-deletion-bugfix.yaml b/changelogs/fragments/473-qos-interfaces-deletion-bugfix.yaml
@@ -0,0 +1,2 @@
+bugfixes:
+  - sonic_qos_interfaces - Fix command deletion bug (https://github.com/ansible-collections/dellemc.enterprise_sonic/pull/473).
diff --git a/plugins/module_utils/network/sonic/config/qos_interfaces/qos_interfaces.py b/plugins/module_utils/network/sonic/config/qos_interfaces/qos_interfaces.py
@@ -380,7 +380,7 @@ def get_delete_qos_interfaces_requests(self, commands, have, is_delete_all):
             if scheduler_policy and scheduler_policy == cfg_scheduler_policy:
                 url = '%s/interface=%s/output/scheduler-policy' % (QOS_INTF_PATH, name)
                 requests.append({'path': url, 'method': DELETE})
-
+                config_dict.update({'name': name, 'scheduler_policy': scheduler_policy})
             if qos_maps and cfg_qos_maps:
                 maps_dict = {}
                 dscp_fwd_group = qos_maps.get('dscp_fwd_group')
@@ -426,7 +426,7 @@ def get_delete_qos_interfaces_requests(self, commands, have, is_delete_all):
                     requests.append(self.get_delete_map_request(name, 'pfc-priority-to-priority-group'))
                     maps_dict['pfc_priority_pg'] = pfc_priority_pg
                 if maps_dict:
-                    config_dict['qos_maps'] = maps_dict
+                    config_dict.update({'name': name, 'qos_maps': maps_dict})
 
             if pfc and cfg_pfc:
                 pfc_dict = {}
@@ -479,7 +479,7 @@ def get_delete_qos_interfaces_requests(self, commands, have, is_delete_all):
                     if priorities_list:
                         pfc_dict['priorities'] = priorities_list
                 if pfc_dict:
-                    config_dict['pfc'] = pfc_dict
+                    config_dict.update({'name': name, 'pfc': pfc_dict})
 
             if queues and cfg_queues:
                 queues_list = []
@@ -505,7 +505,7 @@ def get_delete_qos_interfaces_requests(self, commands, have, is_delete_all):
                     if queues_dict:
                         queues_list.append(queues_dict)
                 if queues_list:
-                    config_dict['queues'] = queues_list
+                    config_dict.update({'name': name, 'queues': queues_list})
             if config_dict:
                 config_list.append(config_dict)
             if not scheduler_policy and not qos_maps and not pfc and not queues: