Skip to content
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

Added DIP=SIP testcase. #666

Merged
merged 3 commits into from
Jul 31, 2018
Merged

Added DIP=SIP testcase. #666

merged 3 commits into from
Jul 31, 2018

Conversation

nazariig
Copy link
Contributor

@nazariig nazariig commented Jul 26, 2018

Description of PR

The purpose of this test is to validate that SONiC switch supports routing of L3 packets with DIP=SIP

Type of change

  • [DIP=SIP] Test case(new/improvement)

Approach

How did you do it?

  • N/A

How did you verify/test it?

sudo -H ansible-playbook test_sonic.yml -i inventory --limit arc-switch1025-t0 -e testbed_name=arc-switch1025-t0 -e testbed_type=t0 -e testcase_name=dip_sip -vvvvv

Any platform specific information?

  • N/A

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

  • t0
  • t0-16
  • t0-56
  • t0-64
  • t0-64-32
  • t0-116
  • t1
  • t1-lag
  • t1-64-lag

Documentation

@msftclas
Copy link

msftclas commented Jul 26, 2018

CLA assistant check
All CLA requirements met.


dip_sip:
filename: dip_sip.yml
topologies: [t0, t1, t1-lag]
Copy link
Contributor

@qiluo-msft qiluo-msft Jul 26, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need t0-56 at least. It would be better to add all. #Closed

Copy link
Contributor Author

@nazariig nazariig Jul 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@qiluo-msft We need to discuss which topologies must be supported by this testcase. Regarding t0-56, I have no way to check it. Basically, we can use any topology with this testcase which contains at least two PORT/LAG RIFs connected to neighbors, providing the valid LLDP info.

Copy link
Contributor

@qiluo-msft qiluo-msft Jul 27, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then it's ok to add all known topo here. Others can help you test. #Closed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@qiluo-msft I have extended this testcase with the additional topologies as we discussed.


from ptf import config
from ptf.base_tests import BaseTest
from ptf.testutils import *
Copy link
Contributor

@qiluo-msft qiluo-msft Jul 26, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't import wildcard. #Closed

ip_ttl=self.pktTtl)
send(self.test, int(self.srcPortIds[0]), pkt)

pkt = simple_udp_packet(eth_dst=self.dstHostMac,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we also have one for IPv6?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prsunny Yes. Extended with IPv6 support.

self.dstHostIpv6 = str(ip_address(unicode(self.testParams['dst_router_ipv6'])) + 1)

self.srcRouterMac = self.testParams['src_router_mac']
self.srcRouterIpv4 = self.testParams['src_router_ipv4']
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note, looks like srcRouterIPvX and srcHostIpvX are not used in the test.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@prsunny Correct. It is just for logging purpose.

@prsunny prsunny merged commit 993c2d6 into sonic-net:master Jul 31, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants