-
Notifications
You must be signed in to change notification settings - Fork 1k
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
run e2e tests using crun #2943
run e2e tests using crun #2943
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: giuseppe The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Codecov Report
@@ Coverage Diff @@
## master #2943 +/- ##
=======================================
Coverage 47.08% 47.08%
=======================================
Files 88 88
Lines 7155 7155
=======================================
Hits 3369 3369
Misses 3496 3496
Partials 290 290 |
0ef0b3c
to
21cba04
Compare
Depends on: cri-o/cri-o#2943 Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
21cba04
to
42c8d89
Compare
I've pushed another version where crun installation can be configured. crun tests won't run until openshift-eng/aos-cd-jobs#2027 is not merged |
f2678a3
to
a5f3bf4
Compare
(this will need #2955 to be in before any jenkins tests will pass BTW) |
f81dcb5
to
bfd0925
Compare
b28fa59
to
b921dcc
Compare
LGTM @giuseppe can WIP be removed? |
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
b921dcc
to
ee6eda7
Compare
@mrunalp @haircommander PTAL |
LGTM, but you still had one test fail. |
Am i correct in thinking there aren't any tests running with crun yet? This seems to lay the skeleton |
LGTM assuming happy tests |
@haircommander Yup that is the idea. The real goal is to get to the point we can run on cgroup V2 if kubernetes patches get in, prior to runc getting support. |
I've triggered the tests again. integration-static-glibc failed twice on the same test, but there are no clear changes here that could cause it (it was in network.bats). I'm hoping some sun spot flips a bit and it goes green |
in a previous version I manually forced to run with crun and the e2e tests passed. So I've changed it back to be configurable and we need: openshift-eng/aos-cd-jobs#2027 to run both crun and runc tests |
finally happy tests |
/retest |
/cherrypick release-1.15 |
@giuseppe: #2943 failed to apply on top of branch "release-1.15":
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
1 similar comment
@giuseppe: #2943 failed to apply on top of branch "release-1.15":
In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
Signed-off-by: Giuseppe Scrivano gscrivan@redhat.com