ppalaga commented on code in PR #4465:
URL: https://github.com/apache/camel-quarkus/pull/4465#discussion_r1095420548


##########
extensions/qute/component/src/generated/resources/org/apache/camel/component/qute/qute.json:
##########
@@ -11,7 +11,7 @@
     "supportLevel": "Preview",
     "groupId": "org.apache.camel.quarkus",
     "artifactId": "camel-quarkus-qute-component",
-    "version": "2.16.0-SNAPSHOT",
+    "version": "2.17.0-SNAPSHOT",

Review Comment:
   > I tried the following steps
   > 
   > ```
   > ./mvnw cq:create -N -Dcq.artifactIdBase=cli-connector 
-Dcq.nativeSupported=false
   > ./mvnw clean install -DskipTests
   > ```
   > 
   > But it added itest in `integration-tests/cli-connector/` and the building 
failed with
   > 
   > ```
   > [ERROR] Failed to execute goal 
org.codehaus.gmaven:groovy-maven-plugin:2.1.1:execute 
(validate-github-workflows) on project camel-quarkus: Execution 
validate-github-workflows of goal 
org.codehaus.gmaven:groovy-maven-plugin:2.1.1:execute failed: Integration tests 
not executed by the CI:
   > [ERROR] 
   > [ERROR]     cli-connector
   > [ERROR] 
   > [ERROR]  Add the missing test module(s) to 
tooling/scripts/test-categories.yaml
   > ```
   > 
   > @ppalaga Is it a bug in cq-maven-plugin?
   
   No, I would not say it is a bug. Adding to test-categories.yaml needs to be 
done manually, because it is better to take the actual test execution times 
into consideration. The plugin does not know those numbers. In the past, I 
thought of storing the the CI module execution times somewhere, so that the 
plugin can add the new test to the currently fastest group, but I never come to 
it. I failed at finding some free metrics storage service. 



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