-
Notifications
You must be signed in to change notification settings - Fork 1k
[Dell-S6100][ARP]: Test port number need to be modified supporting 64 port devices #332
Copy link
Copy link
Open
Labels
Description
Scripts :
ansible/roles/test/tasks/arpall.yml
ansible/roles/test/files/acstests/arptest.py
Issue :
In current test scrript , test ip address is assigned to EThernet4 and Ethernet8.
The traffic is sent and verified in ports 1 and 2 in ptf. ( That corresponds to Ethernet4 and EThernet8 for 32 port devices like S6000)
arpall.yml :
- name: change SONiC DUT interface IP to test IP address
command: /sbin/ifconfig Ethernet4 10.10.1.2 netmask 255.255.255.240
become: yes
- name: change SONiC DUT interface IP to test IP address
command: /sbin/ifconfig Ethernet8 10.10.1.20 netmask 255.255.255.240
become: yes
arptest.py :
root@f5ada488ac2e:/var/johnar/sonic-mgmt/ansible# grep send_packet roles/test/files/acstests/arptest.py
send_packet(self, 1, pkt)
send_packet(self, 1, pkt)
send_packet(self, 2, pkt)
send_packet(self, 1, pkt)
send_packet(self, 1, pkt)
send_packet(self, 1, pkt)
root@f5ada488ac2e:/var/johnar/sonic-mgmt/ansible# grep verify_packet roles/test/files/acstests/arptest.py
verify_packet(self, exp_pkt, 1)
verify_packet(self, exp_pkt, 1)
This doesnt work in 64 port device S6100 as the ports are numbered as Ethernet1,Ethernet2 ..
The scripts for 6100 need to be changed as Ethernet1, Ethernet2, instead of Ethernet4 and Ethernet8 , for it to test properly.
Reactions are currently unavailable