tree 83ccadfddba8b5341fca224c154c60bee822bff6
parent e2f14a5748e2bedc926bdd7a855ff4fb368ae1a5
author Brandon Wyman <bjwyman@gmail.com> 1619468787 -0500
committer Brandon Wyman <bjwyman@gmail.com> 1619470472 -0500

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