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
test: use kata 3.3.0 for testing #7958
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #7958 +/- ##
==========================================
- Coverage 49.58% 49.56% -0.02%
==========================================
Files 153 153
Lines 16926 16955 +29
==========================================
+ Hits 8392 8404 +12
- Misses 7487 7504 +17
Partials 1047 1047 |
/retest-required |
/approve |
/ok-to-test |
/retest-required |
/hold |
3f63b89
to
2003e37
Compare
/test ci-fedora-kata |
/retest |
/retest |
/retest |
Signed-off-by: Julien Ropé <jrope@redhat.com>
2003e37
to
c7e5373
Compare
Looks like I found how to do it... But I'm confused. I thought setting the storage option in the configuration file would work. We're doing it on the kata CI jobs, and it works. Why? Is there an interaction with other options provided by env vars? Or because the storage driver is set on the command line? I will clean things up so we can merge it with env var setting, but if anybody has an explanation for this behavior, I'm willing to learn. |
473c846
to
f5bd3be
Compare
/unhold |
I think this one can be reviewed. @kwilczynski what do you think? |
Signed-off-by: Julien Ropé <jrope@redhat.com>
The kata runtime now runs in its own mount namespace. Because of that, crio should not create a private bind mount, otherwise the container rootfs can't be propagated to kata's namespace, and the kata container can't access the rootfs files. Signed-off-by: Julien Ropé <jrope@redhat.com>
f5bd3be
to
a544f29
Compare
/approve |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: kwilczynski, littlejawa The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What type of PR is this?
/kind ci
What this PR does / why we need it:
Update the version of kata to test with - using the latest 3.3.0 version.
Which issue(s) this PR fixes:
None
Special notes for your reviewer:
Does this PR introduce a user-facing change?