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?

@selldinesh selldinesh force-pushed the pr_phase4_new_cases branch 3 times, most recently from 9ffd0b4 to 42a9248 Compare October 5, 2023 15:26
@selldinesh selldinesh closed this Oct 5, 2023
@selldinesh selldinesh reopened this Oct 5, 2023
@selldinesh selldinesh force-pushed the pr_phase4_new_cases branch from 0406947 to 57801a2 Compare October 5, 2023 16:52
@selldinesh selldinesh closed this Oct 5, 2023
@selldinesh selldinesh force-pushed the pr_phase4_new_cases branch from 57801a2 to 3840d2d Compare October 5, 2023 17:11
@selldinesh selldinesh reopened this Oct 5, 2023
Cases that passed in tofino, failed in CI/CD

Signed-off-by: Dinesh Kumar Sellappan <[email protected]>
@selldinesh selldinesh force-pushed the pr_phase4_new_cases branch from 0e8d649 to 1cd40f6 Compare October 5, 2023 17:20
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