journal log messages are missing with latest version

[Patrick Venture]

On Thu, Jan 3, 2019 at 10:07 AM Matt Spinler
<mspinler at linux.vnet.ibm.com> wrote:
>
> On 2019-01-02 13:09, Patrick Venture wrote:
> > On Thu, Dec 20, 2018 at 4:38 PM Yong Li <yong.b.li at linux.intel.com>
> > wrote:
> >>
> >>
> >>
> >> Hi Patrick,
> >>
> >>
> >>
> >> I encountered a problem after upgrade to the latest opnebmc version:
> >>
> >>
> >>
> >> These log messages in ipmid are missing with the “journalctl -f”.
> >
> > Which log messages?
>
>
> This showed up on all of our systems too, anything logged using
> phosphor-logging
> is missing from the journal.  I even ended up using a git bisect to also
> narrow
> it down to:
>
> phosphor-logging: srcrev bump 30047bf964..31552c05e2
>
>      Patrick Venture (5):
>            add sdjournal interface to inject tests
>            test: use only one main library
>            bugfix: add fixes to mock class
>            bugfix: rename method to match implementation
>            test: add test to force building of sdjournal mock
>

William, you've been digging into sdbus a lot recently, can you take a
deep look at the patchset above and see what I may have messed up?
The phosphor-logging unit-tests were broken until I got it "working."
Which lead me to believe that it's not correct -- and the daemons that
have unit-tests that hit the log message are seeing the calls.

>
> >
> >>
> >>
> >>
> >> My tests shows that the blow change is related, after I revert it, I
> >> can get these log messages.
> >>
> >>
> >>
> >> Did you meet such issue, or any suggestions/comments to narrow down
> >> this issue?
> >>
> >> https://github.com/openbmc/phosphor-logging/commit/59a6b1f27e83066baa6f3711c88d6a3b9a5c4d49
> >>
> >> add sdjournal interface to inject tests
> >
> > There were some follow-on patches for phosphor-logging that you
> > shouldn't need -- it's
> >
> > What SHA is for your https://github.com/openbmc/phosphor-host-ipmid ?
> > (so I know what version of that is in use by you).
> >
> > Checking the HEAD version of
> > https://github.com/openbmc/phosphor-host-ipmid/blob/master/Makefile.am,
> > I see it's including the proper libs to get the full phosphor-logging
> > objects.
> >
> >>
> >>
> >>
> >> Thanks,
> >>
> >> Yong
>