You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kie.apache.org by po...@apache.org on 2024/04/02 18:21:57 UTC

(incubator-kie-kogito-images) branch main updated: Fix #1749 - Rename SonataFlow, jBPM, and Drools extensions (#1750)

This is an automated email from the ASF dual-hosted git repository.

porcelli pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-kie-kogito-images.git


The following commit(s) were added to refs/heads/main by this push:
     new b4cf1cbd Fix #1749 - Rename SonataFlow, jBPM, and Drools extensions (#1750)
b4cf1cbd is described below

commit b4cf1cbdf98a8958eb0fdb10e3372152f43c2b3d
Author: Ricardo Zanini <15...@users.noreply.github.com>
AuthorDate: Tue Apr 2 15:21:53 2024 -0300

    Fix #1749 - Rename SonataFlow, jBPM, and Drools extensions (#1750)
    
    * Fix #1749 - Rename SonataFlow, jBPM, and Drools extensions
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Review org.kie.kogito to org.kie namespace
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Rename to org.apache.kie.sonataflow
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Fix behave tests log regexp to match extensions descriptions
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Trying to switch back to sh from bash
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Changing behave tests fork to experiment with an older cekit version
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Using jmtd cekit-behave ref
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Fix devmodeui behave tests
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    * Remove cekit-behave workaround
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
    
    ---------
    
    Signed-off-by: Ricardo Zanini <za...@redhat.com>
---
 modules/kogito-s2i-core/added/s2i-core              |  4 ++--
 modules/kogito-s2i-core/module.yaml                 |  2 +-
 modules/kogito-swf/builder/build-config/module.yaml |  3 +--
 modules/kogito-swf/common/scripts/module.yaml       |  2 +-
 modules/kogito-swf/devmode/build-config/module.yaml |  2 +-
 tests/features/common.feature                       |  4 ++--
 tests/features/kogito-s2i-builder.feature           |  2 +-
 tests/features/kogito-swf-builder.feature           |  4 ++--
 tests/features/kogito-swf-common.feature            |  2 +-
 tests/features/kogito-swf-devmode.feature           | 16 ++++++++--------
 10 files changed, 20 insertions(+), 21 deletions(-)

