Skip to content

Tracer doesn't continue spans for AbstractInternalSpanDecorator#9584

Merged
davsclaus merged 4 commits intoapache:camel-3.xfrom
marcingrzejszczak:issues_#19124
Mar 28, 2023
Merged

Tracer doesn't continue spans for AbstractInternalSpanDecorator#9584
davsclaus merged 4 commits intoapache:camel-3.xfrom
marcingrzejszczak:issues_#19124

Conversation

@marcingrzejszczak
Copy link
Contributor

@marcingrzejszczak marcingrzejszczak commented Mar 20, 2023

this PR fixes the issue in production code and updates tests for Micrometer Observations.

IMPORTANT: OpenTracing module has its tests skipped with this PR

fixes https://issues.apache.org/jira/browse/CAMEL-19124

@github-actions
Copy link
Contributor

🌟 Thank you for your contribution to the Apache Camel project! 🌟

⚠️ Please note that the changes on this PR may be tested automatically.

If necessary Apache Camel Committers may access logs and test results in the job summaries!

@github-actions
Copy link
Contributor

Components tested:

Total Tested Failed ❌ Passed ✅
2 2 2 0

@davsclaus
Copy link
Contributor

open tracing is deprecated so its most important for opentelemtry.

@marcingrzejszczak
Copy link
Contributor Author

Sure, but opentracing tests are also failing. I mean they can be skipped but I don't know if it's a good idea ;)

@marcingrzejszczak marcingrzejszczak marked this pull request as ready for review March 22, 2023 16:55
@marcingrzejszczak
Copy link
Contributor Author

I've updated OpenTelemetry tests and disabled tests in OpenTracing. Assuming that the build passes I think we're safe to merge this

@github-actions
Copy link
Contributor

Components tested:

Total Tested Failed ❌ Passed ✅
4 4 3 1

@github-actions
Copy link
Contributor

Components tested:

Total Tested Failed ❌ Passed ✅
4 4 3 1

@marcingrzejszczak
Copy link
Contributor Author

This PR needs to be merged for Camel 3.x to work fine. This was fixed for the main branch of Camel.

@davsclaus davsclaus merged commit a793948 into apache:camel-3.x Mar 28, 2023
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.

3 participants