search

kata-containers / tests

Kata Containers tests, CI, and metrics
https://katacontainers.io/
Apache License 2.0
139 stars 196 forks source link

CC | Adapt to the containerd changes done on kata-containers repo #5773

Closed fidencio closed 1 year ago

fidencio commented 1 year ago

This will easily allow us to test against the forked and the upstream containerd.

As the tests had changed completely on main, and as the main branch should never ever see the forked containerd, we're doing this only for the CCv0 case.

fidencio commented 1 year ago

/test

fidencio commented 1 year ago

/test

stevenhorsman commented 1 year ago

Sorry, I forgot to point out that https://github.com/kata-containers/tests/blob/6d7723a1ec99e85f968f6333979dd830a570bad2/integration/containerd/cri/integration-tests.sh#L51 will also need updating

fidencio commented 1 year ago

Sorry, I forgot to point out that

https://github.com/kata-containers/tests/blob/6d7723a1ec99e85f968f6333979dd830a570bad2/integration/containerd/cri/integration-tests.sh#L51

will also need updating

Yeah, I just noticed with the test failing. At least we know the change works, and what will need to be updated.

Let me grow this PR (and the one in the Kata Containers) a little bit so we avoid having to wait for this CI to finish to start doing the other work.

fidencio commented 1 year ago

/test

fidencio commented 1 year ago

/test

fidencio commented 1 year ago

/test

fidencio commented 1 year ago

None of the changes here should directly impact TDX / SEV / SNP CIs, thus I'm going ahead and merging this one to avoid delaying even more v0.8.0 release.