tree eb0ecbb08f2248761238ff4c25eb32687b7580f2
parent ea2873dd60765257aa8df05b2c8e2f7e43e707e6
author Matt Spinler <spinler@us.ibm.com> 1629386585 -0500
committer Matt Spinler <spinler@us.ibm.com> 1629898306 +0000

Stop emitting Entry propChanged before ifacesAdded

Even though the deferSignal parameter was set to true in the event log
entry D-Bus interface constructor, propertiesChanged signals would still
be emitted when setting the properties to their initial values.  These
were being emitted even before the interfacesAdded signal that is sent
when emit_object_added() is called.

As the code was calling emit_object_added() as soon as it initialized
the properties, it can be deduced that the intended plan was that no
propertiesChanged signals were expected to be sent in the first place.

Fix this by using the sdbusplus functions that take a boolean to skip
sending signals when properties are set to their initial values.

Tested:  With this change, propertiesChanged signals are not emitted
before the interfacesAdded signal for the
xyz.openbmc_project.Logging.Entry interface when either creating or
restoring event logs.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: I3548fe29a88719232e26dd806e196c156bcf0078