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


##########
support/camel-k-maven-plugin/src/main/java/org/apache/camel/k/tooling/maven/GenerateCatalogMojo.java:
##########
@@ -464,7 +465,11 @@ private void addCapabilities(RuntimeSpec.Builder 
runtimeSpec, CamelCatalogSpec.B
 
         artifacts.clear();
         artifacts.add(Artifact.from("org.apache.camel.quarkus", 
"camel-quarkus-microprofile-health"));
-        addCapabilityAndDependecies(runtimeSpec, catalogSpec, "health", 
artifacts, false);
+        List<Property> properties = new ArrayList<>();
+        properties.add(Property.from("defaultLivenessProbePath", 
"/q/health/live"));
+        properties.add(Property.from("defaultReadinessProbePath", 
"/q/health/ready"));
+        properties.add(Property.from("defaultStartupProbePath", 
"/q/health/started"));

Review Comment:
   No, I think it would be an additional configuration, not a replacement. The 
same `metadata` can be provided for other stuff that may be required with the 
meaning of "generic" configuration that have to be managed by the operator.



-- 
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