starlingx
/
distcloud
Code Issues Proposed changes

Revert "Add the --upload-only option to dcmanager upgrade_strategy" 

This reverts commit 952cbeb7e4.

Reason for revert: This change will no longer be necessary

Change-Id: I9c3f3da61ed6933e77be2d8fc9d1a67392af41e0
This commit is contained in:
Christopher de Oliveira Souza 2024-02-06 16:57:21 +00:00
parent eae7aed572
commit 8898524ae8
4 changed files with 11 additions and 56 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
distributedcloud/dcmanager/orchestrator/states/software/cache/cache_specifications.py vendored
View File

@ -4,8 +4,6 @@
# SPDX-License-Identifier: Apache-2.0
#
from dcmanager.common import consts
from dcmanager.common import context
from dcmanager.common import utils
from dcmanager.orchestrator.states.software.cache import clients
from dcmanager.orchestrator.states.software.cache.clients import \
CLIENT_READ_EXCEPTIONS
@ -50,7 +48,6 @@ class CacheSpecification(object):
REGION_ONE_LICENSE_CACHE_TYPE = 'RegionOne system license'
REGION_ONE_SYSTEM_INFO_CACHE_TYPE = 'RegionOne system info'
REGION_ONE_RELEASE_USM_CACHE_TYPE = 'RegionOne release usm'
STRATEGY_EXTRA_ARGS_CACHE_TYPE = 'Strategy extra args'
"""Cache specifications"""
@ -71,26 +68,18 @@ REGION_ONE_RELEASE_USM_CACHE_SPECIFICATION = CacheSpecification(
{'state'}
)
STRATEGY_EXTRA_ARGS_CACHE_SPECIFICATION = CacheSpecification(
lambda: utils.get_sw_update_strategy_extra_args(context.get_admin_context()).get(
consts.EXTRA_ARGS_UPLOAD_ONLY
)
)
# Map each expected operation type to its required cache types
CACHE_TYPES_BY_OPERATION_TYPE = {
consts.SW_UPDATE_TYPE_SOFTWARE: {REGION_ONE_LICENSE_CACHE_TYPE,
REGION_ONE_SYSTEM_INFO_CACHE_TYPE,
REGION_ONE_RELEASE_USM_CACHE_TYPE,
STRATEGY_EXTRA_ARGS_CACHE_TYPE}
REGION_ONE_RELEASE_USM_CACHE_TYPE}
}
# Map each cache type to its corresponding cache specification
SPECIFICATION_BY_CACHE_TYPE = {
REGION_ONE_LICENSE_CACHE_TYPE: REGION_ONE_LICENSE_CACHE_SPECIFICATION,
REGION_ONE_SYSTEM_INFO_CACHE_TYPE: REGION_ONE_SYSTEM_INFO_CACHE_SPECIFICATION,
REGION_ONE_RELEASE_USM_CACHE_TYPE: REGION_ONE_RELEASE_USM_CACHE_SPECIFICATION,
STRATEGY_EXTRA_ARGS_CACHE_TYPE: STRATEGY_EXTRA_ARGS_CACHE_SPECIFICATION
REGION_ONE_RELEASE_USM_CACHE_TYPE: REGION_ONE_RELEASE_USM_CACHE_SPECIFICATION
}

15
distributedcloud/dcmanager/orchestrator/states/software/upload.py
View File

@ -14,8 +14,6 @@ from dcmanager.common import utils
from dcmanager.orchestrator.states.base import BaseState
from dcmanager.orchestrator.states.software.cache.cache_specifications import \
REGION_ONE_RELEASE_USM_CACHE_TYPE
from dcmanager.orchestrator.states.software.cache.cache_specifications import \
STRATEGY_EXTRA_ARGS_CACHE_TYPE
# Max time: 30 minutes = 180 queries x 10 seconds between
DEFAULT_MAX_QUERIES = 180
@ -45,6 +43,7 @@ class UploadState(BaseState):
def perform_state_action(self, strategy_step):
"""Upload releases in this subcloud"""
self.info_log(strategy_step, "Uploading releases")
regionone_releases = self._read_from_cache(REGION_ONE_RELEASE_USM_CACHE_TYPE)
applied_releases_ids = list()
for release_id in regionone_releases:
@ -53,8 +52,6 @@ class UploadState(BaseState):
software_v1.COMMITTED]:
applied_releases_ids.append(release_id)
upload_only = self._read_from_cache(STRATEGY_EXTRA_ARGS_CACHE_TYPE)
# Retrieve all subcloud releases
try:
subcloud_releases = self.get_software_client(
@ -155,14 +152,4 @@ class UploadState(BaseState):
self.error_log(strategy_step, message)
raise Exception(message)
if upload_only:
self.info_log(
strategy_step,
(
f"{consts.EXTRA_ARGS_UPLOAD_ONLY} option enabled, skipping"
f" forward to state:({consts.STRATEGY_STATE_COMPLETE})"
)
)
return consts.STRATEGY_STATE_COMPLETE
return self.next_state

3
distributedcloud/dcmanager/orchestrator/sw_update_manager.py
View File

@ -450,8 +450,7 @@ class SwUpdateManager(manager.Manager):
consts.PRESTAGE_SOFTWARE_VERSION:
software_version if software_version else SW_VERSION
}
elif (strategy_type == consts.SW_UPDATE_TYPE_PATCH or
strategy_type == consts.SW_UPDATE_TYPE_UPGRADE):
elif strategy_type == consts.SW_UPDATE_TYPE_PATCH:
upload_only_str = payload.get(consts.EXTRA_ARGS_UPLOAD_ONLY)
upload_only_bool = True if upload_only_str == 'true' else False
extra_args = {consts.EXTRA_ARGS_UPLOAD_ONLY: upload_only_bool}

