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

Adding new set of testcases #202

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

selldinesh
Copy link
Contributor

Description of PR
Summary: Contributing SAI test cases that qualifies the SAI apis. This PR consists of test cases that does Get and Set operation on the API.
The cases have been validated to be passing on the DUT with Tofino based Barefoot ASIC

SAI-Challenger Version: :e864c3f1cae2f8db96d96d7f82479415301b6e3d

@dks0692
Copy link

dks0692 commented Sep 26, 2023

@andriy-kokhan the cases of this PR has been passing on tofino based HW, but they are failing in the CI/CD pipeline so what do we do in that case?

@andriy-kokhan
Copy link
Contributor

@andriy-kokhan the cases of this PR has been passing on tofino based HW, but they are failing in the CI/CD pipeline so what do we do in that case?

We can not use any SAI library implementation as a reference. The test cases must be valid from SAI spec perspective. Are they valid now? What is the fail reason?

Cases that passed in tofino, failed in CI/CD

Signed-off-by: Dinesh Kumar Sellappan <dinesh.sellappan@keysight.com>
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.

3 participants