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
ci: test dragonball stability and cri-containerd #8737
ci: test dragonball stability and cri-containerd #8737
Conversation
3d64e7a
to
ed23a9f
Compare
b2bfe81
to
9fc8235
Compare
4605ecf
to
a363b39
Compare
/test |
bc9f4a1
to
9432428
Compare
/test |
# if [ "$KATA_HYPERVISOR" != "dragonball" ]; then | ||
# TestContainerMemoryUpdate 1 | ||
# TestContainerMemoryUpdate 0 | ||
# fi | ||
|
||
# TestContainerMemoryUpdate 0 |
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.
You want to remove this, no?
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.
Hi @fidencio Honestly, I'm not sure if this is the right way to do it, or if there's a better way to solve the CI error issue. The current situation is that runtime-rs/qemu
does not have this feature implemented.
34f2402
to
4744c09
Compare
|
||
if [ $test_virtio_mem -eq 1 ]; then | ||
if [[ "$ARCH" != "x86_64" ]]; then | ||
return | ||
fi | ||
info "Test container memory update with virtio-mem" | ||
|
||
sudo sed -i -e 's/^#enable_virtio_mem.*$/enable_virtio_mem = true/g' "${kata_config}" |
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.
why can't we use kata_config here?
64b9a5b
to
8f6b812
Compare
It needs to remove the skip setting, and make it work for dragonball. Fixes: kata-containers#8746 Signed-off-by: alex.lyn <alex.lyn@antgroup.com>
Containerd runtime options with wrong setting cause it failed. Correct it as below: ... [plugins.cri.containerd.runtimes.${runtime}.options] ConfigPath= "${KATA_CONFIG_PATH}" ... Fixes: kata-containers#8746 Signed-off-by: alex.lyn <alex.lyn@antgroup.com>
8f6b812
to
25e32af
Compare
Fixes: kata-containers#8746 Signed-off-by: alex.lyn <alex.lyn@antgroup.com>
25e32af
to
b97efc3
Compare
/test |
CI: make CI work for dragonball to test stability and cri-containerd
It needs to remove the skip setting, and make it work for dragonball.
bugfix when CI running cri-containerd
Fixes #8746