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

[ci]Use sonictest instead of sonic-common for test stage. #3485

Merged
merged 1 commit into from
Feb 3, 2025

Conversation

liushilongbuaa
Copy link
Contributor

@liushilongbuaa liushilongbuaa commented Jan 27, 2025

What I did
Something wrong in agent pool sonictest and sonic-common. No one maintain the agent initial script.
Now I did a quick fix on these two agent pool. sonictest agent pool works but sonic-common still has issues.

Why I did it

  1. sonictest is by design to run swss related tests. swss-common and sairedis tests run on sonictest. Only swss run on sonic-common. It is not as expected.

How I verified it

Details if related

@mssonicbld
Copy link
Collaborator

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@prsunny
Copy link
Collaborator

prsunny commented Jan 27, 2025

@liushilongbuaa , could you update the description and why the change is required? Whats the failure with sonic-common?

@liushilongbuaa
Copy link
Contributor Author

@prsunny , description updated.

@prsunny prsunny merged commit 29f2f48 into sonic-net:202311 Feb 3, 2025
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants