Skip to content

Commit 15a2299

Browse files
author
Shuotian Cheng
authored
[vlanmgrd]: Do not bring up VLAN members by default (sonic-net#667)
VLAN members' admin status and MTU are controlled by portmdrd. vlanmgrd shall not bring up and set MTU automatically by default. Signed-off-by: Shu0T1an ChenG <[email protected]>
1 parent 44a4460 commit 15a2299

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

cfgmgr/vlanmgr.cpp

+2-4
Original file line numberDiff line numberDiff line change
@@ -169,13 +169,11 @@ bool VlanMgr::addHostVlanMember(int vlan_id, const string &port_alias, const str
169169

170170
// The command should be generated as:
171171
// /bin/bash -c "/sbin/ip link set {{port_alias}} master Bridge &&
172-
// /sbin/bridge vlan add vid {{vlan_id}} dev {{port_alias}} {{tagging_mode}}
173-
// /sbin/ip link set {{port_alias}} up mtu 9100"
172+
// /sbin/bridge vlan add vid {{vlan_id}} dev {{port_alias}} {{tagging_mode}}"
174173
const std::string cmds = std::string("")
175174
+ BASH_CMD + " -c \""
176175
+ IP_CMD + " link set " + port_alias + " master " + DOT1Q_BRIDGE_NAME + " && "
177-
+ BRIDGE_CMD + " vlan add vid " + std::to_string(vlan_id) + " dev " + port_alias + " " + tagging_cmd + " && "
178-
+ IP_CMD + " link set " + port_alias + " up mtu " + std::to_string(MAX_MTU) + "\"";
176+
+ BRIDGE_CMD + " vlan add vid " + std::to_string(vlan_id) + " dev " + port_alias + " " + tagging_cmd + "\"";
179177

180178
std::string res;
181179
EXEC_WITH_ERROR_THROW(cmds, res);

0 commit comments

Comments
 (0)