diff --git a/modules/kogito-s2i-core/added/s2i-core b/modules/kogito-s2i-core/added/s2i-core
index 46ff1a60..7a4070d1 100644
--- a/modules/kogito-s2i-core/added/s2i-core
+++ b/modules/kogito-s2i-core/added/s2i-core
@@ -154,7 +154,7 @@ function build_kogito_app() {
                     -DprojectGroupId=$PROJECT_GROUP_ID \
                     -DprojectArtifactId=$PROJECT_ARTIFACT_ID \
                     -DprojectVersion=$PROJECT_VERSION \
-                    -Dextensions="org.kie.kogito:kogito-quarkus:${KOGITO_VERSION},quarkus-smallrye-health,quarkus-smallrye-openapi,quarkus-resteasy,quarkus-resteasy-jackson"
+                    -Dextensions="org.jbpm:jbpm-with-drools-quarkus:${KOGITO_VERSION},quarkus-smallrye-health,quarkus-smallrye-openapi,quarkus-resteasy,quarkus-resteasy-jackson"
 
             if [ ! -z ${QUARKUS_EXTRA_EXTENSIONS} ]; then
                 log_info "----> Adding extra Quarkus extensions: ${QUARKUS_EXTRA_EXTENSIONS}"
@@ -236,7 +236,7 @@ function build_kogito_app() {
 
             $MAVEN_HOME/bin/mvn ${MAVEN_ARGS_APPEND} -s "${MAVEN_SETTINGS_PATH}" $(get_quarkus_platform_properties) \
                 "${quarkus_plugin}":"${quarkus_platform_version}":add-extension \
-                -Dextensions="org.kie.kogito:kogito-quarkus-serverless-workflow:${KOGITO_VERSION}"
+                -Dextensions="org.apache.kie.sonataflow:sonataflow-quarkus:${KOGITO_VERSION}"
         fi
 
         $MAVEN_HOME/bin/mvn clean package ${MAVEN_ARGS_APPEND} ${KOGITO_OPTS} ${nativeBuild} -s "${MAVEN_SETTINGS_PATH}" \
diff --git a/modules/kogito-s2i-core/module.yaml b/modules/kogito-s2i-core/module.yaml
index 27f67a4e..d2e8b05a 100644
--- a/modules/kogito-s2i-core/module.yaml
+++ b/modules/kogito-s2i-core/module.yaml
@@ -70,7 +70,7 @@ envs:
 
   - name: QUARKUS_EXTRA_EXTENSIONS
     description: ^ Defines the Quarkus' extra extensions (as comma-separated list) to be added when performing build from assets.
-    example: "org.kie.kogito:kogito-addons-quarkus-knative-eventing,quarkus-kubernetes"
+    example: "org.kie:kie-addons-quarkus-knative-eventing,quarkus-kubernetes"
 
 modules:
   install:
diff --git a/modules/kogito-swf/builder/build-config/module.yaml b/modules/kogito-swf/builder/build-config/module.yaml
index b3563932..f233d8ca 100644
--- a/modules/kogito-swf/builder/build-config/module.yaml
+++ b/modules/kogito-swf/builder/build-config/module.yaml
@@ -25,6 +25,5 @@ envs:
   - name: "SCRIPT_DEBUG"
     value: "false"
   - name: QUARKUS_EXTENSIONS
-    # NOTE: If you change the QUARKUS_EXTENSIONS value remember to update the scripts/logic/build-quarkus-app.sh too!
     # Follow up issue to remove KOGITO_VERSION: https://issues.redhat.com/browse/KOGITO-9270
-    value: quarkus-kubernetes,smallrye-health,org.kie.kogito:kogito-quarkus-serverless-workflow:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-knative-eventing:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-microprofile-config-service-catalog:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-kubernetes:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-events-process:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-process-management:${KOGITO_VERSION},org.k [...]
+    value: quarkus-kubernetes,smallrye-health,org.apache.kie.sonataflow:sonataflow-quarkus:${KOGITO_VERSION},org.kie:kie-addons-quarkus-knative-eventing:${KOGITO_VERSION},org.kie:kogito-addons-quarkus-microprofile-config-service-catalog:${KOGITO_VERSION},org.kie:kie-addons-quarkus-kubernetes:${KOGITO_VERSION},org.kie:kie-addons-quarkus-events-process:${KOGITO_VERSION},org.kie:kie-addons-quarkus-process-management:${KOGITO_VERSION},org.kie:kie-addons-quarkus-source-files:${KOGITO_VERSION} [...]
diff --git a/modules/kogito-swf/common/scripts/module.yaml b/modules/kogito-swf/common/scripts/module.yaml
index 4d10be8a..dd90f59e 100644
--- a/modules/kogito-swf/common/scripts/module.yaml
+++ b/modules/kogito-swf/common/scripts/module.yaml
@@ -23,7 +23,7 @@ description: "Kogito Serverless Workflow image common scripts"
 
 envs:
   - name: QUARKUS_EXTENSIONS
-    example: 'quarkus-kubernetes,org.kie.kogito:kogito-quarkus-serverless-workflow:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-knative-eventing:${KOGITO_VERSION}'
+    example: 'quarkus-kubernetes,org.apache.kie.sonataflow:sonataflow-quarkus:${KOGITO_VERSION},org.kie:kie-addons-quarkus-knative-eventing:${KOGITO_VERSION}'
     description: To add extension to your application
 
   - name: PROJECT_GROUP_ID
diff --git a/modules/kogito-swf/devmode/build-config/module.yaml b/modules/kogito-swf/devmode/build-config/module.yaml
index 292e0535..b67edb9b 100644
--- a/modules/kogito-swf/devmode/build-config/module.yaml
+++ b/modules/kogito-swf/devmode/build-config/module.yaml
@@ -27,4 +27,4 @@ envs:
   - name: QUARKUS_EXTENSIONS
     # NOTE: If you change the QUARKUS_EXTENSIONS value remember to update the scripts/logic/build-quarkus-app.sh too!
     # Follow up issue to remove KOGITO_VERSION: https://issues.redhat.com/browse/KOGITO-9270
-    value: org.kie.kogito:kogito-quarkus-serverless-workflow:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-knative-eventing:${KOGITO_VERSION},smallrye-health,org.kie.kogito:kogito-quarkus-serverless-workflow-devui:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-source-files:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-process-management:${KOGITO_VERSION},org.kie.kogito:kogito-addons-quarkus-jobs-service-embedded:${KOGITO_VERSION},org.kie.kogito:kogito-addons-qua [...]
+    value: org.apache.kie.sonataflow:sonataflow-quarkus:${KOGITO_VERSION},org.kie:kie-addons-quarkus-knative-eventing:${KOGITO_VERSION},smallrye-health,org.apache.kie.sonataflow:sonataflow-quarkus-devui:${KOGITO_VERSION},org.kie:kie-addons-quarkus-source-files:${KOGITO_VERSION},org.kie:kie-addons-quarkus-process-management:${KOGITO_VERSION},org.kie:kogito-addons-quarkus-jobs-service-embedded:${KOGITO_VERSION},org.kie:kogito-addons-quarkus-data-index-inmemory:${KOGITO_VERSION},org.kie:kog [...]
diff --git a/tests/features/common.feature b/tests/features/common.feature
index 77923242..4fcbdec2 100644
--- a/tests/features/common.feature
+++ b/tests/features/common.feature
@@ -13,8 +13,8 @@
 Feature: Common tests for Kogito images
 
   Scenario: Verify if Kogito user is correctly configured
-    When container is started with command bash
-    Then run bash -c 'echo $USER' in container and check its output for kogito
+    When container is started with command sh
+    Then run sh -c 'echo $USER' in container and check its output for kogito
      And run sh -c 'echo $HOME' in container and check its output for /home/kogito
      And run sh -c 'id' in container and check its output for uid=1001(kogito) gid=0(root) groups=0(root),1001(kogito)
 
diff --git a/tests/features/kogito-s2i-builder.feature b/tests/features/kogito-s2i-builder.feature
index 9135d1e2..3031b628 100644
--- a/tests/features/kogito-s2i-builder.feature
+++ b/tests/features/kogito-s2i-builder.feature
@@ -104,7 +104,7 @@ Feature: kogito-s2i-builder image tests
       | PROJECT_ARTIFACT_ID      | myproject                                             |
       | PROJECT_VERSION          | 2.0-SNAPSHOT                                          |
       | K_SINK                   | http://localhost:8181                                 |
-      | QUARKUS_EXTRA_EXTENSIONS | org.kie.kogito:kogito-addons-quarkus-knative-eventing |
+      | QUARKUS_EXTRA_EXTENSIONS | org.kie:kie-addons-quarkus-knative-eventing |
     Then file /home/kogito/bin/quarkus-run.jar should exist
     And s2i build log should contain Generating quarkus project structure for myproject...
     And s2i build log should contain Adding Kogito Quarkus Workflows extension to the generated project.
diff --git a/tests/features/kogito-swf-builder.feature b/tests/features/kogito-swf-builder.feature
index 2f6f0b54..acae47c9 100644
--- a/tests/features/kogito-swf-builder.feature
+++ b/tests/features/kogito-swf-builder.feature
@@ -15,7 +15,7 @@ Feature: Serverless Workflow builder images
     And container log should contain --no-transfer-progress
     And container log should contain -Duser.home=/home/kogito
     And container log should match regex Installed features:.*kogito-serverless-workflow
-    And container log should match regex Installed features:.*kogito-addon-knative-eventing-extension
+    And container log should match regex Installed features:.*kie-addon-knative-eventing-extension
     And container log should match regex Installed features:.*smallrye-health
 
   Scenario: Verify that the application is built and started correctly when QUARKUS_EXTENSIONS env is used
@@ -33,6 +33,6 @@ Feature: Serverless Workflow builder images
     And container log should contain -Duser.home=/home/kogito
     And container log should contain Extension io.quarkus:quarkus-elytron-security-jdbc has been installed
     And container log should match regex Installed features:.*kogito-serverless-workflow
-    And container log should match regex Installed features:.*kogito-addon-knative-eventing-extension
+    And container log should match regex Installed features:.*kie-addon-knative-eventing-extension
     And container log should match regex Installed features:.*smallrye-health
     And container log should match regex Installed features:.*security-jdbc
\ No newline at end of file
diff --git a/tests/features/kogito-swf-common.feature b/tests/features/kogito-swf-common.feature
index d81b28e1..f69a482a 100644
--- a/tests/features/kogito-swf-common.feature
+++ b/tests/features/kogito-swf-common.feature
@@ -5,7 +5,7 @@ Feature: Serverless Workflow images common
   Scenario: Verify if the swf and quarkus files are under /home/kogito/.m2/repository
     When container is started with command bash
     Then file /home/kogito/.m2/repository/io/quarkus/platform/quarkus-bom/3.2.10.Final/quarkus-bom-3.2.10.Final.pom should exist
-      And file /home/kogito/.m2/repository/org/kie/kogito/kogito-quarkus-serverless-workflow/ should exist and be a directory
+      And file /home/kogito/.m2/repository/org/apache/kie/sonataflow/sonataflow-quarkus/ should exist and be a directory
 
   # This check should be enabled again once a similar check is done on runtimes
   # to make sure we only have one version of quarkus bom ...
diff --git a/tests/features/kogito-swf-devmode.feature b/tests/features/kogito-swf-devmode.feature
index 471d0e94..c17ff1f9 100644
--- a/tests/features/kogito-swf-devmode.feature
+++ b/tests/features/kogito-swf-devmode.feature
@@ -16,10 +16,10 @@ Feature: Serverless Workflow devmode images
     And container log should contain -Duser.home=/home/kogito -o
     And container log should contain -Dquarkus.test.continuous-testing=disabled
     And container log should match regex Installed features:.*kogito-serverless-workflow
-    And container log should match regex Installed features:.*kogito-addon-knative-eventing-extension
+    And container log should match regex Installed features:.*kie-addon-knative-eventing-extension
     And container log should match regex Installed features:.*smallrye-health
-    And container log should match regex Installed features:.*kogito-serverless-workflow-devui
-    And container log should match regex Installed features:.*kogito-addon-source-files-extension
+    And container log should match regex Installed features:.*sonataflow-quarkus-devui
+    And container log should match regex Installed features:.*kie-addon-source-files-extension
     And container log should match regex Installed features:.*kogito-addons-quarkus-jobs-service-embedded
     And container log should match regex Installed features:.*kogito-addons-quarkus-data-index-inmemory
 
@@ -55,10 +55,10 @@ Feature: Serverless Workflow devmode images
     And container log should not contain /bin/mvn -B -X --batch-mode -o
     And container log should contain Extension io.quarkus:quarkus-elytron-security-jdbc has been installed
     And container log should match regex Installed features:.*kogito-serverless-workflow
-    And container log should match regex Installed features:.*kogito-addon-knative-eventing-extension
+    And container log should match regex Installed features:.*kie-addon-knative-eventing-extension
     And container log should match regex Installed features:.*smallrye-health
-    And container log should match regex Installed features:.*kogito-serverless-workflow-devui
-    And container log should match regex Installed features:.*kogito-addon-source-files-extension
+    And container log should match regex Installed features:.*sonataflow-quarkus-devui
+    And container log should match regex Installed features:.*kie-addon-source-files-extension
     And container log should match regex Installed features:.*kogito-addons-quarkus-jobs-service-embedded
     And container log should match regex Installed features:.*kogito-addons-quarkus-data-index-inmemory
     And container log should match regex Installed features:.*security-jdbc
@@ -114,14 +114,14 @@ Feature: Serverless Workflow devmode images
     Then check that page is served
       | property             | value                                                              |
       | port                 | 8080                                                               |
-      | path                 | /q/dev-v1/org.kie.kogito.kogito-quarkus-serverless-workflow/dataindex |
+      | path                 | /q/dev-v1/org.kie.kogito-addons-quarkus-data-index-inmemory/dataindex |
       | request_method       | GET                                                                |
       | wait                 | 480                                                                |
       | expected_status_code | 200                                                                |
     And check that page is served
       | property             | value                                                                            |
       | port                 | 8080                                                                             |
-      | path                 | /q/dev-v1/org.kie.kogito.kogito-quarkus-serverless-workflow-devui/workflowInstances |
+      | path                 | /q/dev-ui/org.apache.kie.sonataflow.sonataflow-quarkus-devui/workflows |
       | request_method       | GET                                                                              |
       | wait                 | 480                                                                              |
       | expected_status_code | 200                                                                              |


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org