-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[swss]: Wait for vlan intf to start ndppd #10036
Conversation
Signed-off-by: Lawrence Lee <lawlee@microsoft.com>
# Wait for the interface to come up | ||
# (i.e., interface is present in STATE_DB and state is "ok") | ||
while true; do | ||
RESULT=$(sonic-db-cli STATE_DB HGET "${TABLE_PREFIX}|${IFACE}" "state" 2> /dev/null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
During SWSS init, there is a cleanup of STATE_DB tables. Can you please confirm, this script is always executed after that cleanup?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This script waits for swssconfig.sh
to exit, and swssconfig.sh
waits for the /ready
flag which is only created after the DBs are cleared
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So just to confirm, ndppd won't get started on a T1 topology with this change, right?
/Azp run |
You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list. |
/Azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
/Azp run Azure.sonic-buildimage |
Azure Pipelines successfully started running 1 pipeline(s). |
This reverts commit 9120487.
Signed-off-by: Lawrence Lee lawlee@microsoft.com
Why I did it
If the VLAN interface is not up when ndppd starts, it will fail to enable
allmulti
mode on the interface and be unable to process received NDP packetsThe following logs are seen:
How I did it
Use the
wait_for_link
script currently used byradv
to delay ndppd startup until the vlan interface is readyHow to verify it
Apply the changes to a device.
config reload
the device and confirm that the above error logs are not observed when ndppd starts. Run thearp/test_arp_dualtor.py::test_proxy_arp
test case and verify it passes.Which release branch to backport (provide reason below if selected)
Description for the changelog
Link to config_db schema for YANG module changes
A picture of a cute animal (not mandatory but encouraged)