From: dave kormann Date: Wed, 17 Jul 2019 19:17:18 +0000 (-0400) Subject: FIX: Set default IPMI privilege level X-Git-Url: https://gerrit.akraino.org/r/gitweb?p=ta%2Fcm-plugins.git;a=commitdiff_plain;h=87f9df933749d5467611ec113690d41e326cd92a FIX: Set default IPMI privilege level The previous changes to support setting the IPMI level did not set it as default in the hosts array, causing a failure in the ansible playbooks. This change adds a call to the configmanager to set a default privilege level for all hosts. signed-off-by: dave kormann depends-on: Ic21d5e47de75c783e8d0fd0b321624fc5eb84a0f Change-Id: I1e6e27a20aee42f858375bb184cec836e3cec7d1 --- diff --git a/recuserconfighandlers/rechosthandler/rechosthandler.py b/recuserconfighandlers/rechosthandler/rechosthandler.py index f3ec205..d9c95ce 100644 --- a/recuserconfighandlers/rechosthandler/rechosthandler.py +++ b/recuserconfighandlers/rechosthandler/rechosthandler.py @@ -18,21 +18,26 @@ from cmframework.apis import cmerror from cmdatahandlers.api import configerror """ -This plugin is used to handle REC specific host configs. Currently -its sole purpuse is to set default middleware reserved memory. +This plugin is used to handle REC specific host configs, including +setting default middleware reserved memory and populating default +IPMI privileges. """ class rechosthandler(cmuserconfig.CMUserConfigPlugin): default_middleware_reserved_memory = '4Gi' - + default_ipmi_priv_level = 'ADMINISTRATOR' + def __init__(self): super(rechosthandler, self).__init__() def handle(self, confman): - self._set_default_memory(confman) - - def _set_default_memory(self, confman): hostconf = confman.get_hosts_config_handler() + self._set_default_memory(hostconf) + self._set_default_ipmi_priv(hostconf) + + def _set_default_memory(self, hostconf): hostconf.set_default_reserved_memory_to_all_hosts(self.default_middleware_reserved_memory) + def _set_default_ipmi_priv(self, hostconf): + hostconf.set_default_ipmi_priv_level_to_all_hosts(self.default_ipmi_priv_level)