Merge pull request #74 from criteo/optional-phys-context

bmc: make PhysicalContext and PowerConsumedWatts optional

Author: ErwanAliasr1

hwbench/environment/vendors/bmc.py

@@ -152,7 +152,7 @@ def read_thermals(
 
                 super().add_monitoring_value(
                     cast(dict[str, dict[str, MonitorMetric]], thermals),
-                    t["PhysicalContext"],
+                    t.get("PhysicalContext", "UnknownPhysicalContext"),
                     Temperature(name),
                     t["Name"],
                     t["ReadingCelsius"],
@@ -196,7 +196,7 @@ def read_power_consumption(
         if str(PowerContext.BMC) not in power_consumption:
             power_consumption[str(PowerContext.BMC)] = {str(PowerCategories.SERVER): Power(str(PowerCategories.SERVER))}  # type: ignore[no-redef]
 
-        power = self.get_power().get("PowerControl", [{"PowerConsumedWatts": None}])[0]["PowerConsumedWatts"]
+        power = self.get_power().get("PowerControl", [{"PowerConsumedWatts": None}])[0].get("PowerConsumedWatts", None)
         if power:
             power_consumption[str(PowerContext.BMC)][str(PowerCategories.SERVER)].add(power)
         return power_consumption