Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Docs] Logging format samples outdated #32262

Open
renshuki opened this issue Jul 8, 2022 · 3 comments
Open

[Docs] Logging format samples outdated #32262

renshuki opened this issue Jul 8, 2022 · 3 comments
Labels
docs Stalled Team:Docs Label for the Observability docs team

Comments

@renshuki
Copy link
Contributor

renshuki commented Jul 8, 2022

Description:

Per Beats logs are now ECS-compliant breaking change in 8.0, logging format is now ndjson.

From current Beat - Configure Logging documentation, log samples are still in old format.
Specific sections for reference:

2017-12-17T19:17:42.667-0500    INFO    [metrics]       log/log.go:110  Non-zero metrics in the last 30s: beat.info.uptime.ms=30004 beat.memstats.gc_next=5046416
2017-12-17T18:54:16.241-0500	INFO	logp/core_test.go:13	unnamed global logger

2017-12-17T18:54:16.242-0500	INFO	[example]	logp/core_test.go:16	some message

2017-12-17T18:54:16.242-0500	INFO	[example]	logp/core_test.go:19	some message	{"x": 1}

As the old logging format is not available since 8.0, it would be great to accomodate this part of the documentation by changing the log samples to the current format.

@renshuki renshuki added docs Team:Docs Label for the Observability docs team labels Jul 8, 2022
@elasticmachine

This comment was marked as outdated.

@botelastic
Copy link

botelastic bot commented Jul 8, 2023

Hi!
We just realized that we haven't looked into this issue in a while. We're sorry!

We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1.
Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Jul 8, 2023
@botelastic botelastic bot closed this as completed Jan 4, 2024
@renshuki renshuki reopened this Jan 4, 2024
@botelastic botelastic bot removed the Stalled label Jan 4, 2024
@botelastic
Copy link

botelastic bot commented Jan 3, 2025

Hi!
We just realized that we haven't looked into this issue in a while. We're sorry!

We're labeling this issue as Stale to make it hit our filters and make sure we get back to it as soon as possible. In the meantime, it'd be extremely helpful if you could take a look at it as well and confirm its relevance. A simple comment with a nice emoji will be enough :+1.
Thank you for your contribution!

@botelastic botelastic bot added the Stalled label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Stalled Team:Docs Label for the Observability docs team
Projects
None yet
Development

No branches or pull requests

2 participants