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

SAI Challenger docs & tutorial test-cases #300

Merged
merged 45 commits into from Jan 12, 2023

Conversation

chrispsommers
Copy link
Collaborator

@chrispsommers chrispsommers commented Dec 14, 2022

  • README describing SAI Challenger testbed
  • New SAI Challenger tutorials directory and test-cases which run in CI
  • Update Makefile and script to ensure ALL tests under test-cases/scale/saic are run, not just vnet
  • Add missing dependency trigger in CI .yml file
  • Update/refresh some older Test docs

@mhanif
Copy link
Collaborator

mhanif commented Dec 15, 2022

Hi @chrispsommers, as part of this PR, you may want to update the test/docs/dash-test-sai-challenger.md file as well. It has some outdated information. For example, to prepare a repository, it suggests the following:

git clone https://github.com/PLVision/DASH.git
cd DASH && git checkout test-framework-extension

Which, I believe, shouldn't be there. The code has been merged and this needs to be updated.

Not sure if you plan to delete/replace this file with some other documentations? Thanks

@chrispsommers
Copy link
Collaborator Author

chrispsommers commented Dec 15, 2022

Hi @chrispsommers, as part of this PR, you may want to update the test/docs/dash-test-sai-challenger.md file as well. It has some outdated information. For example, to prepare a repository, it suggests the following:

git clone https://github.com/PLVision/DASH.git
cd DASH && git checkout test-framework-extension

Which, I believe, shouldn't be there. The code has been merged and this needs to be updated.

Not sure if you plan to delete/replace this file with some other documentations? Thanks

Hi Hanif,
Thanks for reviewing. You are correct, this file is actually obsolete. I'd considered deleting it but was hesitating because there might be a few useful nuggets which could be moved to a workflows file. I think for now I'll delete it and refresh the workflow files as a follow-on task. I'll make an issue for that so as not to hold this PR up.

[UPDATE] I updated the doc in question to bring it up to date, thanks again for pointing it out!

@KrisNey-MSFT KrisNey-MSFT merged commit aff7eae into sonic-net:main Jan 12, 2023
@KrisNey-MSFT
Copy link
Collaborator

@xumia can you help us with the new Easy CLA check that was added ( and now not passing) please?

@xumia
Copy link
Collaborator

xumia commented Jan 12, 2023

/EasyCLA

@chrispsommers chrispsommers deleted the saic-tutorial1 branch January 31, 2023 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

6 participants