inspectIT / inspectit-ocelot

inspectIT Ocelot - Java agent for collecting application performance, tracing and behavior data
http://www.inspectit.rocks/
Apache License 2.0
204 stars 69 forks source link

Closes #1534 and #1535 - Otlp*ExporterService does not expose underlying OTEL configuration #1536

Closed Dimi-Ma closed 1 year ago

Dimi-Ma commented 1 year ago

Closes #1534 and #1535


This change is Reviewable

codecov[bot] commented 1 year ago

Codecov Report

Merging #1536 (b9b9436) into master (4ae7a26) will decrease coverage by 0.79%. The diff coverage is 25.44%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1536 +/- ## ============================================ - Coverage 78.64% 77.85% -0.79% - Complexity 2271 2285 +14 ============================================ Files 232 235 +3 Lines 7438 7570 +132 Branches 887 902 +15 ============================================ + Hits 5849 5893 +44 - Misses 1219 1309 +90 + Partials 370 368 -2 ``` | [Impacted Files](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT) | Coverage Δ | | |---|---|---| | [.../src/main/java/com/mindprod/jarcheck/JarCheck.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvY29tL21pbmRwcm9kL2phcmNoZWNrL0phckNoZWNrLmphdmE=) | `0.00% <0.00%> (ø)` | | | [...ocelot/core/exporter/OtlpTraceExporterService.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2V4cG9ydGVyL090bHBUcmFjZUV4cG9ydGVyU2VydmljZS5qYXZh) | `66.67% <73.33%> (+1.80%)` | :arrow_up: | | [...elot/core/exporter/OtlpMetricsExporterService.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2V4cG9ydGVyL090bHBNZXRyaWNzRXhwb3J0ZXJTZXJ2aWNlLmphdmE=) | `69.49% <75.00%> (+3.64%)` | :arrow_up: | | [...elot/config/model/exporters/CompressionMethod.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb25maWcvc3JjL21haW4vamF2YS9yb2Nrcy9pbnNwZWN0aXQvb2NlbG90L2NvbmZpZy9tb2RlbC9leHBvcnRlcnMvQ29tcHJlc3Npb25NZXRob2QuamF2YQ==) | `100.00% <100.00%> (ø)` | | | [...service/DynamicallyActivatableServiceObserver.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL3NlbGZtb25pdG9yaW5nL3NlcnZpY2UvRHluYW1pY2FsbHlBY3RpdmF0YWJsZVNlcnZpY2VPYnNlcnZlci5qYXZh) | `50.00% <0.00%> (ø)` | | | [.../propertysources/http/HttpPropertySourceState.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2NvbmZpZy9wcm9wZXJ0eXNvdXJjZXMvaHR0cC9IdHRwUHJvcGVydHlTb3VyY2VTdGF0ZS5qYXZh) | `77.86% <0.00%> (+0.17%)` | :arrow_up: | | [...ot/core/service/DynamicallyActivatableService.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL3NlcnZpY2UvRHluYW1pY2FsbHlBY3RpdmF0YWJsZVNlcnZpY2UuamF2YQ==) | `83.72% <0.00%> (+0.79%)` | :arrow_up: | | [.../ocelot/core/metrics/system/GCMetricsRecorder.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL21ldHJpY3Mvc3lzdGVtL0dDTWV0cmljc1JlY29yZGVyLmphdmE=) | `72.95% <0.00%> (+0.82%)` | :arrow_up: | | [...ot/core/instrumentation/AsyncClassTransformer.java](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2luc3RydW1lbnRhdGlvbi9Bc3luY0NsYXNzVHJhbnNmb3JtZXIuamF2YQ==) | `85.45% <0.00%> (+0.91%)` | :arrow_up: | | ... and [3 more](https://codecov.io/gh/inspectIT/inspectit-ocelot/pull/1536/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT) | |