From 3a2ef6f4722deb027bfa96253f5dd18321dadd80 Mon Sep 17 00:00:00 2001 From: Samuel Allan Date: Mon, 30 Sep 2024 14:11:23 +0930 Subject: [PATCH] Fix archive step nova-cloud-controller is in the control plane group, not data plane. The wrong group was being passed, so archive could not found the nova-cloud-controller units. Fixes: #547 --- cou/steps/plan.py | 2 +- tests/unit/steps/test_plan.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/cou/steps/plan.py b/cou/steps/plan.py index eb0e77a1..765d9996 100644 --- a/cou/steps/plan.py +++ b/cou/steps/plan.py @@ -582,7 +582,7 @@ def _get_archive_data_steps(analysis_result: Analysis, args: CLIargs) -> list[Pr description="Archive old database data on nova-cloud-controller", coro=archive( analysis_result.model, - analysis_result.apps_data_plane, + analysis_result.apps_control_plane, batch_size=args.archive_batch_size, ), ) diff --git a/tests/unit/steps/test_plan.py b/tests/unit/steps/test_plan.py index 5f251c21..4ecd5b7d 100644 --- a/tests/unit/steps/test_plan.py +++ b/tests/unit/steps/test_plan.py @@ -1111,7 +1111,7 @@ def test_get_archive_data_steps(cli_args, model): PreUpgradeStep( description="Archive old database data on nova-cloud-controller", coro=archive( - mock_analysis_result.model, mock_analysis_result.apps_data_plane, batch_size=2000 + mock_analysis_result.model, mock_analysis_result.apps_control_plane, batch_size=2000 ), ) ]