From 42d18a8e047add12a1ba256dd749884ed6439afd Mon Sep 17 00:00:00 2001 From: Paige Patton <64206430+paigerube14@users.noreply.github.com> Date: Mon, 15 Dec 2025 10:04:35 -0500 Subject: [PATCH] adding fail scenario if unrecovered kubevirt vm killing (#994) Signed-off-by: Paige Patton --- .../kubevirt_vm_outage/kubevirt_vm_outage_scenario_plugin.py | 3 ++- tests/test_kubevirt_vm_outage.py | 2 +- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/krkn/scenario_plugins/kubevirt_vm_outage/kubevirt_vm_outage_scenario_plugin.py b/krkn/scenario_plugins/kubevirt_vm_outage/kubevirt_vm_outage_scenario_plugin.py index 58a31364..f9a289f8 100644 --- a/krkn/scenario_plugins/kubevirt_vm_outage/kubevirt_vm_outage_scenario_plugin.py +++ b/krkn/scenario_plugins/kubevirt_vm_outage/kubevirt_vm_outage_scenario_plugin.py @@ -55,7 +55,8 @@ class KubevirtVmOutageScenarioPlugin(AbstractScenarioPlugin): pods_status.merge(single_pods_status) scenario_telemetry.affected_pods = pods_status - + if len(scenario_telemetry.affected_pods.unrecovered) > 0: + return 1 return 0 except Exception as e: logging.error(f"KubeVirt VM Outage scenario failed: {e}") diff --git a/tests/test_kubevirt_vm_outage.py b/tests/test_kubevirt_vm_outage.py index cded8b51..4d884a14 100644 --- a/tests/test_kubevirt_vm_outage.py +++ b/tests/test_kubevirt_vm_outage.py @@ -147,7 +147,7 @@ class TestKubevirtVmOutageScenarioPlugin(unittest.TestCase): with patch("builtins.open", unittest.mock.mock_open(read_data=yaml.dump(self.config))): result = self.plugin.run("test-uuid", self.scenario_file, {}, self.telemetry, self.scenario_telemetry) - self.assertEqual(result, 0) + self.assertEqual(result, 1) mock_delete.assert_called_once_with("test-vm", "default", False) mock_wait.assert_not_called()