commit 9f9b38d89a751e70cdf61bfb3f78c05800201f95
author jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> Fri Dec 13 17:07:29 2019 +0000
committer jayaprakash Mutyala <mutyalax.jayaprakash@intel.com> Fri Dec 13 17:34:13 2019 +0000
tree 1eb6b7fc5a1feb5332dec7ef77f08d9ad549db35
parent fb64f45d3399b182ceadffb8fa86ee68c0aa0a11

PSUSensor: Fix for PSU details

Issue: When user trying to get PSU details in redfish, PSU detail logs
       are not coming under Chassis/WC_Baseboard/Power.

Fix: changed the order of InitialProperties of dbus connection

Tested:
Verified in Redfish.
GET https://<BMC-IP>/redfish/v1/Chassis/WC_Baseboard/Power
Response: PSU details are fetched under Chassis/WC_Baseboard/Power

"PowerSupplies": [
    {
      "@odata.id": "/redfish/v1/Chassis/WC_Baseboard/Power#/
          PowerSupplies/0",
      "Manufacturer": "SOLUM CO., LTD.                  ",
      "MemberId": "SOLUM_CO_IS162F22_PSU1",
      "Model": "IS162F22",
      "Name": "SOLUM CO IS162F22 PSU1",
      "PartNumber": "G36234-015",
      "PowerInputWatts": 108.0,
      "PowerOutputWatts": 11.53125,
      "SerialNumber": "PSSF162202ACNS1622A4BK7L0399",
      "Status": {
        "Health": "OK",
        "State": "Enabled"
      }
    },

Signed-off-by: jayaprakash Mutyala <mutyalax.jayaprakash@intel.com>
Signed-off-by: Anoop S <anoopx.s@intel.com>
Change-Id: I8016afd9bd0cc40d5b1f9b2143cd602a6056ecaa

src/PSUSensor.cpp

diff --git a/src/PSUSensor.cpp b/src/PSUSensor.cpp
index fc24704..0b71e92 100644
--- a/src/PSUSensor.cpp
+++ b/src/PSUSensor.cpp
@@ -73,9 +73,9 @@
         thresholdInterfaceCritical = objectServer.add_interface(
             dbusPath, "xyz.openbmc_project.Sensor.Threshold.Critical");
     }
-    association = objectServer.add_interface(dbusPath, association::interface);
 
     setInitialProperties(conn);
+    association = objectServer.add_interface(dbusPath, association::interface);
 
     createInventoryAssoc(conn, association, configurationPath);
     setupRead();