SRVLOGIC-1158: Differentiate Quarkus core and platform version by domhanak · Pull Request #466 · kubesmarts/kie-tools

domhanak · 2026-08-28T07:51:21Z

https://redhat.atlassian.net/browse/SRVLOGIC-1158

Cherry-picks 465 to main

run standard e2e
run operator e2e

domhanak · 2026-08-28T08:10:46Z

-QUARKUS_PLATFORM_version=3.33.3
-QUARKUS_VERSION=3.33.3
+QUARKUS_PLATFORM_version=3.33.2.SP2-redhat-00002
+QUARKUS_VERSION=3.33.2.redhat-00008


will be adjusted once we have a proof of this working

domhanak · 2026-08-28T08:29:07Z

So builder image for example built well:
2026-08-28T08:22:41.7325380Z �[32m2026-08-28 08:22:41,729 docker_builder.py:108 INFO Docker: Step 54/74 : ENV KOGITO_VERSION="111-SNAPSHOT" QUARKUS_PLATFORM_GROUPID="com.redhat.quarkus.platform" QUARKUS_PLATFORM_VERSION="3.33.2.SP2-redhat-00002" QUARKUS_VERSION="3.33.2.redhat-00008" SONATAFLOW_QUARKUS_DEVUI_VERSION="111-SNAPSHOT"�[0m

Looking into the test failures that follow

domhanak · 2026-08-28T08:30:32Z

Ok so test needs to be adjusted:
Exception: File /home/kogito/.m2/repository/io/quarkus/platform/quarkus-bom does not exist

fyi @jakubschwan

domhanak · 2026-08-28T10:41:42Z

Adjusted tests, applied new lables ( some are still missing, working on that ), adjusted quarkus-mvn-plugin.sh

domhanak · 2026-08-28T12:29:40Z

Still couple of fixes needed, will santize here and port to 9.106.x-prod after

domhanak · 2026-08-31T14:05:09Z

Okay, just some instabilities left and operator e2e. Working on those

domhanak · 2026-08-31T19:36:38Z

Comparing to https://github.com/kubesmarts/kie-tools/pull/451/changes#diff-0f93e9e480131c2502d82d564b801a60fee9d430632d0bd490c843805fb7fabb we still need to check some workflow plugin e2e. Working on it.

wmedvede

I added some minor comments that I think can be worth to take a second look at.

wmedvede · 2026-09-09T16:09:22Z

  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 should exist and be a directory
+    Then file /home/kogito/.m2/repository/com/redhat/quarkus/platform/quarkus-bom should exist and be a directory


So from now, this will only work wuen using the RH quarkus platform?
I'm confused with this line here https://github.com/kubesmarts/kie-tools/pull/466/changes#diff-70f308b794edcd370a6aaadeb47dfaca08e14bb5e2c6c3b7d864fddb97abf3c8L38

That says:

# Defines the value of used quarkus platform. For CI checks to work it needs to remain on community value # otherwise the build of container images is going to fail since "com.redhat.quarkus.platform" is only # available in "maven.repository.redhat.com" KOGITO_IMAGES_CEKIT_MODULES__quarkusGroupId=io.quarkus.platform KOGITO_IMAGES_CEKIT_MODULES__quarkusGroupId=com.redhat.quarkus.platform

That needs updating, but yes we want to test against productized Quarkus

Not sure how this will be, I mean, are you also going to force kogito-runtines and kogito-apps to work with productized Quarkus? Because AFAIK, we need all the stack aligned with the same Quarkus version.

wmedvede · 2026-09-09T16:24:54Z

 if [[ -n "${QUARKUS_VERSION}" && "${QUARKUS_VERSION}" != \#\#\#* ]]; then
  quarkus_version="${QUARKUS_VERSION}"
+else
+  quarkus_version="${QUARKUS_PLATFORM_VERSION}"


this fallback to QUARKUS_PLATFORM_VERSION I'm not sure if good 🤔
Since below, it's being used e.g. to complete the version when adding a quarkus extension when the version is missing, see:

https://github.com/kubesmarts/kie-tools/pull/466/changes#diff-33d82cd0e94d4c7820db405f666767e43d33e83d2032911c854505cffe73a902L51

we have basically:

# If the version is missing, append the default version based on the groupId if [ -z "$version" ]; then if [[ "$group_id" == "io.quarkus" ]]; then extension="${group_id}:${artifact_id}:${quarkus_version}" elif [[ "$group_id" == *"kie"* || "$group_id" == *"kogito"* || "$artifact_id" == *"kogito"* || "$artifact_id" == *"sonataflow"* ]]; then extension="${group_id}:${artifact_id}:${kogito_version}" fi fi

but, an io.quarkus extension will never have a QUARKUS_PLATFORM_VERSION like 3.33.2.SP2-redhat-00002 I think. Since that numer is only for the platform, but not for the inner extensions.

Yes, correct. Will double check then, I was not aware it is doing this

In general the fallback is here in case the QUARKUS_PLATFORM_VERSION=QUARKUS_VERSION we do not need to have 2 variables, however I would vote to always have them set.

wmedvede · 2026-09-09T16:31:43Z

@@ -166,10 +169,20 @@ def update_quarkus_platform_version(quarkus_platform_version):
    Update quarkus_platform_version version into images/modules
    :param quarkus_platform_version: quarkus version to set


Suggested change

:param quarkus_platform_version: quarkus version to set

:param quarkus_platform_version: quarkus platform version to set

wmedvede · 2026-09-09T16:34:54Z

    parser.add_argument('--source-folder', dest='source_folder')
    parser.add_argument('--quarkus-groupid', dest='quarkus_groupid', help='Sets the image Quarkus groupId')
-    parser.add_argument('--quarkus-version', dest='quarkus_version', help='Sets the image Quarkus Version')
+    parser.add_argument('--quarkus-version', dest='quarkus_version', help='Sets the image Quarkus Platform Version')


Suggested change

parser.add_argument('--quarkus-version', dest='quarkus_version', help='Sets the image Quarkus Platform Version')

parser.add_argument('--quarkus-version', dest='quarkus_version', help='Sets the image Quarkus Version')

domhanak added 3 commits August 28, 2026 09:44

Use com.redhat.quarkus.platform and 3.33.2.SP2-redhat-00002

0a61dbd

Override io.quarkus to com.redhat.quarkus

14d13cb

Attempt to differentiate between core and platform versions

cbb8372

domhanak requested review from baldimir, jankrystof-ibm, ricardozanini and wmedvede August 28, 2026 07:51

domhanak marked this pull request as draft August 28, 2026 07:52

domhanak marked this pull request as ready for review August 28, 2026 07:52

domhanak commented Aug 28, 2026

View reviewed changes

baldimir approved these changes Aug 28, 2026

View reviewed changes

Fix tests, add labels for quarkus core version, improve script

9494550

domhanak requested a review from jakubschwan August 28, 2026 10:49

domhanak added 2 commits August 28, 2026 13:10

Use QUARKUS_version and enhancements

500ab67

Fix version propagation issues, apply test fix for builder and devmode

8b2860a

jankrystof-ibm approved these changes Aug 28, 2026

View reviewed changes

Apply version split to kogito images, fix typo

c12fa33

Fix management-console-image package.json

d904448

jankrystof-ibm mentioned this pull request Aug 28, 2026

[9.106.x-prod] SRVLOGIC-1153 | Add missing install.js to sonataflow-management-console-image so image e2e finds the built image #468

Merged

domhanak added 2 commits August 28, 2026 15:44

Use extra extension that is in productized quarkus in tests

2a283be

Fix devmode image e2e regex

288d95f

domhanak changed the title ~~SRVLOGIC-1158: Attempt to differentiate Quarkus core and platform version~~ SRVLOGIC-1158: Differentiate Quarkus core and platform version Aug 31, 2026

domhanak added 2 commits August 31, 2026 09:53

Increase devmode-image timeout

a36739a

Further increase timeout and fix assert

d0633e2

ricardozanini approved these changes Aug 31, 2026

View reviewed changes

domhanak added 2 commits August 31, 2026 16:26

Explicitly set QUARKUS_PLATFORM_GROUPID in operator workflow

2506c9b

Set QUARKUS_version and QUARKUS_PLATFROM_version in operator wf

7bef170

wmedvede requested changes Sep 9, 2026

View reviewed changes

jankrystof-ibm approved these changes Sep 11, 2026

View reviewed changes

		@@ -166,10 +169,20 @@ def update_quarkus_platform_version(quarkus_platform_version):
		Update quarkus_platform_version version into images/modules
		:param quarkus_platform_version: quarkus version to set

	parser.add_argument('--quarkus-version', dest='quarkus_version', help='Sets the image Quarkus Platform Version')
	parser.add_argument('--quarkus-version', dest='quarkus_version', help='Sets the image Quarkus Version')

Conversation

domhanak commented Aug 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

domhanak commented Aug 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

domhanak commented Aug 28, 2026

Uh oh!

domhanak commented Aug 28, 2026

Uh oh!

domhanak commented Aug 28, 2026

Uh oh!

domhanak commented Aug 31, 2026

Uh oh!

domhanak commented Aug 31, 2026

Uh oh!

wmedvede left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

domhanak commented Aug 28, 2026 •

edited

Loading

domhanak commented Aug 28, 2026 •

edited

Loading