From e86ebe4e11ed38418bf83ae9575b27af4e7f34c7 Mon Sep 17 00:00:00 2001 From: Luca Carrogu Date: Tue, 23 Jan 2024 15:51:45 +0100 Subject: [PATCH] Add assertions for scaling test Signed-off-by: Luca Carrogu --- .../tests/performance_tests/test_scaling.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/tests/integration-tests/tests/performance_tests/test_scaling.py b/tests/integration-tests/tests/performance_tests/test_scaling.py index 4af324f86b..97cbcccba2 100644 --- a/tests/integration-tests/tests/performance_tests/test_scaling.py +++ b/tests/integration-tests/tests/performance_tests/test_scaling.py @@ -193,7 +193,9 @@ def test_scaling_stress_test( request=request, scaling_target_time=_datetime_to_minute_granularity(scaling_target_time) + datetime.timedelta(minutes=1), ) - # Verify that there was no over-scaling - # assert_that(max(compute_nodes_time_series)).is_equal_to(scaling_target) + # Verify that there was no EC2 over-scaling assert_that(max(ec2_capacity_time_series)).is_equal_to(scaling_target) - # assert_that(compute_nodes_time_series[-1]).is_equal_to(0) + # Verify that there was no Slurm nodes over-scaling + assert_that(max(compute_nodes_time_series)).is_equal_to(scaling_target) + # Verify all Slurm nodes were removed on scale down + assert_that(compute_nodes_time_series[-1]).is_equal_to(0)