[intfsorch] Create subport with the entry contains necessary attributes (sonic-net#1650)

peter-yu · web-flow · commit b6db9ddd6ded · 2021-03-05T06:59:35.000-08:00
Subport mtu and admin_status initialized to values of 0 and false resepectively.
diff --git a/orchagent/intfsorch.cpp b/orchagent/intfsorch.cpp
@@ -588,8 +588,8 @@ void IntfsOrch::doTask(Consumer &consumer)
         string vrf_name = "", vnet_name = "", nat_zone = "";
         MacAddress mac;
 
-        uint32_t mtu;
-        bool adminUp;
+        uint32_t mtu = 0;
+        bool adminUp = false;
         uint32_t nat_zone_id = 0;
         string proxy_arp = "";
         string inband_type = "";
@@ -742,7 +742,7 @@ void IntfsOrch::doTask(Consumer &consumer)
             Port port;
             if (!gPortsOrch->getPort(alias, port))
             {
-                if (isSubIntf)
+                if (!ip_prefix_in_key && isSubIntf)
                 {
                     if (!gPortsOrch->addSubPort(port, alias, adminUp, mtu))
                     {

Original file line number	Diff line number	Diff line change
`@@ -588,8 +588,8 @@ void IntfsOrch::doTask(Consumer &consumer)`
`588`	`588`	`string vrf_name = "", vnet_name = "", nat_zone = "";`
`589`	`589`	`MacAddress mac;`
`590`	`590`
`591`		`- uint32_t mtu;`
`592`		`- bool adminUp;`
	`591`	`+ uint32_t mtu = 0;`
	`592`	`+ bool adminUp = false;`
`593`	`593`	`uint32_t nat_zone_id = 0;`
`594`	`594`	`string proxy_arp = "";`
`595`	`595`	`string inband_type = "";`
`@@ -742,7 +742,7 @@ void IntfsOrch::doTask(Consumer &consumer)`
`742`	`742`	`Port port;`
`743`	`743`	`if (!gPortsOrch->getPort(alias, port))`
`744`	`744`	`{`
`745`		`- if (isSubIntf)`
	`745`	`+ if (!ip_prefix_in_key && isSubIntf)`
`746`	`746`	`{`
`747`	`747`	`if (!gPortsOrch->addSubPort(port, alias, adminUp, mtu))`
`748`	`748`	`{`