Skip to content

Commit

Permalink
ci: cri-containerd: Remove KILL_VMM_TEST env var
Browse files Browse the repository at this point in the history
We don't need the env var, we just need to restrict the test according
to the KATA_HYPERVISOR used, as right now it's very specifict to QEMU.

Signed-off-by: Fabiano Fidêncio <fabiano.fidencio@intel.com>
  • Loading branch information
fidencio committed Jul 21, 2023
1 parent bc4919f commit c363703
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions tests/integration/cri-containerd/integration-tests.sh
Expand Up @@ -21,7 +21,6 @@ export PATH="$PATH:/usr/local/sbin"
# Runtime to be used for testing
RUNTIME=${RUNTIME:-containerd-shim-kata-v2}
FACTORY_TEST=${FACTORY_TEST:-""}
KILL_VMM_TEST=${KILL_VMM_TEST:-""}
KATA_HYPERVISOR="${KATA_HYPERVISOR:-qemu}"
USE_DEVMAPPER="${USE_DEVMAPPER:-false}"
ARCH=$(uname -m)
Expand Down Expand Up @@ -74,7 +73,7 @@ ci_cleanup() {
sudo -E PATH=$PATH "$RUNTIME" factory destroy
fi

if [ -n "${KILL_VMM_TEST}" ] && [ -e "$default_containerd_config_backup" ]; then
if [ -e "$default_containerd_config_backup" ]; then
echo "restore containerd config"
sudo systemctl stop containerd
sudo cp "$default_containerd_config_backup" "$default_containerd_config"
Expand Down Expand Up @@ -210,8 +209,9 @@ testContainerStop() {
}

TestKilledVmmCleanup() {
if [ -z "${KILL_VMM_TEST}" ]; then
return
if [[ "${KATA_HYPERVISOR}" != "qemu" ]]; then
info "TestKilledVmmCleanup is skipped for ${KATA_HYPERVISOR}, only QEMU is currently tested"
return 0
fi

info "test killed vmm cleanup"
Expand Down

0 comments on commit c363703

Please sign in to comment.