tree 476d9098b39ac2a75434f86cff7fe86e902fbe2d
parent 35f5e0e3dff9364b7813cca42b1ef29de3535181
author Brad Bishop <bradleyb@fuzziesquirrel.com> 1573764567 -0500
committer Brad Bishop <bradleyb@fuzziesquirrel.com> 1584979855 -0400

entity-manager: remove inventory manager interface

Remove the inventory manager interface added with fd1264a.  It is a
barrier to entity-manager adoption in stacks using
phosphor-inventory-manager.

The rationale given for fd1264a was:

> This interface is used to identify demons that
> contain inventory, add the interface for lookups.

Identifying daemons that contain inventory can simply look for objects
in the /inventory path namespace.  Performing lookups can be done in the
usual ways via the OpenBMC object mapper or entity-manager's
org.freedesktop.DBus.ObjectManager implementation.

The Manager interface documentation is ambiguous and open to
interpretation but the intent of the Manager interface seems to be for
applications that wish to "own" the lifecycle of inventory objects while
not hosting them, and the Notify method of the interface provides the
means for that.  Entity manager does not attempt to meet this design
point at all (as is clearly indicated by the no-op Notify implementation
being removed) so there is little value in providing the interface.

Change-Id: I2b18e45c7b8c4e427ebb76a46c8070466d79e60d
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>