From a0b6412452487fa98b8f4862e92a3aebbb5e4339 Mon Sep 17 00:00:00 2001 From: SinghMinu <64151919+SinghMinu@users.noreply.github.com> Date: Tue, 23 Jun 2020 06:56:28 +0530 Subject: [PATCH] COPP_DEL_fix: DEL for one trap group from SONIC is resetting all the trap IDs (#1273) --- orchagent/copporch.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/orchagent/copporch.cpp b/orchagent/copporch.cpp index a3e1cb4a7801..00da16740272 100644 --- 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; + } } }