All of lore.kernel.org
 help / color / mirror / Atom feed
* Deprecate phosphor-hostlogger?
@ 2025-07-09 13:53 Patrick Williams
  2025-07-09 19:45 ` Ed Tanous
  0 siblings, 1 reply; 2+ messages in thread
From: Patrick Williams @ 2025-07-09 13:53 UTC (permalink / raw)
  To: OpenBMC List

[-- Attachment #1: Type: text/plain, Size: 1298 bytes --]

Hello,

Does anyone have opinions on if we deprecate phosphor-hostlogger?

I ended up becoming the "maintainer" of it due to the original maintainers
no longer being able to contribute to OpenBMC.  I've read through the
README and it is very much not clear to me what advantages it has over
using the similar functionality contained in obmc-console directly.
There is some mention of rsyslog and logrotate integration, but
obmc-console already supports a logrotate-like functionality, and I
think rsyslog could be easily configured to pick up the log files
created by obmc-console, if someone were so inclined.

It appears to only be used by the following meta layers:

    * meta-amd
    * meta-ampere
    * meta-bytedance
    * meta-quanta / s6q
    * meta-tyan
    * meta-yadro

It seems that this repository also incurred some bmcweb support that is
enabled for most systems (by default), even though they do not have the
corresponding daemon.

Can any of the system owners above comment as to what functionality they
are gaining by using phosphor-hostlogger over using obmc-console
directly?  Is this some functionality missing from obmc-console or is it
just a relationship between host logging, bmcweb and rsyslog that we
need to add?

-- 
Patrick Williams

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Deprecate phosphor-hostlogger?
  2025-07-09 13:53 Deprecate phosphor-hostlogger? Patrick Williams
@ 2025-07-09 19:45 ` Ed Tanous
  0 siblings, 0 replies; 2+ messages in thread
From: Ed Tanous @ 2025-07-09 19:45 UTC (permalink / raw)
  To: Patrick Williams; +Cc: OpenBMC List

On Wed, Jul 9, 2025 at 7:01 AM Patrick Williams <patrick@stwcx.xyz> wrote:
>
> Hello,
>
> Does anyone have opinions on if we deprecate phosphor-hostlogger?
>
> I ended up becoming the "maintainer" of it due to the original maintainers
> no longer being able to contribute to OpenBMC.  I've read through the
> README and it is very much not clear to me what advantages it has over
> using the similar functionality contained in obmc-console directly.
> There is some mention of rsyslog and logrotate integration, but
> obmc-console already supports a logrotate-like functionality, and I
> think rsyslog could be easily configured to pick up the log files
> created by obmc-console, if someone were so inclined.
>
> It appears to only be used by the following meta layers:
>
>     * meta-amd
>     * meta-ampere
>     * meta-bytedance
>     * meta-quanta / s6q
>     * meta-tyan
>     * meta-yadro
>
> It seems that this repository also incurred some bmcweb support that is
> enabled for most systems (by default), even though they do not have the
> corresponding daemon.
>
> Can any of the system owners above comment as to what functionality they
> are gaining by using phosphor-hostlogger over using obmc-console
> directly?  Is this some functionality missing from obmc-console or is it
> just a relationship between host logging, bmcweb and rsyslog that we
> need to add?
>
> --
> Patrick Williams

I'm not aware of any bmcweb users. Since the feature was merged it has
been relatively quiet in terms of additions/bug fixes.  I'm in support
of removing/deprecating it.

-Ed


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2025-07-09 19:45 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-09 13:53 Deprecate phosphor-hostlogger? Patrick Williams
2025-07-09 19:45 ` Ed Tanous

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.