Add Intf config state to StateDB (sonic-net#642)

Signed-off-by: Jipan Yang <jipan.yang@alibaba-inc.com>
madhanmellanox · Oct 16, 2018 · e1cc0de · e1cc0de
1 parent 1f8e499
commit e1cc0de
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 1 deletion.
diff --git a/cfgmgr/intfmgr.cpp b/cfgmgr/intfmgr.cpp
@@ -21,6 +21,7 @@ IntfMgr::IntfMgr(DBConnector *cfgDb, DBConnector *appDb, DBConnector *stateDb, c
         m_statePortTable(stateDb, STATE_PORT_TABLE_NAME),
         m_stateLagTable(stateDb, STATE_LAG_TABLE_NAME),
         m_stateVlanTable(stateDb, STATE_VLAN_TABLE_NAME),
+        m_stateIntfTable(stateDb, STATE_INTERFACE_TABLE_NAME),
         m_appIntfTableProducer(appDb, APP_INTF_TABLE_NAME)
 {
 }
@@ -108,10 +109,12 @@ void IntfMgr::doTask(Consumer &consumer)
                 continue;
             }
             setIntfIp(alias, "add", ip_prefix.to_string(), ip_prefix.isV4());
+            m_stateIntfTable.hset(keys[0] + state_db_key_delimiter + keys[1], "state", "ok");
         }
         else if (op == DEL_COMMAND)
         {
             setIntfIp(alias, "del", ip_prefix.to_string(), ip_prefix.isV4());
+            m_stateIntfTable.del(keys[0] + state_db_key_delimiter + keys[1]);
         }
         else
         {

diff --git a/cfgmgr/intfmgr.h b/cfgmgr/intfmgr.h
@@ -19,7 +19,7 @@ class IntfMgr : public Orch
 private:
     ProducerStateTable m_appIntfTableProducer;
     Table m_cfgIntfTable, m_cfgVlanIntfTable;
-    Table m_statePortTable, m_stateLagTable, m_stateVlanTable;
+    Table m_statePortTable, m_stateLagTable, m_stateVlanTable, m_stateIntfTable;
 
     bool setIntfIp(const string &alias, const string &opCmd, const string &ipPrefixStr, const bool ipv4 = true);
     void doTask(Consumer &consumer);

diff --git a/orchagent/orch.h b/orchagent/orch.h
@@ -30,6 +30,7 @@ const char ref_end             = ']';
 const char comma               = ',';
 const char range_specifier     = '-';
 const char config_db_key_delimiter = '|';
+const char state_db_key_delimiter  = '|';
 
 #define MLNX_PLATFORM_SUBSTRING "mellanox"
 #define BRCM_PLATFORM_SUBSTRING "broadcom"