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.

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

Add an opentracing capability #289

Closed
nicolaferraro opened this issue Apr 1, 2020 · 1 comment · Fixed by #353
Closed

Add an opentracing capability #289

nicolaferraro opened this issue Apr 1, 2020 · 1 comment · Fixed by #353

Comments

@nicolaferraro
Copy link
Member

It's currently possible to configure opentracing via quarkus using instructions from https://camel.apache.org/camel-quarkus/latest/extensions/opentracing.html#_usage.

Running the integration with command:

kamel run Example.java -t quarkus.enabled=true -d mvn:org.apache.camel.quarkus:camel-quarkus-opentracing --property-file application.properties

But it uses quarkus properties and the same feature is not available in plain main.

I wonder if this is a case for adding a capability.
I don't know also if we can uniform those properties between environments.

@lburgazzoli
Copy link
Contributor

Adding a capability is definitively easy but harmonizing the properties could be a very hard and fragile work as quarkus may introduce new properties and we also need to distinguish between build and runtime ones.

I think one day we may default to quarkus as runtime ....

lburgazzoli added a commit to lburgazzoli/apache-camel-k-runtime that referenced this issue Jun 4, 2020
lburgazzoli added a commit to lburgazzoli/apache-camel-k-runtime that referenced this issue Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants