openbmc/phosphor-health-monitor issues, please help check.

[Vijay Khemka]

Hi Bruce,
Let me look into this.

Regards
-Vijay

From: "Bruce Lee (李昀峻)" <Bruce_Lee at quantatw.com>
Date: Monday, September 21, 2020 at 2:42 AM
To: "openbmc at lists.ozlabs.org" <openbmc at lists.ozlabs.org>, Vijay Khemka <vijaykhemka at fb.com>
Subject: openbmc/phosphor-health-monitor issues, please help check.

Hi Vijay,

I found one question in openbmc/phosphor-health-monitor
The service healthMon via above functions created has not the org.freedesktop.DBus.ObjectManager on the root,
and this will cause the sensor not be generated correctly via redfish url /redfish/v1/Chassis/chassis/Sensors/CPU, /redfish/v1/Chassis/chassis/Sensors/Memory
Because redfish will method GetSubTree to collect sensors via org.freedesktop.DBus.ObjectManager
I found a solution for your reference and show at bellow, you can use sdbusplus/server/manager.hpp which provide a function to add ObjectManager on the root.
Hence, I can get CPU, Memory information correctly via redfish url /redfish/v1/Chassis/chassis/Sensors/CPU, /redfish/v1/Chassis/chassis/Sensors/Memory

#include <sdbusplus/server/manager.hpp>

class HealthMon
{
  public:
    HealthMon() = delete;
    HealthMon(const HealthMon&) = delete;
    HealthMon& operator=(const HealthMon&) = delete;
    HealthMon(HealthMon&&) = delete;
    HealthMon& operator=(HealthMon&&) = delete;
    virtual ~HealthMon() = default;

    /** @brief Constructs HealthMon
     *
     * @param[in] bus     - Handle to system dbus
     */
    HealthMon(sdbusplus::bus::bus& bus) : bus(bus)
    {
        // read json file
        sensorConfigs = getHealthConfig();
        createHealthSensors();
        add_manager("/");
    }

    /** @brief Parsing Health config JSON file  */
    Json parseConfigFile(std::string configFile);

    /** @brief reading config for each health sensor component */
    void getConfigData(Json& data, HealthConfig& cfg);

    /** @brief Map of the object HealthSensor */
    std::unordered_map<std::string, std::shared_ptr<HealthSensor>>
        healthSensors;

    /** @brief Create sensors for health monitoring */
    void createHealthSensors();

    void add_manager(const std::string& path)
    {
        managers_.emplace_back(
            std::make_unique<sdbusplus::server::manager::manager>(
                sdbusplus::server::manager::manager(
                bus, path.c_str())));
    }
  private:
    sdbusplus::bus::bus& bus;
    std::vector<HealthConfig> sensorConfigs;
    std::vector<HealthConfig> getHealthConfig();
    std::vector<std::unique_ptr<sdbusplus::server::manager::manager>> managers_;
};

This pic is the add org.freedesktop.DBus.ObjectManager on the root for reference.
[cid:image001.jpg at 01D69135.E3EBE470]

[cid:image002.jpg at 01D69135.E3EBE470]

Best Regards,
Bruce

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20200923/ccbc44d8/attachment-0001.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image001.jpg
Type: image/jpeg
Size: 26230 bytes
Desc: image001.jpg
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20200923/ccbc44d8/attachment-0002.jpg>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: image002.jpg
Type: image/jpeg
Size: 25742 bytes
Desc: image002.jpg
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20200923/ccbc44d8/attachment-0003.jpg>