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

[v2] Implement sanitizers to operate on OTLP data #5551

Closed
wants to merge 22 commits into from
+189 −0

fix vars

28def9c
Select commit
Loading
Failed to load commit list.
Closed

[v2] Implement sanitizers to operate on OTLP data #5551

fix vars
28def9c
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Jun 29, 2024 in 0s

0.00% of diff hit (target 95.00%)

View this Pull Request on Codecov

0.00% of diff hit (target 95.00%)

Annotations

Check warning on line 19 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go#L18-L19 

Added lines #L18 - L19 were not covered by tests

Check warning on line 29 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go#L23-L29 

Added lines #L23 - L29 were not covered by tests

Check warning on line 32 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go#L31-L32 

Added lines #L31 - L32 were not covered by tests

Check warning on line 34 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go#L34 

Added line #L34 was not covered by tests

Check warning on line 37 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/empty_service_name_sanitizer.go#L37 

Added line #L37 was not covered by tests

Check warning on line 17 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go#L14-L17 

Added lines #L14 - L17 were not covered by tests

Check warning on line 25 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go#L23-L25 

Added lines #L23 - L25 were not covered by tests

Check warning on line 29 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go#L27-L29 

Added lines #L27 - L29 were not covered by tests

Check warning on line 31 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/sanitizer.go#L31 

Added line #L31 was not covered by tests

Check warning on line 21 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L20-L21 

Added lines #L20 - L21 were not covered by tests

Check warning on line 28 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L25-L28 

Added lines #L25 - L28 were not covered by tests

Check warning on line 31 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L31 

Added line #L31 was not covered by tests

Check warning on line 35 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L33-L35 

Added lines #L33 - L35 were not covered by tests

Check warning on line 38 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L38 

Added line #L38 was not covered by tests

Check warning on line 42 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L40-L42 

Added lines #L40 - L42 were not covered by tests

Check warning on line 49 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L45-L49 

Added lines #L45 - L49 were not covered by tests

Check warning on line 53 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L53 

Added line #L53 was not covered by tests

Check warning on line 57 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L57 

Added line #L57 was not covered by tests

Check warning on line 61 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L61 

Added line #L61 was not covered by tests

Check warning on line 64 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L63-L64 

Added lines #L63 - L64 were not covered by tests

Check warning on line 66 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L66 

Added line #L66 was not covered by tests

Check warning on line 69 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L68-L69 

Added lines #L68 - L69 were not covered by tests

Check warning on line 73 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L72-L73 

Added lines #L72 - L73 were not covered by tests

Check warning on line 75 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L75 

Added line #L75 was not covered by tests

Check warning on line 83 in cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

cmd/jaeger/internal/exporters/storageexporter/sanitizer/utf8_sanitizer.go#L79-L83 

Added lines #L79 - L83 were not covered by tests
View more details on Codecov