Skip to content

Commit

Permalink
[bug]: do not enumerate all ports in portsyncd init (sonic-net#380)
Browse files Browse the repository at this point in the history
portsyncd enumerate all ports in initialization phase via netlink.
which is not neccessary by design. portsyncd is supposed to read
the port_config.ini and translate them into appdb, and then once
all ports are created, it sets PortInitDone.

enumerating all ports in the initialization causes issue in swss
restart. When swss restarts, the old ports are still here which
triggers portsyncd to set PortInitDone falsely.
  • Loading branch information
lguohan authored Nov 13, 2017
1 parent d9bb152 commit 0be8b8b
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion portsyncd/portsyncd.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,6 @@ int main(int argc, char **argv)

netlink.registerGroup(RTNLGRP_LINK);
cout << "Listen to link messages..." << endl;
netlink.dumpRequest(RTM_GETLINK);

handlePortConfigFile(p, port_config_file);

Expand Down

0 comments on commit 0be8b8b

Please sign in to comment.