commit c48317f79eb841f1a4c8a159cbca15d328babd68
author Rohit PAI <ropai@nvidia.com> Mon Apr 28 22:06:00 2025 +0530
committer Rohit PAI <ropai@nvidia.com> Tue May 20 15:11:43 2025 +0000
tree 477725e9fd8a69d78f319139613a7ad9b55e0fa1
parent 5a3d934a15a7ba2e2c5269143d95df6581e793c6

Update AGGREGATION.md

1. Support multi SatMC configuration
2. Prefix generation from EM config

Change-Id: If03f45ea5d88b23bd7ba6e61577b1ab177eb7fa0
Signed-off-by: Rohit PAI <ropai@nvidia.com>

AGGREGATION.md

diff --git a/AGGREGATION.md b/AGGREGATION.md
index 9fa4b7e..9c1df34 100644
--- a/AGGREGATION.md
+++ b/AGGREGATION.md
@@ -42,9 +42,9 @@
 
 ## Satellite BMC Restrictions
 
-- Can only aggregate a single satellite BMC
 - HTTP only connection to satellite BMC
 - No authentication on satellite BMC
+- Each satellite BMC must have a unique "Name" property in its configuration
 
 ## Supported Resources
 
@@ -79,18 +79,19 @@
 
 Aggregated resources will have a prefix appended to their URIs in order to
 distinguish them from resources that are local to the aggregating BMC. The
-aggregation prefix is planned to be derived from the uuid of the aggregating
-BMC. This should result in the aggregation prefix being unique to each
-aggregating BMC.
+prefix is derived from the "Name" property specified in the satellite BMC's
+configuration in Entity Manager. This allows for meaningful and unique prefixes
+for each satellite BMC.
 
 The general URI format for aggregated resources is
 
 ```bash
-/redfish/v1/<resource_collection>/<prefix>_<resource_id>
+/redfish/v1/<resource_collection>/<name>_<resource_id>
 ```
 
-When support is added for aggregating more than one satellite BMC, then each
-satellite will be assigned its own unique satellite prefix.
+Where `<name>` is the value of the "Name" property from the satellite's
+configuration. For example, if a satellite is configured with `"Name": "sat0"`,
+its resources would appear as `/redfish/v1/Systems/sat0_system`.
 
 ### Aggregating Collections
 
@@ -122,10 +123,13 @@
       "@odata.id": "/redfish/v1/Systems/system"
     },
     {
-      "@odata.id": "/redfish/v1/Systems/5B247A_system"
+      "@odata.id": "/redfish/v1/Systems/sat0_system"
+    },
+    {
+      "@odata.id": "/redfish/v1/Systems/sat1_system"
     }
   ],
-  "Members@odata.count": 2,
+  "Members@odata.count": 3,
   "Name": "Computer System Collection"
 }
 ```
@@ -150,18 +154,16 @@
 5. Return the updated response from the satellite BMC
 
 ```bash
-curl -k -H "X-Auth-Token: $token" https://${bmc}/redfish/v1/Systems/5B247A_system
-{
-  "@odata.id": "/redfish/v1/Systems/5B247A_system",
-  ...
-  "LogServices": {
-    "@odata.id": "/redfish/v1/Systems/5B247A_system/LogServices"
-  },
-  "Memory": {
-    "@odata.id": "/redfish/v1/Systems/5B247A_system/Memory"
-  },
-  ...
-}
+curl -k -H "X-Auth-Token: $token" https://${bmc}/redfish/v1/Systems/sat0_system
+    {
+        "@odata.id" :
+            "/redfish/v1/Systems/sat0_system",
+            ... "LogServices" :
+            {"@odata.id" : "/redfish/v1/Systems/sat0_system/LogServices"},
+            "Memory" :
+            {"@odata.id" : "/redfish/v1/Systems/sat0_system/Memory"},
+            ...
+    }
 ```
 
 We consider any response to be a valid response code from the satellite BMC. The
@@ -179,9 +181,7 @@
 
 1. Add links to satellite only collections in responses from service root and
    other appropriate locations
-2. Generate aggregation prefix from uuid
-3. Aggregate more than one satellite BMC
-4. Support HTTPS connection to satellite BMC
+2. Support HTTPS connection to satellite BMC
 
 Support for the following items should be added as well. However, their
 implementation is not a requirement to being able to enable Redfish aggregation