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.

Sign up for GitHub

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

Jump to bottom

libvirt: run CI for k8s configured with CRI-O #1981

Closed

wainersm opened this issue Aug 6, 2024 · 0 comments · Fixed by #2068

Labels

CI provider/libvirt

Member

wainersm commented Aug 6, 2024

Recently kata-deploy got support for configuring CRI-O to do guest-pull (kata-containers/kata-containers#9537), and finally CRI-O was added on CoCo operator (confidential-containers/operator#376).

The current CI jobs (on pull and daily) for libvirt are all testing k8s configured with containerd. This issue is to track the work required to test with CRI-O as well.

It seems the major changes will be:

Adapt kcli_cluster.sh to configure the cluster with CRI-O
The container runtime (containerd or cri-o) should be passed to the libvirt's cluster provisioner via properties
Convert e2e_libvirt.yaml workflow to matrix of jobs with entries for cri-o and containerd

wainersm added CI provider/libvirt labels

wainersm mentioned this issue

CI: revisit the configurations tested confidential-containers/operator#409

Open

stevenhorsman added this to CoCo Releases

github-project-automation bot moved this to 🆕 New in CoCo Releases

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          libvirt/kcli_cluster.sh: add support to CRI-O

71b2c67

By exporting the CONTAINER_RUNTIME=crio variable, kcli will create
a k8s cluster configured CRI-O:

$ export CONTAINER_RUNTIME=crio
$ ./src/cloud-api-adaptor/libvirt/kcli_cluster.sh create

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          test/e2e: new container_runtime property for libvirt

1a91d70

Commit a0247ae introduced a new parameter (CONTAINER_RUNTIME) for docker
provider, allowing users to specify the container runtime used. Some tests will
take decisions based on that property, for example, whether nydus snapshotter
messages should be inspected or not. This added the handler for that property
for libvirt, so that peer-pods be tested with cri-o too.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: add e2e job to test with CRI-O

ec93510

Added a new container_runtime matrix column to generate
one job for each runtime: containerd and crio.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: allow the e2e crio job to fail

b8dd9f3

Let's keep it running for a while on CI, once it's stable we can
remove the continue-on-error.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm mentioned this issue

CI: run e2e tests for libvirt with CRI-O #2068

Merged

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          libvirt/kcli_cluster.sh: add support to CRI-O

2af368a

By exporting the CONTAINER_RUNTIME=crio variable, kcli will create
a k8s cluster configured CRI-O:

$ export CONTAINER_RUNTIME=crio
$ ./src/cloud-api-adaptor/libvirt/kcli_cluster.sh create

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          test/e2e: new container_runtime property for libvirt

710b05b

Commit a0247ae introduced a new parameter (CONTAINER_RUNTIME) for docker
provider, allowing users to specify the container runtime used. Some tests will
take decisions based on that property, for example, whether nydus snapshotter
messages should be inspected or not. Likewise, this added the handler for that
property for libvirt, so allowing to test with cri-o too.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: add e2e job to test with CRI-O

45629a0

Added a new container_runtime matrix column to generate
one job for each runtime: containerd and crio.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: allow the e2e crio job to fail

a33d3aa

Let's keep it running for a while on CI, once it's stable we can
remove the continue-on-error.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm mentioned this issue

test/e2e: track the libvirt tests failing with cri-o #2100

Open

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          libvirt/kcli_cluster.sh: add support to CRI-O

c179d8c

By exporting the CONTAINER_RUNTIME=crio variable, kcli will create
a k8s cluster configured CRI-O:

$ export CONTAINER_RUNTIME=crio
$ ./src/cloud-api-adaptor/libvirt/kcli_cluster.sh create

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          test/e2e: new container_runtime property for libvirt

dc89125

Commit a0247ae introduced a new parameter (CONTAINER_RUNTIME) for docker
provider, allowing users to specify the container runtime used. Some tests will
take decisions based on that property, for example, whether nydus snapshotter
messages should be inspected or not. Likewise, this added the handler for that
property for libvirt, so allowing to test with cri-o too.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: add e2e job to test with CRI-O

8a12b1a

Added a new container_runtime matrix column to generate
one job for each runtime: containerd and crio.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: allow the e2e crio job to fail

Let's keep it running for a while on CI, once it's stable we can
remove the continue-on-error.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          libvirt/kcli_cluster.sh: add support to CRI-O

c612c3b

By exporting the CONTAINER_RUNTIME=crio variable, kcli will create
a k8s cluster configured CRI-O:

$ export CONTAINER_RUNTIME=crio
$ ./src/cloud-api-adaptor/libvirt/kcli_cluster.sh create

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          test/e2e: new container_runtime property for libvirt

7edea5e

Commit a0247ae introduced a new parameter (CONTAINER_RUNTIME) for docker
provider, allowing users to specify the container runtime used. Some tests will
take decisions based on that property, for example, whether nydus snapshotter
messages should be inspected or not. Likewise, this added the handler for that
property for libvirt, so allowing to test with cri-o too.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: add e2e job to test with CRI-O

65bbaa4

Added a new container_runtime matrix column to generate
one job for each runtime: containerd and crio.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: allow the e2e crio job to fail

48a87d1

Let's keep it running for a while on CI, once it's stable we can
remove the continue-on-error.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          libvirt/kcli_cluster.sh: add support to CRI-O

a9f7e19

By exporting the CONTAINER_RUNTIME=crio variable, kcli will create
a k8s cluster configured CRI-O:

$ export CONTAINER_RUNTIME=crio
$ ./src/cloud-api-adaptor/libvirt/kcli_cluster.sh create

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          test/e2e: new container_runtime property for libvirt

Commit a0247ae introduced a new parameter (CONTAINER_RUNTIME) for docker
provider, allowing users to specify the container runtime used. Some tests will
take decisions based on that property, for example, whether nydus snapshotter
messages should be inspected or not. Likewise, this added the handler for that
property for libvirt, so allowing to test with cri-o too.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: add e2e job to test with CRI-O

37a847c

Added a new container_runtime matrix column to generate
one job for each runtime: containerd and crio.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: allow the e2e crio job to fail

bb04542

Let's keep it running for a while on CI, once it's stable we can
remove the continue-on-error.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          libvirt/kcli_cluster.sh: add support to CRI-O

ca70ca3

By exporting the CONTAINER_RUNTIME=crio variable, kcli will create
a k8s cluster configured CRI-O:

$ export CONTAINER_RUNTIME=crio
$ ./src/cloud-api-adaptor/libvirt/kcli_cluster.sh create

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          test/e2e: new container_runtime property for libvirt

4ae0e03

Commit a0247ae introduced a new parameter (CONTAINER_RUNTIME) for docker
provider, allowing users to specify the container runtime used. Some tests will
take decisions based on that property, for example, whether nydus snapshotter
messages should be inspected or not. Likewise, this added the handler for that
property for libvirt, so allowing to test with cri-o too.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: add e2e job to test with CRI-O

dbabcf0

Added a new container_runtime matrix column to generate
one job for each runtime: containerd and crio.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit to wainersm/cc-cloud-api-adaptor that referenced this issue


          workflows/libvirt: allow the e2e crio job to fail

eb97e21

Let's keep it running for a while on CI, once it's stable we can
remove the continue-on-error.

Fixes confidential-containers#1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm closed this as completed in #2068

wainersm added a commit that referenced this issue


          libvirt/kcli_cluster.sh: add support to CRI-O

2494ee0

By exporting the CONTAINER_RUNTIME=crio variable, kcli will create
a k8s cluster configured CRI-O:

$ export CONTAINER_RUNTIME=crio
$ ./src/cloud-api-adaptor/libvirt/kcli_cluster.sh create

Fixes #1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit that referenced this issue


          test/e2e: new container_runtime property for libvirt

c7da95b

Commit a0247ae introduced a new parameter (CONTAINER_RUNTIME) for docker
provider, allowing users to specify the container runtime used. Some tests will
take decisions based on that property, for example, whether nydus snapshotter
messages should be inspected or not. Likewise, this added the handler for that
property for libvirt, so allowing to test with cri-o too.

Fixes #1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit that referenced this issue


          workflows/libvirt: add e2e job to test with CRI-O

2d6fccb

Added a new container_runtime matrix column to generate
one job for each runtime: containerd and crio.

Fixes #1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

wainersm added a commit that referenced this issue


          workflows/libvirt: allow the e2e crio job to fail

d1967b9

Let's keep it running for a while on CI, once it's stable we can
remove the continue-on-error.

Fixes #1981
Signed-off-by: Wainer dos Santos Moschetta <[email protected]>

github-project-automation bot moved this from 🆕 New to ✅ Done in CoCo Releases

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment