Skip to content

feat(ci): add stress test to CI #817

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

Merged
merged 8 commits into from
Jan 30, 2025
Merged

Conversation

Mossaka
Copy link
Member

@Mossaka Mossaka commented Jan 28, 2025

This commit adds a stress test to CI with 100 tasks and either # of cores or 10 parallelism

@Mossaka
Copy link
Member Author

Mossaka commented Jan 28, 2025

/home/runner/work/runwasi/runwasi/dist/bin/containerd-shim-wamr-v1 --count=100 --parallel=4 --timeout=2s`
Error: Scheme "" is not supported

I wonder where this error message is from

@Mossaka Mossaka closed this Jan 28, 2025
@Mossaka Mossaka reopened this Jan 28, 2025
@jprendes
Copy link
Collaborator

I think it's from the address provided to the shim, iirc it should be "unix://path/to/socket", and scheme refers to that "unix://", but I don't know why that would be the case

@jprendes
Copy link
Collaborator

Finally figured it out.
It's because in CI we use cross, and cross run executes things in a qemu environment, which has some unexpected behaviours.
Changing to cargo run should fix the issue.

Mossaka and others added 8 commits January 30, 2025 13:24
This commit adds a stress test to CI with 100 tasks and either # of cores or 10 parallelism

Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
Signed-off-by: Jiaxiao (mossaka) Zhou <[email protected]>
Signed-off-by: Jorge Prendes <[email protected]>
@Mossaka Mossaka merged commit 5c7825d into containerd:main Jan 30, 2025
74 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.

2 participants