Skip to content

[add-topo] Refresh ARP table of neighbors with new MAC of new PTF#836

Merged
lguohan merged 1 commit intosonic-net:masterfrom
wangxin:add-topo-pr
Apr 5, 2019
Merged

[add-topo] Refresh ARP table of neighbors with new MAC of new PTF#836
lguohan merged 1 commit intosonic-net:masterfrom
wangxin:add-topo-pr

Conversation

@wangxin
Copy link
Collaborator

@wangxin wangxin commented Mar 21, 2019

Description of PR

Summary:
Fixes # (issue)
The PTF container will be destroyed if testbed-cli.sh remove-topo is
executed. Run testbed-cli.sh add-topo will add a new PTF conainer.
Usually the new PTF container will have a new MAC address. If add-topo
is executed immediately after remove-topo, ARP table of neighbor
switches and hosts may still have entry of the old PTF MAC address. This
would cause connectivity issue to the new PTF container for a while
until the old PTF MAC address is expired.

This workaround is to send out an ARPing from the PTF container querying
mgmt_gw after new PTF container is deployed and attached to network.
The ARPing request will be broadcasted to all neighbors on same LAN and
will refresh ARP table of neighbors with new MAC address of new PTF.

Type of change

  • Bug fix
  • [] Testbed and Framework(new/improvement)
  • [] Test case(new/improvement)

Approach

How did you do it?

Send an ARPing request from the PTF container after it is deployed and attached to network.

How did you verify/test it?

Tested on Mellanox platform

Any platform specific information?

No

Supported testbed topology if it's a new test case?

Documentation

… PTF container

The PTF container will be destroyed if testbed-cli.sh remove-topo is
executed. Run testbed-cli.sh add-topo will add a new PTF conainer.
Usually the new PTF container will have a new MAC address. If add-topo
is executed immediately after remove-topo, ARP table of neighbor
switches and hosts may still have entry of the old PTF MAC address. This
would cause connectivity issue to the new PTF container for a while
until the old PTF MAC address is expired.

This workaround is to send out an ARPing from the PTF container querying
mgmt_gw after new PTF container is deployed and attached to network.
The ARPing request will be broadcasted to all neighbors on same LAN and
will refresh ARP table of neighbors with new MAC address of new PTF.

Signed-off-by: Xin Wang <[email protected]>
@lguohan
Copy link
Contributor

lguohan commented Mar 22, 2019

does this only affect the mgmt network? It looks like you only ARP for the mgmt gw.

@wangxin
Copy link
Collaborator Author

wangxin commented Mar 22, 2019

@lguohan Yes, it only affects the mgmt network. Without this workaround, we experienced connectivity issue to mgmt IP of re-deployed PTF container.

@liat-grozovik liat-grozovik requested a review from lguohan March 28, 2019 13:51
@lguohan lguohan merged commit 90d0eaf into sonic-net:master Apr 5, 2019
yxieca pushed a commit that referenced this pull request Apr 5, 2019
… PTF container (#836)

The PTF container will be destroyed if testbed-cli.sh remove-topo is
executed. Run testbed-cli.sh add-topo will add a new PTF conainer.
Usually the new PTF container will have a new MAC address. If add-topo
is executed immediately after remove-topo, ARP table of neighbor
switches and hosts may still have entry of the old PTF MAC address. This
would cause connectivity issue to the new PTF container for a while
until the old PTF MAC address is expired.

This workaround is to send out an ARPing from the PTF container querying
mgmt_gw after new PTF container is deployed and attached to network.
The ARPing request will be broadcasted to all neighbors on same LAN and
will refresh ARP table of neighbors with new MAC address of new PTF.

Signed-off-by: Xin Wang <[email protected]>
@wangxin wangxin deleted the add-topo-pr branch May 24, 2019 03:33
kazinator-arista pushed a commit to kazinator-arista/sonic-mgmt that referenced this pull request Mar 4, 2026
Make sure db_migrator is run after all config are loaded during (sonic-net#926)
Vnet alias mapping (sonic-net#924)
Changes to make lldp show command for multi-npu platforms. (sonic-net#914)
[Mellanox] Fix thermal control issue: use natural sort for fan
status and thermal status (sonic-net#836)
[Mellanox] add document for thermal control related cli (sonic-net#832)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants