[sonataflow-operator] - Migrate SonataFlow operator from incubator-kie-kogito-serverless-operator #2747

ricardozanini · 2024-11-15T23:24:01Z

In this PR:

Upgrade repo Go version to 1.22
Remove the old sonataflow-operator for the 10.0.0 release
Push the new sonataflow-operator from https://github.com/apache/incubator-kie-kogito-serverless-operator repository

TODO:

Remove unrelated commits from the bddframework module
Upgrade CI to Golang 1.22

Signed-off-by: Spolti <[email protected]>

Bumps [github.com/docker/docker](https://github.com/docker/docker) from 23.0.1+incompatible to 23.0.3+incompatible. - [Release notes](https://github.com/docker/docker/releases) - [Commits](moby/moby@v23.0.1...v23.0.3) --- updated-dependencies: - dependency-name: github.com/docker/docker dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Bumps [github.com/docker/distribution](https://github.com/docker/distribution) from 2.8.1+incompatible to 2.8.2+incompatible. - [Release notes](https://github.com/docker/distribution/releases) - [Commits](distribution/distribution@v2.8.1...v2.8.2) --- updated-dependencies: - dependency-name: github.com/docker/distribution dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Fix Jenkinsfile.deploy * Update Jenkinsfile.deploy

* Update bump-version script * remove base images from samples * Updated release process

* Correct bump version script * Revert "[main] Update version to 2.0.0-snapshot" This reverts commit 6857bad. * Update check-container-builder.yml * Update RELEASE_NOTES.md

* [KOGITO-8644] Review applied status attribute Signed-off-by: desmax74 <[email protected]> * [KSW-Operator] Review applied status attribute Signed-off-by: desmax74 <[email protected]> --------- Signed-off-by: desmax74 <[email protected]>

apache#131) * [KOGITO-9078] [KSW-Operator] Create the cekit image descriptor for Operator image Signed-off-by: Davide Salerno <[email protected]> Co-authored-by: Filippe Spolti <[email protected]>

* [KOGITO-9106] - Add support for OpenShift BC --------- Signed-off-by: Ricardo Zanini <[email protected]>

…QUARKUS_DEVSERVICES_ENABLED properties immutable in devmode (apache#130) Signed-off-by: desmax74 <[email protected]>

…QUARKUS_DEVSERVICES_ENABLED properties immutable in devmode (apache#133)

…ch (apache#136)

Co-authored-by: Tristan Radisson <[email protected]>

* [BXMSPROD-2016] Updating references to the new kie-ci actions folder. * [BXMSPROD-2016] Updating references to the new kie-ci actions folder.

Signed-off-by: Ricardo Zanini <[email protected]>

apache#124) * [KOGITO-8648] [KSW-Operator] Implement the Knative Addressable interface in dev profile Signed-off-by: Davide Salerno <[email protected]> Update controllers/profiles/status_enricher_dev_test.go Co-authored-by: Ricardo Zanini <[email protected]> Fixing namespace into unit tests Signed-off-by: Davide Salerno <[email protected]> Update utils/kubernetes/service.go Co-authored-by: Filippe Spolti <[email protected]> Changes coming from code review Signed-off-by: Davide Salerno <[email protected]> * fix generate-all --------- Signed-off-by: Davide Salerno <[email protected]> Co-authored-by: radtriste <[email protected]>

* [KOGITO-9177] Introduce workflowproj module Signed-off-by: Ricardo Zanini <[email protected]> * Fix unit test with subresource Signed-off-by: Ricardo Zanini <[email protected]> * Incorporating reviews, adding nil check on save files Signed-off-by: Ricardo Zanini <[email protected]> * Adding filename formatting options Signed-off-by: Ricardo Zanini <[email protected]> --------- Signed-off-by: Ricardo Zanini <[email protected]>

) Signed-off-by: desmax74 <[email protected]>

* add hack/get-images-sha.sh script This script allows to fetch the sha256 of any image built in Brew. Currently it is fetching just the Operator as the sha256 of the Operator is required when building the Bundle. * check if brew command is available

…e#152) * [KOGITO-9290] Removing Owner references from generated objects Signed-off-by: Ricardo Zanini <[email protected]> * Regen code Signed-off-by: Ricardo Zanini <[email protected]> --------- Signed-off-by: Ricardo Zanini <[email protected]>

Signed-off-by: desmax74 <[email protected]>

* KOGITO-9011 Test e2e on Openshift * swf examples: add required fields

* [KOGITO-9263] Add the new resources to workflow CR Signed-off-by: Ricardo Zanini <[email protected]> * Make the volume projection algorithm easier to understand Signed-off-by: Ricardo Zanini <[email protected]> * Fix codegen Signed-off-by: Ricardo Zanini <[email protected]> * Incorporating tristan's review Signed-off-by: Ricardo Zanini <[email protected]> * Rebase tristan's pr Signed-off-by: Ricardo Zanini <[email protected]> --------- Signed-off-by: Ricardo Zanini <[email protected]>

Signed-off-by: Davide Salerno <[email protected]>

…fo from flow (apache#154) * [KOGITO-9323] - Review KogitoServerlessWorkflow api to remove meta info from flow Signed-off-by: Ricardo Zanini <[email protected]> * Regen code Signed-off-by: Ricardo Zanini <[email protected]> * Add unit tests, fix CRD specversion conversion Signed-off-by: Ricardo Zanini <[email protected]> * Fix codegen Signed-off-by: Ricardo Zanini <[email protected]> * Apply suggestions from code review --------- Signed-off-by: Ricardo Zanini <[email protected]> Co-authored-by: Tristan Radisson <[email protected]>

…le (apache#159) Signed-off-by: desmax74 <[email protected]>

…ogito events grouping by default in preview and gitops workflow deployments (apache#551)

…r knative integration (apache#558)

…d in the SonataFlow or SonataFlowPlatform CRs (apache#553)

… Service to 0 (apache#563)

…inary and compress properties for the kogito events grouping in preview and gitops workflow deployments (apache#567)

…ability health checks when Knative Eventing is configured (apache#569)

This PR updates sdk-go to v2.4.2 and also changes go 1.22.8 to 1.22.0 to avoid using patch versions which may compromise client code.

…native K_SINK injection (apache#565)

…pache#564) * Fix apache#560 - Check if the .spec.flow has changed before building Signed-off-by: Ricardo Zanini <[email protected]> * Use CRC32 instead of comparing flows directly Signed-off-by: Ricardo Zanini <[email protected]> --------- Signed-off-by: Ricardo Zanini <[email protected]>

Signed-off-by: Ricardo Zanini <[email protected]>

…o migrate-sonataflow-operator

ricardozanini · 2024-11-18T23:17:00Z

I will fix the windows problem tomorrow.

tiagobento

Thanks for the updates @ricardozanini! Good to go when green. I especially like the stats ;)

tiagobento · 2024-11-20T22:49:49Z

Mental note for not merging it before squashing the new commits.

tiagobento · 2024-11-20T22:50:07Z

BIG Mental note for NOT SQUASHING this PR when merging.

ricardozanini · 2024-11-21T00:47:10Z

DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH
DO NOT SQUASH

Signed-off-by: Ricardo Zanini <[email protected]>

ricardozanini · 2024-11-21T17:57:43Z

@wmedvede @treblereel @jakubschwan wanna take a look?

treblereel

looks good for me

packages/kn-plugin-workflow/README.md

spolti and others added 30 commits May 16, 2023 16:48

[KOGITO-9082] - [KSW-Operator] Review the Workflow CRD API (apache#118)

3760827

Signed-off-by: Spolti <[email protected]>

CI: Fix Jenkinsfile.deploy

97d85fb

* Fix Jenkinsfile.deploy * Update Jenkinsfile.deploy

Update bump-version script (apache#125)

ecde5da

* Update bump-version script * remove base images from samples * Updated release process

[main] Update version to 2.0.0-snapshot

6857bad

Correct bump version script (apache#129)

63f6d77

* Correct bump version script * Revert "[main] Update version to 2.0.0-snapshot" This reverts commit 6857bad. * Update check-container-builder.yml * Update RELEASE_NOTES.md

[KOGITO-9078] [KSW-Operator] Create the cekit image descriptor for Op… (

2cf3859

apache#131) * [KOGITO-9078] [KSW-Operator] Create the cekit image descriptor for Operator image Signed-off-by: Davide Salerno <[email protected]> Co-authored-by: Filippe Spolti <[email protected]>

[KOGITO-9106] - Add support for OpenShift BC (apache#126)

f7664c3

* [KOGITO-9106] - Add support for OpenShift BC --------- Signed-off-by: Ricardo Zanini <[email protected]>

[Kogito-9207] [Operator] Make QUARKUS_KOGITO_DEVSERVICES_ENABLED and …

06f38a0

…QUARKUS_DEVSERVICES_ENABLED properties immutable in devmode (apache#130) Signed-off-by: desmax74 <[email protected]>

KOGITO-9207 - [Operator] Make QUARKUS_KOGITO_DEVSERVICES_ENABLED and …

1a43888

…QUARKUS_DEVSERVICES_ENABLED properties immutable in devmode (apache#133)

Jenkins: Fix cekit calls (apache#134)

078cef1

update ref in osl-image-prod.yaml to the latest commit from main bran…

67aa342

…ch (apache#136)

Fix Jenkinsfile.deploy (apache#137) (apache#141)

550b82e

Co-authored-by: Tristan Radisson <[email protected]>

Updating references to the new kie-ci actions folder. (apache#96)

c8ff21c

* [BXMSPROD-2016] Updating references to the new kie-ci actions folder. * [BXMSPROD-2016] Updating references to the new kie-ci actions folder.

[KOGITO-9262] extend devmode probes threshold (apache#147)

0de169b

Signed-off-by: Ricardo Zanini <[email protected]>

KOGITO-9259 CI: RHEL8 fixes (apache#148)

0190a5f

[KOGITO-8961][KSW-Operator] Split use case and test examples (apache#139

5b0bdba

) Signed-off-by: desmax74 <[email protected]>

[KOGITO-9255]Cannot create Workflow on Openshift if is set (apache#144)

a59d720

Signed-off-by: desmax74 <[email protected]>

KOGITO-9011 Nightly testing on Openshift (apache#95)

cfdd1f6

* KOGITO-9011 Test e2e on Openshift * swf examples: add required fields

Setup Jenkinsfile (apache#155)

7bb1dd4

Fixing workflowproj module importing latest Operator apis (apache#156)

9664ebd

Signed-off-by: Davide Salerno <[email protected]>

[KOGITO-9348][OPERATOR-KSW] API version not updated in workspace modu…

0b73677

…le (apache#159) Signed-off-by: desmax74 <[email protected]>

wmedvede and others added 12 commits October 24, 2024 08:39

kie-kogito-serverless-operator-548: Make the operator configure the k…

ee8019b

…ogito events grouping by default in preview and gitops workflow deployments (apache#551)

[issue-521] Enhance error messages in SonataFlowPlatform CR status fo…

c0e6a4d

…r knative integration (apache#558)

[issue-547] Allow workflow deployment if Knative broker is not define…

957ca3d

…d in the SonataFlow or SonataFlowPlatform CRs (apache#553)

kie-kogito-serverless-operator-559: Add the ability to scale the Jobs…

ae7dde3

… Service to 0 (apache#563)

kie-kogito-serverless-operator-566: Make the operator configure the b…

5f52a11

…inary and compress properties for the kogito events grouping in preview and gitops workflow deployments (apache#567)

kie-kogito-serverless-operator-568: Disable Workflows DI and JS avail…

316e81f

…ability health checks when Knative Eventing is configured (apache#569)

update sdk-go to vesion v2.4.2 (apache#570)

6b32553

This PR updates sdk-go to v2.4.2 and also changes go 1.22.8 to 1.22.0 to avoid using patch versions which may compromise client code.

[issue-562] Serverless workflow pod gets restarted repeatedly after K…

5dae96c

…native K_SINK injection (apache#565)

Moving repository to kie-tools structure

b7f5f8c

Signed-off-by: Ricardo Zanini <[email protected]>

Remove the old sonataflow-operator version

25580f4

Signed-off-by: Ricardo Zanini <[email protected]>

Merge remote-tracking branch 'local-operator/migration-kie-tools' int…

8966b32

…o migrate-sonataflow-operator

ricardozanini requested a review from tiagobento as a code owner November 15, 2024 23:24

ricardozanini requested review from wmedvede, treblereel and jakubschwan November 15, 2024 23:34

tiagobento added area:sonataflow area:monorepo labels Nov 19, 2024

tiagobento approved these changes Nov 20, 2024

View reviewed changes

sonataflow-operator package migration

82b42be

Signed-off-by: Ricardo Zanini <[email protected]>

ricardozanini force-pushed the migrate-sonataflow-operator branch from c47048c to 82b42be Compare November 21, 2024 17:53

treblereel approved these changes Nov 22, 2024

View reviewed changes

packages/kn-plugin-workflow/README.md Show resolved Hide resolved

tiagobento merged commit eee1b79 into apache:main Nov 22, 2024
15 checks passed

ricardozanini deleted the migrate-sonataflow-operator branch November 22, 2024 21:35

fantonangeli mentioned this pull request Nov 25, 2024

Sync main branch with Apache main branch kiegroup/kie-tools#65

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[sonataflow-operator] - Migrate SonataFlow operator from incubator-kie-kogito-serverless-operator #2747

[sonataflow-operator] - Migrate SonataFlow operator from incubator-kie-kogito-serverless-operator #2747

ricardozanini commented Nov 15, 2024 •

edited

Loading

ricardozanini commented Nov 18, 2024

tiagobento left a comment

tiagobento commented Nov 20, 2024

tiagobento commented Nov 20, 2024

ricardozanini commented Nov 21, 2024

ricardozanini commented Nov 21, 2024

treblereel left a comment

[sonataflow-operator] - Migrate SonataFlow operator from incubator-kie-kogito-serverless-operator #2747

[sonataflow-operator] - Migrate SonataFlow operator from incubator-kie-kogito-serverless-operator #2747

Conversation

ricardozanini commented Nov 15, 2024 • edited Loading

ricardozanini commented Nov 18, 2024

tiagobento left a comment

Choose a reason for hiding this comment

tiagobento commented Nov 20, 2024

tiagobento commented Nov 20, 2024

ricardozanini commented Nov 21, 2024

ricardozanini commented Nov 21, 2024

treblereel left a comment

Choose a reason for hiding this comment

ricardozanini commented Nov 15, 2024 •

edited

Loading