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

[Bug][202405]: Remote module mismatch for port in test_fabric_reach.py #15753

Closed
arista-hpandya opened this issue Nov 26, 2024 · 1 comment · Fixed by #15828
Closed

[Bug][202405]: Remote module mismatch for port in test_fabric_reach.py #15753

arista-hpandya opened this issue Nov 26, 2024 · 1 comment · Fixed by #15828

Comments

@arista-hpandya
Copy link
Contributor

Issue Description

The test fails on our T2 testbed because the supervisorAsicBase offset added to the remote module number creates a mismatch with the value fetched by the CLI from the STATE_DB

A simple fix would be to remove the supervisorAsicBase offset from the remote module number calculation here:

remoteMod = supervisorAsicBase + (remoteSlot - 1)*2 + remoteAsic

Results you see

Failed: Remote module mismatch for port 0

Results you expected to see

We expect the test to pass

Is it platform specific

generic

Relevant log output

No response

Output of show version

SONiC Software Version: SONiC.branch.202405-ars.a2ba5e5c-buildimage.origin.202405-nightly-2024.11.24.22.44
SONiC OS Version: 12
Distribution: Debian 12.8
Kernel: 6.1.0-22-2-amd64
Build commit: a4aa25a7f
Build date: Mon Nov 25 04:16:46 UTC 2024
Built by: jenkins@jenkins-arsonic-k8s-6-j8lwz

Platform: x86_64-arista_7800r3ak_36dm2_lc
HwSKU: Arista-7800R3AK-36DM2-C36
ASIC: broadcom
ASIC Count: 2
Serial Number: SSN21060161
Model Number: 7800R3AK-36DM2-LC
Hardware Revision: 29.00
Uptime: 22:36:24 up 1 day,  4:56,  1 user,  load average: 1.28, 1.55, 1.62
Date: Tue 26 Nov 2024 22:36:24

Attach files (if any)

No response

@yxieca
Copy link
Collaborator

yxieca commented Dec 11, 2024

@rlhui for viz.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants