From 7fc11de9eebef60ef768b33d9183eb8182d99989 Mon Sep 17 00:00:00 2001 From: Poornima Y N Date: Wed, 20 Mar 2024 07:15:52 -0400 Subject: [PATCH] Modify Host Personality for attribute max_cpu_mhz_configured Max_cpu_mhz_personality is the attribute of the host which can be configured in host where turbo freq is enabled.In case of host whose role is both controller and worker, the personality for the attribute was not taken care to include such scenario. Made the changes in the sysinv conductor to update the host personalities based on the function that node operates, which handles the scenario when the host acts as both controller and worker node. TEST PLAN: PASS: Build and deploy ISO on Simplex PASS: Check whether the max cpu freq set on a simplex Below are the commands: system host-show | grep is_max_cpu_configurable system service-parameter-list --name cpu_max_freq_min_percentage system service-parameter-modify platform config cpu_max_freq_min_percentage=<> system host-update max_cpu_mhz_configured= After above commands check whether cpu is set using below command: sudo turbostat Closes-Bug: 2058476 Change-Id: I08a5d1400834afca6a0eeaaa8813ac8d71a9db15 Signed-off-by: Poornima Y N --- sysinv/sysinv/sysinv/sysinv/conductor/manager.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/sysinv/sysinv/sysinv/sysinv/conductor/manager.py b/sysinv/sysinv/sysinv/sysinv/conductor/manager.py index 0257166089..10da2051db 100644 --- a/sysinv/sysinv/sysinv/sysinv/conductor/manager.py +++ b/sysinv/sysinv/sysinv/sysinv/conductor/manager.py @@ -15789,7 +15789,11 @@ class ConductorManager(service.PeriodicService): labels = self.dbapi.label_get_by_host(host['uuid']) if not cutils.has_power_management_enabled(labels): - personalities = [constants.WORKER] + if cutils.is_aio_system(self.dbapi): + personalities = [constants.WORKER, + constants.CONTROLLER] + else: + personalities = [constants.WORKER] config_uuid = self._config_update_hosts(context, personalities,