Skip to content

feat: add sending of logs directly to otel collector#1431

Open
mutambaraf wants to merge 11 commits intodevelopfrom
150-improve-log-monitoring-send-logs-directly-to-collector
Open

feat: add sending of logs directly to otel collector#1431
mutambaraf wants to merge 11 commits intodevelopfrom
150-improve-log-monitoring-send-logs-directly-to-collector

Conversation

@mutambaraf
Copy link
Copy Markdown
Contributor

@mutambaraf mutambaraf commented Mar 23, 2026

Description

This PR introduces the feature of sending logs directly to the OpenTelemetry (OTEL) collector.
Part of: https://github.com/isisbusapps/observability/issues/150

Motivation and Context

The purpose of this change is to enhance the observability and debugging capabilities of the system. By sending logs directly to the OTEL collector, removing the need of scrapping logs.

Changes

  1. Added the OTLPLogExporter from the @opentelemetry/exporter-logs-otlp-http package to enable sending of logs to the OTEL collector.
  2. Configured the WinstonLogger to send logs directly to collector if the OTEL_EXPORTER_OTLP_LOGS_ENDPOINT environment variable is set.
  3. Updated the tracing middleware to include the BatchLogRecordProcessor from @opentelemetry/sdk-logs for processing the logs before they are sent to the OTEL collector.

How Has This Been Tested?

Fixes Jira Issue

https://jira.ess.eu//browse/

Depends On

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@mutambaraf mutambaraf marked this pull request as ready for review March 26, 2026 14:38
@mutambaraf mutambaraf requested a review from a team as a code owner March 26, 2026 14:38
@mutambaraf mutambaraf requested review from a team, EdwardHaynes and martin-trajanovski and removed request for a team, EdwardHaynes and martin-trajanovski March 26, 2026 14:38
@mutambaraf mutambaraf requested a review from janosbabik March 26, 2026 14:47
@Scott-James-Hurley
Copy link
Copy Markdown
Contributor

How do we test this?

@mutambaraf
Copy link
Copy Markdown
Contributor Author

How do we test this?

Will demo it today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants