Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Journald config #252

Merged
merged 3 commits into from
Aug 28, 2021
Merged

Journald config #252

merged 3 commits into from
Aug 28, 2021

Conversation

rockb1017
Copy link
Contributor

implements #251

@rockb1017 rockb1017 requested a review from a team August 27, 2021 23:10
Copy link
Member

@djaglowski djaglowski left a comment

Choose a reason for hiding this comment

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

Nice feature. Thanks @rockb1017!

operator/builtin/input/journald/journald.go Outdated Show resolved Hide resolved
@djaglowski djaglowski self-requested a review August 27, 2021 23:51
@codecov
Copy link

codecov bot commented Aug 28, 2021

Codecov Report

Merging #252 (f296a3f) into main (eeab999) will decrease coverage by 0.0%.
The diff coverage is 66.6%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #252     +/-   ##
=======================================
- Coverage   75.7%   75.7%   -0.1%     
=======================================
  Files         95      95             
  Lines       4450    4453      +3     
=======================================
+ Hits        3369    3371      +2     
- Misses       751     752      +1     
  Partials     330     330             
Impacted Files Coverage Δ
operator/builtin/input/journald/journald.go 52.8% <66.6%> (+0.4%) ⬆️

@djaglowski djaglowski merged commit 1b149bb into open-telemetry:main Aug 28, 2021
@rockb1017 rockb1017 deleted the journaldConfig branch August 28, 2021 01:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants