search

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

Revert "Metrics tracing tags" #1628

Closed EddeCCC closed 11 months ago

EddeCCC commented 11 months ago

Reverts inspectIT/inspectit-ocelot#1626 since the PR https://github.com/inspectIT/inspectit-ocelot/pull/1625 should be merged first

This change is Reviewable

codecov[bot] commented 11 months ago

Codecov Report

Merging #1628 (dec01d1) into master (219f9ed) will decrease coverage by 0.18%. The diff coverage is 94.59%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1628 +/- ## ============================================ - Coverage 78.56% 78.38% -0.18% + Complexity 2578 2519 -59 ============================================ Files 266 257 -9 Lines 8453 8257 -196 Branches 995 984 -11 ============================================ - Hits 6641 6472 -169 + Misses 1377 1358 -19 + Partials 435 427 -8 ``` | [Files](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT) | Coverage Δ | | |---|---|---| | [...t/ocelot/config/model/metrics/MetricsSettings.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb25maWcvc3JjL21haW4vamF2YS9yb2Nrcy9pbnNwZWN0aXQvb2NlbG90L2NvbmZpZy9tb2RlbC9tZXRyaWNzL01ldHJpY3NTZXR0aW5ncy5qYXZh) | `50.00% <ø> (ø)` | | | [...l/metrics/definition/MetricDefinitionSettings.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb25maWcvc3JjL21haW4vamF2YS9yb2Nrcy9pbnNwZWN0aXQvb2NlbG90L2NvbmZpZy9tb2RlbC9tZXRyaWNzL2RlZmluaXRpb24vTWV0cmljRGVmaW5pdGlvblNldHRpbmdzLmphdmE=) | `100.00% <ø> (ø)` | | | [...nfig/model/selfmonitoring/AgentHealthSettings.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb25maWcvc3JjL21haW4vamF2YS9yb2Nrcy9pbnNwZWN0aXQvb2NlbG90L2NvbmZpZy9tb2RlbC9zZWxmbW9uaXRvcmluZy9BZ2VudEhlYWx0aFNldHRpbmdzLmphdmE=) | `0.00% <ø> (ø)` | | | [...t/ocelot/config/model/tracing/TracingSettings.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb25maWcvc3JjL21haW4vamF2YS9yb2Nrcy9pbnNwZWN0aXQvb2NlbG90L2NvbmZpZy9tb2RlbC90cmFjaW5nL1RyYWNpbmdTZXR0aW5ncy5qYXZh) | `100.00% <ø> (ø)` | | | [...core/command/handler/impl/LogsCommandExecutor.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2NvbW1hbmQvaGFuZGxlci9pbXBsL0xvZ3NDb21tYW5kRXhlY3V0b3IuamF2YQ==) | `85.00% <ø> (ø)` | | | [.../propertysources/http/HttpConfigurationPoller.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2NvbmZpZy9wcm9wZXJ0eXNvdXJjZXMvaHR0cC9IdHRwQ29uZmlndXJhdGlvblBvbGxlci5qYXZh) | `84.00% <100.00%> (+2.52%)` | :arrow_up: | | [...elot/core/exporter/OtlpMetricsExporterService.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2V4cG9ydGVyL090bHBNZXRyaWNzRXhwb3J0ZXJTZXJ2aWNlLmphdmE=) | `70.49% <ø> (ø)` | | | [...tation/config/MethodHookConfigurationResolver.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2luc3RydW1lbnRhdGlvbi9jb25maWcvTWV0aG9kSG9va0NvbmZpZ3VyYXRpb25SZXNvbHZlci5qYXZh) | `96.18% <100.00%> (ø)` | | | [...core/instrumentation/hook/MethodHookGenerator.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2luc3RydW1lbnRhdGlvbi9ob29rL01ldGhvZEhvb2tHZW5lcmF0b3IuamF2YQ==) | `85.95% <100.00%> (-1.64%)` | :arrow_down: | | [.../instrumentation/hook/actions/MetricsRecorder.java](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT#diff-aW5zcGVjdGl0LW9jZWxvdC1jb3JlL3NyYy9tYWluL2phdmEvcm9ja3MvaW5zcGVjdGl0L29jZWxvdC9jb3JlL2luc3RydW1lbnRhdGlvbi9ob29rL2FjdGlvbnMvTWV0cmljc1JlY29yZGVyLmphdmE=) | `100.00% <100.00%> (ø)` | | | ... and [8 more](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT) | | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/inspectIT/inspectit-ocelot/pull/1628/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=inspectIT)