Skip to content

[Dell-S6100][ARP]: Test port number need to be modified supporting 64 port devices #332

@chitra-raghavan

Description

@chitra-raghavan

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.

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions