apache / incubator-kie-kogito-serverless-operator

Kubernetes operator for SonataFlow
Apache License 2.0
17 stars 42 forks source link

Add sonataflow postgres db migrator tool for data index and jobs service #517

Closed rhkp closed 2 months ago

rhkp commented 3 months ago

Closes #516 : Add data index/jobs service postgres database migrator tool

Description of the change:

Motivation for the change:

Checklist

How to backport a pull request to a different branch? In order to automatically create a **backporting pull request** please add one or more labels having the following format `backport-`, where `` is the name of the branch where the pull request must be backported to (e.g., `backport-7.67.x` to backport the original PR to the `7.67.x` branch). > **NOTE**: **backporting** is an action aiming to move a change (usually a commit) from a branch (usually the main one) to another one, which is generally referring to a still maintained release branch. Keeping it simple: it is about to move a specific change or a set of them from one branch to another. Once the original pull request is successfully merged, the automated action will create one backporting pull request per each label (with the previous format) that has been added. If something goes wrong, the author will be notified and at this point a manual backporting is needed. > **NOTE**: this automated backporting is triggered whenever a pull request on `main` branch is labeled or closed, but both conditions must be satisfied to get the new PR created.
rhkp commented 3 months ago

Great work! Thanks for the efforts so far. Can you please also add unit tests to the application and create a GitHub action in a separate file that runs only when there's a change in this directory to run the tests?

Hi @ricardozanini I think all comments have been addressed now. Please advise of any more changes or please acknowledge and merge. Thanks.

ricardozanini commented 2 months ago

@rhkp you might need to rebase your branch.

rhkp commented 2 months ago

@jakubschwan @ricardozanini Not sure are there any more comments? If not can we please approve/merge?