[qos 202012] Refresh ARP entry with mac address of PTF port instead of DUT VLAN interface#7801
Merged
XuChen-MSFT merged 2 commits intosonic-net:202012from Mar 22, 2023
Conversation
neethajohn
approved these changes
Mar 21, 2023
Collaborator
|
This PR is for the 202012 branch. No need to add label "Request for 202012 branch". |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…fault vlan
Description of PR
Summary:
Fixes # (issue)
Type of change
Back port request
Approach
What is the motivation for this PR?
n testcase sai_qos_tests.HdrmPoolSizeTest:
Test traffic's destination port is one of PTF port.
To correctly forward test traffic to dest PTF port, we should populate a correct ARP entry on DUT before test.
But in HdrmPoolSizeTest case, build a incorrect ARP packet, as below:
RCA:
improperly use self.dst_port_mac to fill both ARP request sender address and test traffice's dest address:
We want to set test traffic's dest address to default vlan interface mac adress, so we change self.dst_port_mac to defalut vlan mac address before populate ARP. This address changing occured too early, and cuased incorrect ARP packet.
How did you do it?
When populate ARP, should use PTF port's mac address instead of DUT's default VLAN interface.
and then change traffca's dest mac address to default vlan interface during test.
How did you verify/test it?
Any platform specific information?
Supported testbed topology if it's a new test case?
Documentation