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

chore: modify LogExporter and TraceExporter interfaces to support returning failure #2381

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

more error changes

3c68efa
Select commit
Loading
Failed to load commit list.
Open

chore: modify LogExporter and TraceExporter interfaces to support returning failure #2381

more error changes
3c68efa
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 20, 2024 in 0s

58.4% of diff hit (target 76.1%)

View this Pull Request on Codecov

58.4% of diff hit (target 76.1%)

Annotations

Check warning on line 54 in opentelemetry-otlp/src/exporter/http/logs.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/logs.rs#L52-L54 

Added lines #L52 - L54 were not covered by tests

Check warning on line 70 in opentelemetry-otlp/src/exporter/http/trace.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-otlp/src/exporter/http/trace.rs#L68-L70 

Added lines #L68 - L70 were not covered by tests

Check warning on line 92 in opentelemetry-otlp/src/exporter/tonic/logs.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/logs.rs#L92 

Added line #L92 was not covered by tests

Check warning on line 94 in opentelemetry-otlp/src/exporter/tonic/logs.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/logs.rs#L94 

Added line #L94 was not covered by tests

Check warning on line 94 in opentelemetry-otlp/src/exporter/tonic/trace.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/trace.rs#L94 

Added line #L94 was not covered by tests

Check warning on line 96 in opentelemetry-otlp/src/exporter/tonic/trace.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-otlp/src/exporter/tonic/trace.rs#L96 

Added line #L96 was not covered by tests

Check warning on line 135 in opentelemetry-sdk/src/export/trace.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/export/trace.rs#L133-L135 

Added lines #L133 - L135 were not covered by tests

Check warning on line 59 in opentelemetry-sdk/src/logs/error.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/logs/error.rs#L59 

Added line #L59 was not covered by tests

Check warning on line 86 in opentelemetry-sdk/src/logs/error.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/logs/error.rs#L84-L86 

Added lines #L84 - L86 were not covered by tests

Check warning on line 146 in opentelemetry-sdk/src/logs/log_emitter.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/logs/log_emitter.rs#L146 

Added line #L146 was not covered by tests

Check warning on line 153 in opentelemetry-sdk/src/logs/log_processor.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/logs/log_processor.rs#L153 

Added line #L153 was not covered by tests

Check warning on line 657 in opentelemetry-sdk/src/logs/log_processor.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/logs/log_processor.rs#L657 

Added line #L657 was not covered by tests

Check warning on line 660 in opentelemetry-sdk/src/logs/log_processor.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/logs/log_processor.rs#L660 

Added line #L660 was not covered by tests

Check warning on line 432 in opentelemetry-sdk/src/trace/span_processor.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/trace/span_processor.rs#L432 

Added line #L432 was not covered by tests

Check warning on line 435 in opentelemetry-sdk/src/trace/span_processor.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-sdk/src/trace/span_processor.rs#L435 

Added line #L435 was not covered by tests

Check warning on line 63 in opentelemetry-stdout/src/logs/exporter.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/logs/exporter.rs#L63 

Added line #L63 was not covered by tests

Check warning on line 65 in opentelemetry-stdout/src/logs/exporter.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/logs/exporter.rs#L65 

Added line #L65 was not covered by tests

Check warning on line 62 in opentelemetry-stdout/src/trace/exporter.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/trace/exporter.rs#L62 

Added line #L62 was not covered by tests

Check warning on line 64 in opentelemetry-stdout/src/trace/exporter.rs

See this annotation in the file changed.

@codecov codecov / codecov/patch

opentelemetry-stdout/src/trace/exporter.rs#L64 

Added line #L64 was not covered by tests
View more details on Codecov