Skip TestExportAndImportMultiLayer on s390x#13149
Conversation
The test image ghcr.io/containerd/volume-copy-up:2.1 does not include a manifest for s390x, causing the test to fail with: "no manifest found for platform: not found". Signed-off-by: Ricardo Branco <rbranco@suse.de>
samuelkarp
left a comment
There was a problem hiding this comment.
We don't actually have access to any s390x machines to test on, so we can't really provide any guarantees about containerd behavior on that architecture. With that said, I'm fine accepting a test skip here. If s390x needs further architecture-specific changes within the core of containerd, that will probably need to be a different discussion around both machine access and maintainer bandwidth to troubleshoot.
We're running these tests on s390x :) |
mxpv
left a comment
There was a problem hiding this comment.
Curious, is this the only test that fails on s390x ? As Sam pointed out, we don't run these tests on s390x, so it's a blind area for us. But also I'm surprised that that's the only problematic test for this architecture.
Either way, should be ok to have this PR in.
You are, we (the containerd project) are not. |
Yes. I tested this on the containerd v1.7.29 we have on SLES 16.0. We don't test critest on SLES because cri-tools is only packaged for openSUSE Tumbleweed. Will test containerd v2.2.1 and report back. Here you can see an overview of the tests we run on openSUSE Tumbleweed: https://github.com/os-autoinst/os-autoinst-distri-opensuse/blob/master/tests/containers/README.md We expect to fix coverage for s390x on openqa.opensuse.org hopefully this year. |
I tested containerd v2.2.1 on SLES 16.1 and I only got these failures due to missing test images: I opened a new PR for these. |
I assume you want this cherry-picked back to release/1.7 then? |
It wouldn't hurt but, unlike Fedora/Redhat, we don't ship |
|
What's your use-case for this then? How are you checking out the code and running tests? Are you mostly looking at future releases (upcoming 2.3 soon)? |
We run tests independently in openQA. We backport test fixes as needed. There are some build-time tests but I don't know how is the packager doing it. How can we backport this one to 1.7.x? |
|
If you need it on release/1.7 we can take that change. I'm just trying to understand if you need it, and which branches you'd need the change on. |
|
/cherrypick release/1.7 |
|
@samuelkarp: new pull request created: #13152 DetailsIn 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-sigs/prow repository. |
|
@samuelkarp: new pull request created: #13153 DetailsIn 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-sigs/prow repository. |
|
@samuelkarp: new pull request could not be created: failed to create pull request against containerd/containerd#release/1.7 from head k8s-infra-cherrypick-robot:cherry-pick-13149-to-release/1.7: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for k8s-infra-cherrypick-robot:cherry-pick-13149-to-release/1.7."}],"documentation_url":"https://docs.github.com/rest/pulls/pulls#create-a-pull-request","status":"422"} DetailsIn 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-sigs/prow repository. |
|
@samuelkarp: new pull request could not be created: failed to create pull request against containerd/containerd#release/2.1 from head k8s-infra-cherrypick-robot:cherry-pick-13149-to-release/2.1: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for k8s-infra-cherrypick-robot:cherry-pick-13149-to-release/2.1."}],"documentation_url":"https://docs.github.com/rest/pulls/pulls#create-a-pull-request","status":"422"} DetailsIn 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-sigs/prow repository. |
|
@samuelkarp: new pull request created: #13154 DetailsIn 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-sigs/prow repository. |
We use 1.7 on SLES & openSUSE, and SUSE Rancher uses 2.0. Thanks a lot. |
|
@samuelkarp: new pull request could not be created: failed to create pull request against containerd/containerd#release/2.2 from head k8s-infra-cherrypick-robot:cherry-pick-13149-to-release/2.2: status code 422 not one of [201], body: {"message":"Validation Failed","errors":[{"resource":"PullRequest","code":"custom","message":"A pull request already exists for k8s-infra-cherrypick-robot:cherry-pick-13149-to-release/2.2."}],"documentation_url":"https://docs.github.com/rest/pulls/pulls#create-a-pull-request","status":"422"} DetailsIn 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-sigs/prow repository. |
2.0 is currently EOL: https://containerd.io/releases/#current-state-of-containerd-releases |
Skip TestExportAndImportMultiLayer on s390x
The test image
ghcr.io/containerd/volume-copy-up:2.1 does not include a manifest for s390x, causing the test to fail with:"no manifest found for platform: not found".