[mirrororch]: Remove mirror session state after it is remvoed (#1066)

Remove deprecated mirror session states in the state database

Signed-off-by: Shu0t1an Cheng <[email protected]>

Author: stcheng

orchagent/mirrororch.cpp

@@ -113,17 +113,17 @@ bool MirrorOrch::bake()
             }
         }
 
-        if (!active)
+        if (active)
         {
-            continue;
-        }
+            SWSS_LOG_NOTICE("Found mirror session %s active before warm reboot",
+                    key.c_str());
 
-        SWSS_LOG_NOTICE("Found mirror session %s active before warm reboot",
-                key.c_str());
+            // Recover saved active session's monitor port
+            m_recoverySessionMap.emplace(
+                    key, monitor_port + state_db_key_delimiter + next_hop_ip);
+        }
 
-        // Recover saved active session's monitor port
-        m_recoverySessionMap.emplace(
-                key, monitor_port + state_db_key_delimiter + next_hop_ip);
+	removeSessionState(key);
     }
 
     return Orch::bake();
@@ -382,6 +382,8 @@ void MirrorOrch::deleteEntry(const string& name)
         m_policerOrch->decreaseRefCount(session.policer);
     }
 
+    removeSessionState(name);
+
     m_syncdMirrors.erase(sessionIter);
 
     SWSS_LOG_NOTICE("Removed mirror session %s", name.c_str());
@@ -436,6 +438,13 @@ void MirrorOrch::setSessionState(const string& name, const MirrorEntry& session,
     m_mirrorTable.set(name, fvVector);
 }
 
+void MirrorOrch::removeSessionState(const string& name)
+{
+	SWSS_LOG_ENTER();
+
+	m_mirrorTable.del(name);
+}
+
 bool MirrorOrch::getNeighborInfo(const string& name, MirrorEntry& session)
 {
     SWSS_LOG_ENTER();

orchagent/mirrororch.h

@@ -108,6 +108,8 @@ class MirrorOrch : public Orch, public Observer, public Subject
      * attr is the field name will be stored, if empty then all fields will be stored
      */
     void setSessionState(const std::string& name, const MirrorEntry& session, const std::string& attr = "");
+    void removeSessionState(const std::string& name);
+
     bool getNeighborInfo(const string&, MirrorEntry&);
 
     void updateNextHop(const NextHopUpdate&);