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

Add logs test with OpenTelemetry log receiver #548

Merged
merged 2 commits into from
Jun 5, 2024

Conversation

keiko713
Copy link
Contributor

Follow-up of #544

Here is how it looks with success:

/ $ LOG_OTEL_SERVER=0.0.0.0:4318 /home/pganalyze/collector --test-logs
2024/05/16 03:01:31 I Running collector test with pganalyze-collector 0.56.0
2024/05/16 03:01:31 I [default] Testing log collection (OpenTelemetry Log receiving)...
2024/05/16 03:01:31 I [default] Setting up OTLP HTTP server receiving logs with 0.0.0.0:4318
2024/05/16 03:01:35 I [default]   Log test successful

When the collector is already running well and receiving logs in background, it'll look like the following:

/ $ /home/pganalyze/collector --test-logs
2024/05/16 03:02:24 I Running collector test with pganalyze-collector 0.56.0
2024/05/16 03:02:24 I [default] Testing log collection (OpenTelemetry Log receiving)...
2024/05/16 03:02:24 I [default] Setting up OTLP HTTP server receiving logs with 0.0.0.0:4318
2024/05/16 03:02:34 E [default] ERROR - OpenTelemetry log tail timed out after 10 seconds - did not find expected log event in stream
2024/05/16 03:02:24 I [default] HINT - This error may be a false positive if the collector is also running in the background and receiving logs

@keiko713 keiko713 requested a review from a team May 16, 2024 03:07
Copy link
Contributor

@msakrejda msakrejda left a comment

Choose a reason for hiding this comment

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

Thanks for adding this 👍

@keiko713 keiko713 merged commit 41f40ba into main Jun 5, 2024
9 checks passed
@keiko713 keiko713 deleted the add-logs-test-with-otel branch June 5, 2024 08:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants