psu-ng: Call updateInventory() in PSU constructor
After adding the changes to get configuration data from entity-manager,
the timing with how processes start up changed a bit.
We used to get bus, address, and inventory path information from the
JSON file, created the power supplies, prior to the
phosphor-gpio-presence application doing the interfaces added call for
the Present property. Then we would get the interfaces added signal,
which would call updateInventory().
With the entity-manager changes, it appears that we typically startup
before the Presence interface added, and before the bus, address, and
other information from entity-manager is ready. The Present property
gets added, the entity-manager adds the IBMCFFPSConnector information,
then we create the power suppplies, but now there is no call to
updateInventory() as we missed the interfaces added signal for the
Present property.
We probably should have been calling updateInventory() in the
constructor all this time. If the power supply is not present at that
time, it just skips doing anything. If it is present, it will update the
VPD properties on the D-Bus inventory interface.
Tested:
Created rainier image containing phosphor-power changes.
Flashed image to Rainier 2S4U system.
Removed powersupply* directories from
/var/lib/phosphor-inventory-manager persisted data tree.
Rebooted BMC.
Verify D-Bus for powersupply* has VPD keyword properties updated.
Signed-off-by: Brandon Wyman <bjwyman@gmail.com>
Change-Id: Iabff937b25910faeca6ffe8aa1f376132e514a81
diff --git a/phosphor-power-supply/power_supply.cpp b/phosphor-power-supply/power_supply.cpp
index dbdfb20..847e686 100644
--- a/phosphor-power-supply/power_supply.cpp
+++ b/phosphor-power-supply/power_supply.cpp
@@ -50,6 +50,7 @@
// Get the current state of the Present property.
updatePresence();
+ updateInventory();
}
void PowerSupply::updatePresence()