-
Notifications
You must be signed in to change notification settings - Fork 346
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
Added podman local system tests #1009
Conversation
Signed-off-by: Stefin <stefin@pm.me>
Codecov Report
@@ Coverage Diff @@
## main #1009 +/- ##
=======================================
Coverage 69.22% 69.23%
=======================================
Files 118 118
Lines 12440 12439 -1
=======================================
Hits 8612 8612
+ Misses 3828 3827 -1 |
@stefins Thanks for your PR. How can I see the test result? |
run: sudo OCI_RUNTIME=/usr/local/bin/youki ./hack/bats 2>&1 | tee build.log | ||
- name: Adding Summary | ||
run: | | ||
echo "Total tests: 360 Failed tests: $(cat build.log | grep " ok " | wc -l)" >> $GITHUB_STEP_SUMMARY |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The expression is currently not evaluated. The test just prints
"Total tests: 360 Failed tests: $(cat build.log | grep " ok " | wc -l)" >> $GITHUB_STEP_SUMMARY
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I didn't wanted to add a RED status to the CI.
Maybe we should run this nightly? Running tests for 31 minutes on every pull request is quite long. |
Yeah that what I also thought. |
Thank you @stefins |
Hi,
This PR adds podman's local system tests to the CI and produces the summary like this.
As of now 134 tests out of 360 fails to run as of using
youki
as the runtime opposed tocrun
Related to #902
Thanks :)