Gitiles
Code Review Sign In
gerrit.openbmc.org / openbmc / phosphor-host-ipmid / d09db492c545a333839f933914cb6f2180dc6f4f
commitd09db492c545a333839f933914cb6f2180dc6f4f[log] [tgz]
authorTim Lee <timlee660101@gmail.com>Mon May 30 10:22:34 2022 +0800
committerTom Joseph <rushtotom@gmail.com>Wed Jul 13 19:39:57 2022 +0000
treed0cef54c394bafaf8e3f1ff3dd62c70ccbe95cc4
parentb4905919f245a860fa423d13a5c8eaafa075f6f8 [diff]
Fix UnknownProperty error when getting sensor data record

Symptom:
When getting sensor data record by ipmitool sdr command,
there is error from dbus-monitor debug log as below:
method call time=1653875077.002911 sender=:1.56 -> destination=xyz.openbmc_project.Hwmon-3446998316.Hwmon1 serial=69975
   string "xyz.openbmc_project.Sensor.Threshold.Warning"
   string "WarningAlarmlow"
error time=1653875077.002975 sender=:1.52 -> destination=:1.56 error_name=org.freedesktop.DBus.Error.UnknownProperty re
   string "Unknown interface xyz.openbmc_project.Sensor.Threshold.Warning or property WarningAlarmlow."
method return time=1653875077.003043 sender=:1.56 -> destination=:1.107 serial=69976 reply_serial=11

Root cause:
The property "WarningAlarmlow" name is case sensitive

Solution:
Correct the property name "WarningAlarmlow" to "WarningAlarmLow"

Verified:
Run ipmitool sdr command and check dbus-monitor debug log

Signed-off-by: Tim Lee <timlee660101@gmail.com>
Change-Id: Ic92b4849373756bf94a4960d803fae5269d5b8e9
1 file changed
tree: d0cef54c394bafaf8e3f1ff3dd62c70ccbe95cc4
  1. app/
  2. dbus-sdr/
  3. docs/
  4. include/
  5. libipmid/
  6. libipmid-host/
  7. scripts/
  8. softoff/
  9. subprojects/
  10. test/
  11. user_channel/
  12. xyz/
  13. .build.sh
  14. .clang-format
  15. .gitignore
  16. .shellcheck
  17. .travis.yml
  18. apphandler.cpp
  19. apphandler.hpp
  20. chassishandler.cpp
  21. chassishandler.hpp
  22. dcmihandler.cpp
  23. dcmihandler.hpp
  24. elog-errors.hpp
  25. entity_map_json.cpp
  26. entity_map_json.hpp
  27. error-HostEvent.hpp
  28. fruread.hpp
  29. generate_whitelist.sh
  30. generate_whitelist_create.sh
  31. globalhandler.cpp
  32. globalhandler.hpp
  33. groupext.cpp
  34. host-cmd-manager.cpp
  35. host-cmd-manager.hpp
  36. host-interface.cpp
  37. host-interface.hpp
  38. host-ipmid-whitelist.conf
  39. ipmi_fru_info_area.cpp
  40. ipmi_fru_info_area.hpp
  41. ipmid-new.cpp
  42. ipmisensor.cpp
  43. ipmiwhitelist.hpp
  44. LICENSE
  45. MAINTAINERS
  46. meson.build
  47. meson_options.txt
  48. OWNERS
  49. read_fru_data.cpp
  50. read_fru_data.hpp
  51. README.md
  52. selutility.cpp
  53. selutility.hpp
  54. sensordatahandler.cpp
  55. sensordatahandler.hpp
  56. sensorhandler.cpp
  57. sensorhandler.hpp
  58. settings.cpp
  59. settings.hpp
  60. storageaddsel.cpp
  61. storageaddsel.hpp
  62. storagehandler.cpp
  63. storagehandler.hpp
  64. sys_info_param.cpp
  65. sys_info_param.hpp
  66. systemintfcmds.cpp
  67. systemintfcmds.hpp
  68. testaddsel.cpp
  69. testit.cpp
  70. transporthandler.cpp
  71. transporthandler.hpp
  72. whitelist-filter.cpp
README.md

To build this package, do the following steps:

1. ./bootstrap.sh
2. ./configure ${CONFIGURE_FLAGS}
3. make

To clean the repository run ./bootstrap.sh clean.