databases/influxdb: Improve syslog logging title
ClosedPublic
Actions

Authored by driesm on Jan 16 2022, 9:55 AM.

Details

Reviewers

0mp
philip

Commits

R11:dc765dc71a5b: databases/influxdb: Improve syslog logging title

Summary

databases/influxdb: Improve syslog logging title

While here also remove trailing spaces.

 Approved by:		tbd (mentor)
 Pull Request:		https://github.com/freebsd/freebsd-ports/pull/121
 Differential Revision:	https://reviews.freebsd.org/D33907

Diff Detail

Repository

rP FreeBSD ports repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 43937
Build 40825: arc lint + arc unit

Event Timeline

driesm created this revision.Jan 16 2022, 9:55 AM

Herald added a subscriber: mat. · View Herald TranscriptJan 16 2022, 9:55 AM

driesm requested review of this revision.Jan 16 2022, 9:55 AM

Harbormaster completed remote builds in B43937: Diff 101498.Jan 16 2022, 9:55 AM

driesm edited the summary of this revision. (Show Details)Jan 16 2022, 9:56 AM

driesm added reviewers: 0mp, philip.

As explained in the PR, this changes the log entry from:

Jan 13 02:08:13 icarus daemon[43241]: ts=2022-01-13T10:08:13.768315Z lvl=info msg="InfluxDB starting" log_id=0Z0jOen0000 version=1.8.9 branch=unknown commit=unknown

Jan 13 02:09:10 icarus influxd[43292]: ts=2022-01-13T10:09:10.424810Z lvl=info msg="InfluxDB starting" log_id=0Z0jS760000 version=1.8.9 branch=unknown commit=unknown

In D33907#766946, @driesm wrote:

As explained in the PR, this changes the log entry from:

Jan 13 02:08:13 icarus daemon[43241]: ts=2022-01-13T10:08:13.768315Z lvl=info msg="InfluxDB starting" log_id=0Z0jOen0000 version=1.8.9 branch=unknown commit=unknown

To

Jan 13 02:09:10 icarus influxd[43292]: ts=2022-01-13T10:09:10.424810Z lvl=info msg="InfluxDB starting" log_id=0Z0jS760000 version=1.8.9 branch=unknown commit=unknown

I'd include it in the commit message.

LGTM otherwise!

This revision is now accepted and ready to land.Jan 25 2022, 12:38 AM

Closed by commit R11:dc765dc71a5b: databases/influxdb: Improve syslog logging title (authored by Jeremy Chadwick <jdc@koitsu.org>, committed by driesm). · Explain WhyJan 25 2022, 7:38 PM

This revision was automatically updated to reflect the committed changes.

driesm added a commit: R11:dc765dc71a5b: databases/influxdb: Improve syslog logging title.