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.

Sign up for GitHub

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

Add container parser blog post #4489

Merged
merged 6 commits into from
May 22, 2024
+231 −0
Merged

Add container parser blog post #4489

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
c47b246
add container parser blogpost
ChrsMark May 15, 2024
9fc5d8a
switch to single voice
ChrsMark May 17, 2024
db56d91
add note about reduced configuration
ChrsMark May 20, 2024
07bddcc
Merge branch 'main' into add_container_parser_blog_post
svrnm May 22, 2024
eba145b
Merge branch 'main' into add_container_parser_blog_post
svrnm May 22, 2024
c472855
Merge branch 'main' into add_container_parser_blog_post
svrnm May 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Loading
Loading status checks…
add note about reduced configuration 
Signed-off-by: ChrsMark <[email protected]>
@ChrsMark
ChrsMark committed May 22, 2024

Verified

This commit was signed with the committer’s verified signature.
ronnnnn Seiya Kokushi
GPG key ID: B927D86C6064A7CB
Verified
Learn about vigilant mode
commit db56d910d3800f9ca693196c017d9db2bd653dda
7 changes: 5 additions & 2 deletions content/en/blog/2024/otel-collector-container-log-parser/index.md
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
---
title: Introducing the new container log parser for OpenTelemetry Collector
linkTitle: Collector container log parser
date: 2024-05-17
date: 2024-05-22
author: '[Christos Markou](https://github.com/ChrsMark) (Elastic)'
cSpell:ignore: Christos containerd Filelog filelog Jaglowski kube Markou
---
@@ -119,7 +119,10 @@ receivers:
```
Copy link
Member

@AlexanderWert AlexanderWert May 18, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about showing the comparison to the config previously (with the container parser) required (that you mention in the issue above: open-telemetry/opentelemetry-collector-contrib#31959)?
Or at least mention that the config now is 8 lines long vs. ~80 lines to achieve the same result.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point! Added at 44778bd, thank's.


That configuration is more than enough to properly parse the log line and
extract all the useful Kubernetes metadata.
extract all the useful Kubernetes metadata. It's quite obvious how much less
configuration is required now. Using a combination of operators would result in
about 69 lines of configuration as it was pointed out at the
[original proposal](https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/31959).

A log line
`{"log":"INFO: This is a docker log line","stream":"stdout","time":"2024-03-30T08:31:20.545192187Z"}`