| b84653e7 | 08-Jan-2026 |
Chandramohan Harkude <chandramohan.harkude@gmail.com> |
Fix Bmcweb crash due to HTTP2 connection reset
Changes added : handle dangling pointer in response completion handler
Problem : When high number of connections made to BMC and suddenly reset those
Fix Bmcweb crash due to HTTP2 connection reset
Changes added : handle dangling pointer in response completion handler
Problem : When high number of connections made to BMC and suddenly reset those connections, crash was observed in bmcweb.
Fix : Since the connection already torn but bmcweb trying to close those connections in response completion handler causing the crash. Added code to handle the closed or reset connections
Tested :
made around 200 connections and reset them immediately, repeated these steps around 15 times no crash observed
Change-Id: I03b5fd563275c4138a6eb004e09d277547af9692 Signed-off-by: Chandra Harkude <chandramohan.harkude@gmail.com>
show more ...
|
| d1348257 | 12-Dec-2025 |
Amithash Prasad <amithash@meta.com> |
Lower severity of HTTP logs from INFO to DEBUG
Couple of these logs are heavy hitters and fill up the journal especially with external entities constantly hitting the redfish endpoints. This change
Lower severity of HTTP logs from INFO to DEBUG
Couple of these logs are heavy hitters and fill up the journal especially with external entities constantly hitting the redfish endpoints. This change moves the largest two logs we saw (on a yosemite4 machine) from INFO to DEBUG.
Change-Id: Iac7337e3040e0348e41bba79c8200320e8ba12ef Signed-off-by: Amithash Prasad <amithash@meta.com>
show more ...
|
| b934d565 | 25-Nov-2025 |
Oliver Brewka <oliver.brewka@9elements.com> |
Renaming of eventlog variables and functions
Following a comment on [1], fix some of the variable and function names, to match redfish terms.
Tested: Unit tests pass. Validation succeeded.
[1] htt
Renaming of eventlog variables and functions
Following a comment on [1], fix some of the variable and function names, to match redfish terms.
Tested: Unit tests pass. Validation succeeded.
[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/83354/13
Change-Id: Ie52309126c798b17136be99fd4a3d6650721257e Signed-off-by: Oliver Brewka <oliver.brewka@9elements.com>
show more ...
|
| e777eff2 | 30-Dec-2025 |
Joey Berkovitz <jberkovitz@coreweave.com> |
Enable TCP keepalives for HTTP connections
This fix enables TCP keepalives at the OS layer. It also enables a 15 minute deadline timer at the bmcweb level when waiting on an idle HTTP keepalive conn
Enable TCP keepalives for HTTP connections
This fix enables TCP keepalives at the OS layer. It also enables a 15 minute deadline timer at the bmcweb level when waiting on an idle HTTP keepalive connection.
Tested: romulus image running bmcweb, start connections with keepalive, block incoming connections `iptables -P INPUT DROP`, validate that sockets eventually die and are tracked with keepalives `ss -nto`
Change-Id: I8f5040440348c060dae1d0516ec202a0e4dc349e Signed-off-by: Joey Berkovitz <joey@berkovitz.us>
show more ...
|
| 157cf910 | 23-Dec-2025 |
Bowei Yu <yubowei0982@phytium.com.cn> |
Improve the efficiency of the chassis
Removed redundant inner for loop of chassis.
Tested: I watched webui still see UUID and Location Code in chassis.
Change-Id: I4abd5bf7a46eb997ade6537c198cccde
Improve the efficiency of the chassis
Removed redundant inner for loop of chassis.
Tested: I watched webui still see UUID and Location Code in chassis.
Change-Id: I4abd5bf7a46eb997ade6537c198cccde1abf3196 Signed-off-by: Bowei Yu <yubowei0982@phytium.com.cn>
show more ...
|
| 96a194c4 | 18-Aug-2025 |
Ed Tanous <ed@tanous.net> |
Implement modernize-use-ranges
This tidy check can transform code to use std::ranges. Enable the check, apply the fixes it proposes.
Tested: Redfish service validator passes in qemu
Change-Id: I3
Implement modernize-use-ranges
This tidy check can transform code to use std::ranges. Enable the check, apply the fixes it proposes.
Tested: Redfish service validator passes in qemu
Change-Id: I3f21b27d3d30277f71b9c8a2c584a22bc16865e9 Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
| 9ad2c6f4 | 22-Oct-2025 |
Ed Tanous <etanous@nvidia.com> |
Update to boost 1.89
Update the subtree boost to 1.89; Keep the "required" version at 1.88 because that's currently what yocto uses. Now that we don't need old versions, the branch for boost 1.84
Update to boost 1.89
Update the subtree boost to 1.89; Keep the "required" version at 1.88 because that's currently what yocto uses. Now that we don't need old versions, the branch for boost 1.84 support is removed, and we can universally pull in boost::process as a library.
Additionally boost::core::string_view has defined a std::formatter nearly identical to what was done there, so there's now a conflict. Add a version check and shift to the boost provided formatter when it's available.
Tested: Code builds out of tree correctly.
Change-Id: I15a10da084da8f9d9460781b16a0fdc92987fc9a Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
| e32edd47 | 20-Nov-2025 |
Janet Adkins <janeta@us.ibm.com> |
Sensors: Reorganize objectPropertiesToJson for clarity
A review comment for a different commit noted the length of the objectPropertiesToJson() method. [1] Here I am restructuring the method to redu
Sensors: Reorganize objectPropertiesToJson for clarity
A review comment for a different commit noted the length of the objectPropertiesToJson() method. [1] Here I am restructuring the method to reduce its complexity.
The restructure factors out different sections of the work: - Filling in the basic identity of the sensor. This is split into two different methods. One for handling through the Redfish Sensors path and the others for the deprecated Redfish Power/Thermal paths. (Identity includes the Name/Id and type information for the sensor. - Common function for filling in Sensor status. - Mapping other available D-Bus properties to their Redfish property name.
[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/85103/comment/621fe109_9157ec50/
Tested: Using hardware simulator - Enabled redfish-allow-deprecated-power-thermal and confirmed response same before and after change for: - /redfish/v1/Chassis/chassis/Sensors - /redfish/v1/Chassis/chassis/Thermal - /redfish/v1/Chassis/chassis/Power - /redfish/v1/Chassis/chassis/ThermalSubsystem/ThermalMetrics - /redfish/v1/Chassis/chassis/EnvironmentMetrics - Each Member under /redfish/v1/Chassis/chassis/Sensors/ (i.e. /redfish/v1/Chassis/chassis/Sensors/{})
Change-Id: I1524c5c3b1f98a95dc2ed82d395897cd5f8af7d2 Signed-off-by: Janet Adkins <janeta@us.ibm.com>
show more ...
|
| aa0368ae | 05-Dec-2025 |
Ramya Sivakumar <sramya@ami.com> |
Removed unused header in redfish-core/include
Remove unused includes and include boost/circular_buffer/base.hpp in event_service_manager and event_matches_filter
Tested: Code compiles.
Change-Id:
Removed unused header in redfish-core/include
Remove unused includes and include boost/circular_buffer/base.hpp in event_service_manager and event_matches_filter
Tested: Code compiles.
Change-Id: Ib90374b822560c063f3047037bdb87f36685f48c Signed-off-by: Ramya Sivakumar <sramya@ami.com>
show more ...
|
| c583f900 | 01-Dec-2025 |
Igor Kanyuka <ifelmail@gmail.com> |
Expose Chassis state on Transitioning
If chassis is in one of transitioning states, Redfish interface does not expose PowerState and State.Status. This causes tests failure and users cannot determin
Expose Chassis state on Transitioning
If chassis is in one of transitioning states, Redfish interface does not expose PowerState and State.Status. This causes tests failure and users cannot determine the actual state using Redfish. Expose PowerState and State.Status for TransitioningToOff and TransitioningToOn to fix this.
Tested: Robot tests testing power operations now pass, before when the D-Bus state was TransitioningToOff or TransitioningToOn they would error because they could not find attributes they need. Also, ran rf_service_validator against BMC with D-Bus state set to TransitioningToOff and made sure it succeeded.
Change-Id: I9c446a107bac6a69e962ef81de82920c9cce21ae Signed-off-by: Igor Kanyuka <ifelmail@gmail.com>
show more ...
|
| d32aff8d | 23-Sep-2025 |
Myung Bae <myungbae@us.ibm.com> |
Add LocationIndicatorActive for Assembly
Implement LocationIndicatorActive for Assembly schema to set and get the status of the location LED. A client uses the `LocationIndicatorActive` property to
Add LocationIndicatorActive for Assembly
Implement LocationIndicatorActive for Assembly schema to set and get the status of the location LED. A client uses the `LocationIndicatorActive` property to physically identify or locate the assembly.
The assembly is an array of AssemblyData [1], and the element of the array can be patched as explained in [2].
``` { "Assemblies": [ {}, {}, { "LocationIndicatorActive": true }, {} ] } ```
Tested: - Validator passes. - 1. Get LocationIndicatorActive ``` curl -k -H "X-Auth-Token: $token" -X GET https://${bmc}/redfish/v1/Chassis/chassis/Assembly { "@odata.id": "/redfish/v1/Chassis/chassis/Assembly", "@odata.type": "#Assembly.v1_6_0.Assembly", "Assemblies": [ { "@odata.id": "/redfish/v1/Chassis/chassis/Assembly#/Assemblies/0", "@odata.type": "#Assembly.v1_6_0.AssemblyData", "LocationIndicatorActive": false, "MemberId": "0",
...
}, { "@odata.id": "/redfish/v1/Chassis/chassis/Assembly#/Assemblies/1", "@odata.type": "#Assembly.v1_6_0.AssemblyData", "LocationIndicatorActive": false, "MemberId": "1",
...
} ], "Assemblies@odata.count": 2, "Id": "Assembly", "Name": "Assembly Collection" } ```
2. Set LocationIndicatorActive to true ``` curl -k -H "X-Auth-Token: $token" -H "Content-Type: application/json" \ -X PATCH -d '{"Assemblies":[{"LocationIndicatorActive":true},{}]}' \ https://${bmc}/redfish/v1/Chassis/chassis/Assembly ```
Then we will see the location LED lit up, and the LocationIndicatorActive value becomes true.
``` curl -k -H "X-Auth-Token: $token" -X GET https://${bmc}/redfish/v1/Chassis/chassis/Assembly { "@odata.id": "/redfish/v1/Chassis/chassis/Assembly", "@odata.type": "#Assembly.v1_6_0.Assembly", "Assemblies": [ { "@odata.id": "/redfish/v1/Chassis/chassis/Assembly#/Assemblies/0", "@odata.type": "#Assembly.v1_6_0.AssemblyData", "LocationIndicatorActive": true, "MemberId": "0",
...
}, { "@odata.id": "/redfish/v1/Chassis/chassis/Assembly#/Assemblies/1", "@odata.type": "#Assembly.v1_6_0.AssemblyData", "LocationIndicatorActive": false, "MemberId": "1",
...
} ], "Assemblies@odata.count": 2, "Id": "Assembly", "Name": "Assembly Collection" } ```
If the input array size is different from the existing assemblies, it will cause an error like
``` curl -k -H "X-Auth-Token: $token" -H "Content-Type: application/json" \ -X PATCH -d '{"Assemblies":[{},{"LocationIndicatorActive":true},{}]}' \ https://${bmc}/redfish/v1/Chassis/chassis/Assembly { "error": { "@Message.ExtendedInfo": [ { "@odata.type": "#Message.v1_1_1.Message", "Message": "The array provided for property Assemblies exceeds the size limit 2.", "MessageArgs": [ "Assemblies", "2" ], "MessageId": "Base.1.19.ArraySizeTooLong", "MessageSeverity": "Warning", "Resolution": "Resubmit the request with an appropriate array size." } ], "code": "Base.1.19.ArraySizeTooLong", "message": "The array provided for property Assemblies exceeds the size limit 2." } }% ```
[1] https://redfish.dmtf.org/schemas/v1/Assembly.v1_6_0.json [2] https://redfishforum.com/thread/437/patch-individual-items-array-objects
Change-Id: Ic2e87f5daeb7ebed161654bb54ac29e7d5daa482 Signed-off-by: Myung Bae <myungbae@us.ibm.com>
show more ...
|
| 1aa94df4 | 31-Jul-2025 |
Harshit Aghera <haghera@nvidia.com> |
sensor_utils: Add PeakReading property
Add support for PeakReading and PeakReadingTime for sensors. This enhancement allows sensor readings to include max observed value information in the Redfish A
sensor_utils: Add PeakReading property
Add support for PeakReading and PeakReadingTime for sensors. This enhancement allows sensor readings to include max observed value information in the Redfish API, along with timestamp. It uses PDI xyz.openbmc_project.Telemetry.Report. Property PeakReading is added if OperationType in PDI property ReadingParameters is set to Maximum.
Current Limitation - The ResetMetrics action is currently not supported for sensor URIs. As a result, the ability to clear PeakReading values for GPU Power Sensors has not been implemented.
Future Consideration - If ResetMetrics action support is added in the future, the corresponding functionality will also need to be implemented in the dbus-sensor application to ensure full compatibility.
Schema: https://redfish.dmtf.org/schemas/v1/Sensor.v1_2_0.yaml (PeakReading)
Backend implementation for reference: https://gerrit.openbmc.org/c/openbmc/dbus-sensors/+/82479
Tested: Build an image for nvl32-obmc machine with the following patches cherry picked.
https://gerrit.openbmc.org/c/openbmc/openbmc/+/85490 https://gerrit.openbmc.org/c/openbmc/bmcweb/+/82449.
The patch cherry-picks the following patches that are currently under review.
``` 1. device tree https://lore.kernel.org/all/aRbLqH8pLWCQryhu@molberding.nvidia.com/ 2. mctpd patches https://github.com/CodeConstruct/mctp/pull/85 3. u-boot changes https://lore.kernel.org/openbmc/20251121-msx4-v1-0-fc0118b666c1@nvidia.com/T/#t 4. kernel changes as specified in the openbmc patch (for espi) 5. entity-manager changes https://gerrit.openbmc.org/c/openbmc/entity-manager/+/85455 6. platform-init changes https://gerrit.openbmc.org/c/openbmc/platform-init/+/85456 7. spi changes https://lore.kernel.org/all/20251121-w25q01jv_fixup-v1-1-3d175050db73@nvidia.com/ ```
``` > curl -s -k -u 'root:0penBmc' https://10.137.203.137/redfish/v1/Chassis/NVIDIA_GB200_1/Sensors/power_NVIDIA_GB200_GPU_0_Power_0 { "@odata.id": "/redfish/v1/Chassis/NVIDIA_GB200_1/Sensors/power_NVIDIA_GB200_GPU_0_Power_0", "@odata.type": "#Sensor.v1_2_0.Sensor", "Id": "power_NVIDIA_GB200_GPU_0_Power_0", "Name": "NVIDIA GB200 GPU 0 Power 0", "PeakReading": 52.671, "PeakReadingTime": 0, "Reading": 27.214, "ReadingRangeMax": 5000.0, "ReadingRangeMin": 0.0, "ReadingType": "Power", "ReadingUnits": "W", "Status": { "Health": "OK", "State": "Enabled" } }% ````
Change-Id: I8c1ab6ce85f31419db4a1d931bf99722d24afbd7 Signed-off-by: Harshit Aghera <haghera@nvidia.com>
show more ...
|
| 57d41fed | 13-Nov-2025 |
Janet Adkins <janeta@us.ibm.com> |
Fans: Create utility class for fans
Support for fans from separate routes of ThermalSubsystem and EnvironmentMetrics will need the same utility functions. By moving these shared functions into singl
Fans: Create utility class for fans
Support for fans from separate routes of ThermalSubsystem and EnvironmentMetrics will need the same utility functions. By moving these shared functions into single file will help with build time by not needing to included full fan support in all of the related locations.
Initially only the getFanPaths() is moved. Expectation is more functions will be added as the expanded support is added.
Tested: - Compiles - Verified ThermalSubsystem/Fans output unchanged
Change-Id: I22c22bdf38155e93aa13e259dd8c904a977f8a07 Signed-off-by: Janet Adkins <janeta@us.ibm.com>
show more ...
|
| 456512bc | 02-Dec-2025 |
Chandramohan Harkude <chandramohan.harkude@gmail.com> |
Fix RFA aggregation issue
Aggregation was failing because of method and target URI missing in HTTP request
This was a regression from 897e4c80f35b5bd963923f5794a7d3b229dba306
HTTP/1.1 Accept: appl
Fix RFA aggregation issue
Aggregation was failing because of method and target URI missing in HTTP request
This was a regression from 897e4c80f35b5bd963923f5794a7d3b229dba306
HTTP/1.1 Accept: application/json, application/octet-stream Host: 172.31.13.251
Fix : Added the Target URI and method from original request
Testing :
``` curl -k -u root:0penBmc -X GET https://${BMC_IP}/redfish/v1/Chassis
Returned items from both BMC and Satellite BMC ``` Change-Id: Ib446adfeb94d0cce3a272fc39ee5cc60ce3df7f8 Signed-off-by: Chandramohan Harkude <chandramohan.harkude@gmail.com> Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
| c2f428f5 | 17-Oct-2025 |
Oliver Brewka <oliver.brewka@9elements.com> |
Add remaining functions to systems_utils namespace
Initially, systems_utils.hpp has not defined its own namespace, all functions were under the redfish namespace. Patch 82078 [1] introduced systems_
Add remaining functions to systems_utils namespace
Initially, systems_utils.hpp has not defined its own namespace, all functions were under the redfish namespace. Patch 82078 [1] introduced systems_utils namespace to the header for new functionality.
This patch adds all functions, that haven't been part of the new namespace to systems_utils namespace and updates all call sites accordingly.
[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/82078
Tested: Code compiles.
Change-Id: Ia6ac3edd36aacc22cbf1a6adae4e2b264932b43c Signed-off-by: Oliver Brewka <oliver.brewka@9elements.com>
show more ...
|
| 32ea63d3 | 03-Sep-2025 |
Ed Tanous <etanous@nvidia.com> |
Use boost::urls::format
boost::urls::format is specifically meant for URL construction. It handles encoding like percentage encoding which std::format does not.
Change-Id: Ib6e6bc8bcf30b138b9311157
Use boost::urls::format
boost::urls::format is specifically meant for URL construction. It handles encoding like percentage encoding which std::format does not.
Change-Id: Ib6e6bc8bcf30b138b93111577d9f2150709dda5c Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
| d5a39575 | 24-Nov-2025 |
Ed Tanous <etanous@nvidia.com> |
Inline globals
These globals shouldn't have been added to the redfish namespace, and can cause -Wshadow issues. Inline the variables for now, and we can assess whether we want to promote these to a
Inline globals
These globals shouldn't have been added to the redfish namespace, and can cause -Wshadow issues. Inline the variables for now, and we can assess whether we want to promote these to a common API later.
Change-Id: I57bf1754cdad6ccd2a37375fad4b26996f0e9e9a Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
| 6926d430 | 24-Nov-2025 |
Oliver Brewka <oliver.brewka@9elements.com> |
Combine handlers for systems eventlog
Following patch 85522 [1], combine the handlers in systems eventlog.
Tested: Unit tests pass.
[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/85522
Change-
Combine handlers for systems eventlog
Following patch 85522 [1], combine the handlers in systems eventlog.
Tested: Unit tests pass.
[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/85522
Change-Id: Ib1649de83e64abb289e0379139d0a75466e15257 Signed-off-by: Oliver Brewka <oliver.brewka@9elements.com>
show more ...
|
| c621786e | 24-Nov-2025 |
Ed Tanous <etanous@nvidia.com> |
Combine handlers
Fix one comment made on [1] to combine method calls into a single one.
Tested: Unit tests pass, @Oliver Brewka need your help to sanity check this.
[1] https://gerrit.openbmc.org/
Combine handlers
Fix one comment made on [1] to combine method calls into a single one.
Tested: Unit tests pass, @Oliver Brewka need your help to sanity check this.
[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/83355
Change-Id: Iad90d57c6cd0fe3926186f238aba887b88e99c12 Signed-off-by: Ed Tanous <etanous@nvidia.com>
show more ...
|
| e5ab2df5 | 21-Jul-2025 |
rajeeranjan <ranjan.rajeev1609@gmail.com> |
Add temp file and FD support to TemporaryFileHandle
This commit adds file descriptor and temporary file management to DuplicatableFileHandle, removing the redundant test-only TemporaryFileHandle uti
Add temp file and FD support to TemporaryFileHandle
This commit adds file descriptor and temporary file management to DuplicatableFileHandle, removing the redundant test-only TemporaryFileHandle utility.
Changes: - Add file descriptor constructor and setFd() method - Add temporary file constructor with string_view content - Add filePath member and automatic cleanup in destructor - Add configurable temp-dir meson option (default: /tmp/bmcweb) - Remove include/file_test_utilities.hpp - Update all tests to use DuplicatableFileHandle - Rename stringPath to filePath
These features will be used by the multipart parser to stream large uploads to temporary files instead of keeping them in memory, and by the update service to pass file descriptors over D-Bus.
Change-Id: I982f5928d453f9f0c13d91c3525006134ddc87b3 Signed-off-by: Rajeev Ranjan <ranjan.rajeev1609@gmail.com>
show more ...
|
| 7bf29ab3 | 23-Oct-2025 |
Christopher Meis <christopher.meis@9elements.com> |
redfish-core: Refactor storage.hpp
This change splits out storage controller and storage drive route setup and code into new files and unifies the call of route setup for storage, storage controller
redfish-core: Refactor storage.hpp
This change splits out storage controller and storage drive route setup and code into new files and unifies the call of route setup for storage, storage controller and storage drive collection and single storage controller properties.
Tested: CI ran through without errors, build testing succeeded.
Change-Id: I234eb034205b7e6c3a5f307172019d53d42de6e3 Signed-off-by: Christopher Meis <christopher.meis@9elements.com>
show more ...
|
| a0170837 | 13-Oct-2025 |
Oliver Brewka <oliver.brewka@9elements.com> |
Rename Systems EventLog route handler
Rename the route handler to clearly indicate, that the handler handles the ComputerSystem redfish resource.
Tested: Code compiles.
Change-Id: Ib722427ab53f35b
Rename Systems EventLog route handler
Rename the route handler to clearly indicate, that the handler handles the ComputerSystem redfish resource.
Tested: Code compiles.
Change-Id: Ib722427ab53f35b2270fe6e102614f21fe28ffcb Signed-off-by: Oliver Brewka <oliver.brewka@9elements.com>
show more ...
|
| 79a916a2 | 15-Oct-2025 |
Oliver Brewka <oliver.brewka@9elements.com> |
Fix includes in log_services_utils
Remove unused includes and include boost_formatter util, in order to pass error codes to std::format.
Tested: Code compiles.
Change-Id: I9e2f0886695a5e12dee929b7
Fix includes in log_services_utils
Remove unused includes and include boost_formatter util, in order to pass error codes to std::format.
Tested: Code compiles.
Change-Id: I9e2f0886695a5e12dee929b7dd2c08d288effc94 Signed-off-by: Oliver Brewka <oliver.brewka@9elements.com>
show more ...
|
| ab8aaeba | 27-Aug-2025 |
Oliver Brewka <oliver.brewka@9elements.com> |
Add DBus EventLog to Manager
In order to get access to the EventLog on multi-host platforms, add Journal EventLog to Manager. This implementation is based on the discussion we had on patch 76319 [1]
Add DBus EventLog to Manager
In order to get access to the EventLog on multi-host platforms, add Journal EventLog to Manager. This implementation is based on the discussion we had on patch 76319 [1].
TLDR: On multi-host, we technically would have to split the event log on a per host node basis, so that each host node has its own specific event log.
However, this is currently not supported so we had to decide, whether we put it on a specific ComputerSystem, or refactor the current implementation of the EventLog, to allow for the EventLog LogService to be part of the Managers resource. We chose the latter one, because a), it is not clear on which ComputerSystem to put the EventLog, as long as we aren't splitting the event log per host node, and b), if that particular ComputerSystem is not existing at runtime, there would be no access to the EventLog at all.
This feature can be enabled with the redfish-eventlog-location meson option. By default it is set to 'systems', which translates to the EventLog being under the Systems resource. To enable the EventLog under the Managers resource set
``` -Dredfish-eventlog-location=managers ``` This in turn, disables the EventLog under the ComputerSystem resource.
Tested: Redfish validation succeeded for both ComputerSystem and Managers tree.
Patch LogEntry: ``` $ curl -k -X PATCH 'https://'"${BMC}"':'"${BMC_WEBPORT}"'/redfish/v1/Managers/bmc/LogServices/EventLog/Entries/4444' \ -H 'X-Auth-Token: '"$BMCWEB_SESSION_TOKEN"'' \ -H "Content-Type: application/json" -d "{"Resolved":true}"
$ busctl introspect xyz.openbmc_project.Logging /xyz/openbmc_project/logging/entry/4444
NAME TYPE SIGNATURE RESULT/VALUE FLAGS
...
xyz.openbmc_project.Logging.Entry interface - - - .GetEntry method - h - .AdditionalData property a{ss} 5 "CALLOUT_INVENTORY_PATH" "/xyz/open... emits-change writable .EventId property s "" emits-change writable .Id property u 4444 emits-change writable .Message property s "xyz.openbmc_project.Sensor.Threshold... emits-change writable .Resolution property s "" emits-change writable .Resolved property b true emits-change writable
```
Delete LogEntry: ``` $ busctl tree xyz.openbmc_project.Logging ... | |- /xyz/openbmc_project/logging/entry/6498 | |- /xyz/openbmc_project/logging/entry/6499 | |- /xyz/openbmc_project/logging/entry/6500 | |- /xyz/openbmc_project/logging/entry/6501 ...
$ curl -k -X DELETE 'https://'"${BMC}"':4443/redfish/v1/Managers/bmc/LogServices/EventLog/Entries/6500' \ -H 'X-Auth-Token: '"$BMCWEB_SESSION_TOKEN"'' \ -H "Content-Type: application/json"
| |- /xyz/openbmc_project/logging/entry/6498 | |- /xyz/openbmc_project/logging/entry/6499 | |- /xyz/openbmc_project/logging/entry/6501 ```
ClearLog action: ``` $ curl -k -X POST 'https://'"${BMC}"':4443/redfish/v1/Managers/bmc/LogServices/EventLog/Actions/LogService.ClearLog' \ -H 'X-Auth-Token: '"$BMCWEB_SESSION_TOKEN"'' \ -H "Content-Type: application/json"
{ "@Message.ExtendedInfo": [ { "@odata.type": "#Message.v1_1_1.Message", "Message": "The request completed successfully.", "MessageArgs": [], "MessageId": "Base.1.19.Success", "MessageSeverity": "OK", "Resolution": "None." } ] } ```
[1] https://gerrit.openbmc.org/c/openbmc/bmcweb/+/76319
Change-Id: Ie619af8e10a723aef2e02ef03796851511b05008 Signed-off-by: Oliver Brewka <oliver.brewka@9elements.com>
show more ...
|
| 42d4e63b | 27-Aug-2025 |
Oliver Brewka <oliver.brewka@9elements.com> |
Dynamic DBus EventLog handlers
In order to reuse handlers for DBus EventLog under Managers, add an additional argument that specifies the redfish resource.
Tested: Code compiles. Redfish validation
Dynamic DBus EventLog handlers
In order to reuse handlers for DBus EventLog under Managers, add an additional argument that specifies the redfish resource.
Tested: Code compiles. Redfish validation succeeds. Additionally the curl output with and without the changes has been diffed. No differences observed.
Change-Id: I13ec2f6e54f078b5b1064fb62c5561daa50f881a Signed-off-by: Oliver Brewka <oliver.brewka@9elements.com>
show more ...
|