34
distributedcloud/dcmanager/tests/unit/orchestrator/states/software/test_upload.py
View File

@ -74,8 +74,9 @@ class TestUploadState(TestSoftwareOrchestrator):
def test_software_upload_strategy_success(self, mock_is_file,
mock_read_from_cache):
"""Test software upload when the API call succeeds."""
mock_read_from_cache.side_effect = [REGION_ONE_RELEASES, False]
mock_read_from_cache.return_value = REGION_ONE_RELEASES
mock_is_file.return_value = True
self.software_client.query.side_effect = [SUBCLOUD_RELEASES]
# invoke the strategy state operation on the orch thread
@ -90,32 +91,11 @@ class TestUploadState(TestSoftwareOrchestrator):
self.assert_step_updated(self.strategy_step.subcloud_id,
self.on_success_state)
@mock.patch.object(UploadState, '_read_from_cache')
@mock.patch.object(os.path, 'isfile')
def test_software_upload_strategy_upload_only(self, mock_is_file,
mock_read_from_cache):
"""Test software upload when the API call succeeds."""
mock_read_from_cache.side_effect = [REGION_ONE_RELEASES, True]
mock_is_file.return_value = True
self.software_client.query.side_effect = [SUBCLOUD_RELEASES]
# invoke the strategy state operation on the orch thread
self.worker.perform_state_action(self.strategy_step)
self.software_client.upload.assert_called_once_with([
consts.RELEASE_VAULT_DIR + '/20.12/DC_20.12.3.patch',
consts.RELEASE_VAULT_DIR + '/20.12/DC_20.12.4.patch'
])
# On success, the state should transition to the next state
self.assert_step_updated(self.strategy_step.subcloud_id,
consts.SW_UPDATE_STATE_COMPLETE)
@mock.patch.object(UploadState, '_read_from_cache')
def test_software_upload_strategy_no_operation_required(self,
mock_read_from_cache):
"""Test software upload when no software operation is required."""
mock_read_from_cache.side_effect = [REGION_ONE_RELEASES, False]
mock_read_from_cache.return_value = REGION_ONE_RELEASES
self.software_client.query.side_effect = [REGION_ONE_RELEASES]
@ -133,7 +113,7 @@ class TestUploadState(TestSoftwareOrchestrator):
def test_software_upload_strategy_missing_sig(self, mock_is_dir, mock_listdir,
mock_read_from_cache):
"""Test software upload when release is missing signature"""
mock_read_from_cache.side_effect = [REGION_ONE_RELEASES_2, False]
mock_read_from_cache.return_value = REGION_ONE_RELEASES_2
mock_is_dir.return_value = True
mock_listdir.return_value = ["DC_22.12.0.iso"]
self.software_client.query.side_effect = [SUBCLOUD_RELEASES]
@ -153,7 +133,7 @@ class TestUploadState(TestSoftwareOrchestrator):
def test_software_upload_strategy_success_load(self, mock_is_dir, mock_listdir,
mock_read_from_cache):
"""Test software upload when the API call succeeds."""
mock_read_from_cache.side_effect = [REGION_ONE_RELEASES_2, False]
mock_read_from_cache.return_value = REGION_ONE_RELEASES_2
mock_is_dir.return_value = True
mock_listdir.return_value = ["DC_22.12.0.iso", "DC_22.12.0.sig"]
self.software_client.query.side_effect = [
@ -179,7 +159,7 @@ class TestUploadState(TestSoftwareOrchestrator):
mock_is_dir, mock_listdir,
mock_read_from_cache):
"""Test software upload when release is a prepatched iso."""
mock_read_from_cache.side_effect = [REGION_ONE_RELEASES_3, False]
mock_read_from_cache.return_value = REGION_ONE_RELEASES_3
mock_is_dir.return_value = True
mock_isfile.return_value = False
mock_listdir.return_value = ["DC_22.12.0.iso", "DC_22.12.0.sig"]
@ -205,7 +185,7 @@ class TestUploadState(TestSoftwareOrchestrator):
mock_is_dir, mock_listdir,
mock_read_from_cache):
"""Test software upload when both patch and load is uploaded."""
mock_read_from_cache.side_effect = [REGION_ONE_RELEASES_3, False]
mock_read_from_cache.return_value = REGION_ONE_RELEASES_3
mock_is_dir.return_value = True
mock_isfile.return_value = True
mock_listdir.return_value = ["DC_22.12.0.iso", "DC_22.12.0.sig"]