Added support to take max_proc_units as input from user.

In the previous implementation, we did not support taking the max_proc_units value from user input. Instead, we first calculated the stealable resources and used that value, with a default setting of max_proc_units=2. Now, we have added the ability to take max_proc_units as input from a configuration file. If the user does not provide this value, we fall back to the stealable resource count. Signed-off-by: Samir Mulani <[email protected]>
open-power · Sep 26, 2024 · cd2e2aa · cd2e2aa
1 parent 654c8b7
commit cd2e2aa
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/testcases/MachineConfig.py b/testcases/MachineConfig.py
@@ -250,10 +250,9 @@ def LparSetup(self, lpar_config=""):
             except AttributeError:
                 self.desired_proc_units = 2.0
             try:
-                self.max_proc_units = int(float(
-                    self.cv_HMC.get_available_proc_resources()[0])) + self.cv_HMC.get_stealable_proc_resources_lpar()
+                self.max_proc_units = float(conf.args.max_proc_units)
             except AttributeError:
-                self.max_proc_units = 2.0
+                self.max_proc_units = int(float(self.cv_HMC.get_available_proc_resources()[0])) + self.cv_HMC.get_stealable_proc_resources_lpar()
             try:
                 self.overcommit_ratio = int(conf.args.overcommit_ratio)
             except AttributeError: