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

Change log level for missing file case #357

Merged
merged 1 commit into from
Jan 26, 2022

Conversation

djaglowski
Copy link
Member

When a file cannot be opened by the file_input
operator, we currently print an error log. In cases
where the same file is repeatedly unopenable, this will
spam error logs every poll interval. By making this a
debug log, users can still discover the issue if closely
concerned.

Resolves #342

When a file cannot be opened by the file_input
operator, we currently print an error log. In cases
where the same file is repeatedly unopenable, this will
spam error logs every poll interval. By making this a
debug log, users can still discover the issue if closely
concerned.
@codecov
Copy link

codecov bot commented Jan 24, 2022

Codecov Report

Merging #357 (96b7e43) into main (ad3817c) will decrease coverage by 0.0%.
The diff coverage is 0.0%.

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #357     +/-   ##
=======================================
- Coverage   77.2%   77.1%   -0.1%     
=======================================
  Files         94      94             
  Lines       4471    4471             
=======================================
- Hits        3452    3449      -3     
- Misses       699     702      +3     
  Partials     320     320             
Impacted Files Coverage Δ
operator/builtin/input/file/file.go 72.3% <0.0%> (ø)
...perator/builtin/transformer/recombine/recombine.go 74.3% <0.0%> (-2.1%) ⬇️

@djaglowski djaglowski marked this pull request as ready for review January 24, 2022 22:28
@djaglowski djaglowski requested review from a team and jsirianni January 24, 2022 22:28
@djaglowski djaglowski merged commit e1f6b76 into open-telemetry:main Jan 26, 2022
@djaglowski djaglowski deleted the missing-file-loglevel branch January 26, 2022 00:06
jsirianni pushed a commit to jsirianni/opentelemetry-log-collection that referenced this pull request Mar 28, 2022
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.

Keep on trying to open an invalid symbol link file
2 participants