feat: Retrieve positional info from GB200 chassis info (#16)

Author: ddejong-spec

src/model/oem/mod.rs

@@ -4,6 +4,7 @@ pub mod dell;
 pub mod hpe;
 pub mod lenovo;
 pub mod nvidia_dpu;
+pub mod nvidia_openbmc;
 pub mod nvidia_viking;
 pub mod supermicro;
 
@@ -24,5 +25,5 @@ pub struct SystemExtensions {
 #[derive(Debug, Serialize, Deserialize, Clone)]
 #[serde(rename_all = "PascalCase")]
 pub struct ChassisExtensions {
-    pub nvidia: Option<nvidia_dpu::Chassis>,
+    pub nvidia: Option<nvidia_openbmc::ChassisExtensions>,
 }

src/model/oem/nvidia_dpu.rs

@@ -63,32 +63,6 @@ impl fmt::Display for HostPrivilegeLevel {
     }
 }
 
-/// This OEM specific extension is mainly applicable for querying chassis information for the ERoT subsystem
-/// odata_type is always present regardless of the subsystem we are querying for (Bluefield_BMC, Bluefield_ERoT, or Card1)
-/// the remaining attributes are only present when querying the Bluefield_ERoT
-#[derive(Debug, Serialize, Deserialize, Clone)]
-#[serde(rename_all = "PascalCase")]
-pub struct Chassis {
-    #[serde(rename = "@odata.type")]
-    pub odata_type: String,
-    pub automatic_background_copy_enabled: Option<bool>,
-    pub background_copy_status: Option<BackgroundCopyStatus>,
-    pub inband_update_policy_enabled: Option<bool>,
-}
-
-#[derive(Debug, Serialize, Deserialize, Copy, Clone, Eq, PartialEq)]
-pub enum BackgroundCopyStatus {
-    InProgress,
-    Completed,
-    Pending,
-}
-
-impl fmt::Display for BackgroundCopyStatus {
-    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        fmt::Debug::fmt(self, f)
-    }
-}
-
 #[derive(Copy, Clone, PartialEq, Eq, Debug, Serialize, Deserialize)]
 #[serde(rename_all = "PascalCase")]
 pub enum NicMode {

src/model/oem/nvidia_openbmc.rs

@@ -0,0 +1,35 @@
+use std::fmt;
+
+use serde::{Deserialize, Serialize};
+
+/// This OEM specific extension is mainly applicable for querying chassis information for the ERoT subsystem
+/// odata_type is always present regardless of the subsystem we are querying for (Bluefield_BMC, Bluefield_ERoT, or Card1)
+/// the remaining attributes are only present when querying the Bluefield_ERoT
+/// Due to the indistinguishable names, this is used for DPUs, GB200, and potentially others; comments describe
+/// what platforms it may be expected on.
+#[derive(Debug, Serialize, Deserialize, Clone)]
+#[serde(rename_all = "PascalCase")]
+pub struct ChassisExtensions {
+    #[serde(rename = "@odata.type")]
+    pub odata_type: String,
+    pub automatic_background_copy_enabled: Option<bool>, // DPU
+    pub background_copy_status: Option<BackgroundCopyStatus>, // DPU
+    pub inband_update_policy_enabled: Option<bool>,      // DPU
+    pub chassis_physical_slot_number: Option<i32>,       // GB200
+    pub compute_tray_index: Option<i32>,                 // GB200
+    pub topology_id: Option<i32>,                        // GB200
+    pub revision_id: Option<i32>,                        // GB200
+}
+
+#[derive(Debug, Serialize, Deserialize, Copy, Clone, Eq, PartialEq)]
+pub enum BackgroundCopyStatus {
+    InProgress,
+    Completed,
+    Pending,
+}
+
+impl fmt::Display for BackgroundCopyStatus {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        fmt::Debug::fmt(self, f)
+    }
+}

tests/integration_test.rs

@@ -471,6 +471,14 @@ async fn run_integration_test(
 
         let firmware = redfish.get_firmware_for_component("ERoT_BMC_0").await;
         assert!(firmware.is_err());
+
+        let chassis_cbc0 = redfish.get_chassis("CBC_0").await.unwrap();
+        let vendor = chassis_cbc0.oem.unwrap();
+        let nvidia = vendor.nvidia.unwrap();
+        assert_eq!(nvidia.chassis_physical_slot_number.unwrap(), 1);
+        assert_eq!(nvidia.compute_tray_index.unwrap(), 3);
+        assert_eq!(nvidia.revision_id.unwrap(), 2);
+        assert_eq!(nvidia.topology_id.unwrap(), 4);
     }
 
     if vendor_dir == "dell" {

tests/mockups/nvidia_gb200/redfish/v1/Chassis/CBC_0/index.json

@@ -39,9 +39,9 @@
         "Nvidia": {
             "@odata.type": "#NvidiaChassis.v1_4_0.NvidiaCBCChassis",
             "ChassisPhysicalSlotNumber": 1,
-            "ComputeTrayIndex": 0,
+            "ComputeTrayIndex": 3,
             "RevisionId": 2,
-            "TopologyId": 0
+            "TopologyId": 4
         }
     },
     "PCIeDevices": {