squakez commented on code in PR #1161:
URL: https://github.com/apache/camel-k-runtime/pull/1161#discussion_r1474374012


##########
support/camel-k-maven-plugin/src/main/java/org/apache/camel/k/tooling/maven/GenerateCatalogMojo.java:
##########
@@ -524,23 +525,26 @@ private void addCapabilities(RuntimeSpec.Builder 
runtimeSpec, CamelCatalogSpec.B
         artifacts.clear();
         artifacts.add(Artifact.from("org.apache.camel.quarkus", 
"camel-quarkus-management"));
         artifacts.add(Artifact.from("org.apache.camel.quarkus", 
"camel-quarkus-jaxb"));
-        artifacts.add(Artifact.from("org.jolokia", "jolokia-jvm"));
-        addCapabilityAndDependecies(runtimeSpec, catalogSpec, "jolokia", 
artifacts, false);
+        artifacts.add(Artifact.from("org.jolokia", "jolokia-agent-jvm", 
"javaagent"));

Review Comment:
   `jolokia-server-core` seems to be missing. In any case I am not 100% sure of 
which are the dependencies required as I did some quick try/error experiment. 
Maybe you can run a test to confirm that nothing else but the agent jar is 
really required. BTW, since we're there, let's move to 2.0.1.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@camel.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to