COPP_DEL_fix: DEL for one trap group from SONIC is resetting all the …

…trap IDs (sonic-net#1273)
qiluo-msft · Jun 23, 2020 · a0b6412 · a0b6412
1 parent 17a2f93
commit a0b6412
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/orchagent/copporch.cpp b/orchagent/copporch.cpp
@@ -731,12 +731,12 @@ task_process_status CoppOrch::processCoppRule(Consumer& consumer)
             if (it.second.trap_group_obj == m_trap_group_map[trap_group_name])
             {
                 trap_ids_to_reset.push_back(it.first);
-            }
-            sai_status = sai_hostif_api->remove_hostif_trap(it.second.trap_obj);
-            if (sai_status != SAI_STATUS_SUCCESS)
-            {
-                SWSS_LOG_ERROR("Failed to remove trap object %" PRId64 "", it.second.trap_obj);
-                return task_process_status::task_failed;
+                sai_status = sai_hostif_api->remove_hostif_trap(it.second.trap_obj);
+                if (sai_status != SAI_STATUS_SUCCESS)
+                {
+                    SWSS_LOG_ERROR("Failed to remove trap object %" PRId64 "", it.second.trap_obj);
+                    return task_process_status::task_failed;
+                }
             }
         }