Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EPIC - Support deploy/undeploy SonataFlow Flow definitions events on Data Index #1201

Open
2 tasks
ricardozanini opened this issue May 13, 2024 · 5 comments
Open
2 tasks
Assignees
Labels
area:sonataflow Related to CNCF Serverless Workflow Spec and SonataFlow type:new-feature Something is missing, we need to build it!

Comments

@ricardozanini
Copy link
Member

ricardozanini commented May 13, 2024

Issues in this epic:

Originally posted by @fjtirado in apache/incubator-kie-kogito-serverless-operator#458 (comment)

@ricardozanini ricardozanini self-assigned this May 13, 2024
@ricardozanini ricardozanini added the area:sonataflow Related to CNCF Serverless Workflow Spec and SonataFlow label May 13, 2024
@ricardozanini
Copy link
Member Author

@fjtirado @nmirasch can you please open the mentioned issues? 🙏

@ricardozanini ricardozanini changed the title EPIC - Support deploy/undeploy SonataFlow definition events on Data Index EPIC - Support deploy/undeploy SonataFlow Flow definitions events on Data Index May 13, 2024
@nmirasch
Copy link

@ricardozanini here it is the dataindex related one apache/incubator-kie-kogito-apps#2050

@fjtirado
Copy link

fjtirado commented May 14, 2024

After some discussion, we agree that the simplest approach is to add a new event topic (to not interfere with audit component), which events will contain the id and version of the workflow plus the "deployment status".
Data Index will consume that information and merge it into the process definition existing structure (therefore the result will be that the existing process definition in data index will include the deployment status as result of the queries)
With this approach, we do not need any issue for runtime.
In my original comment, I forgot to mention that we need an issue for tooling (for dealing with the new "deployment status" field)

@ricardozanini
Copy link
Member Author

@paulovmr @caponetto can you assess?

In my original comment, I forgot to mention that we need an issue for tooling (for dealing with the new "deployment status" field)

We can add a task in this EPIC.

@caponetto
Copy link

IIUC, on the tooling side, we can filter out data by the new deployment status field on the queries. This way, we maintain the behavior of showing only those deployments that are indeed alive.

@wmedvede wmedvede assigned wmedvede and unassigned ricardozanini Jun 14, 2024
@wmedvede wmedvede added the type:new-feature Something is missing, we need to build it! label Jun 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:sonataflow Related to CNCF Serverless Workflow Spec and SonataFlow type:new-feature Something is missing, we need to build it!
Projects
Status: 📋 Backlog
Development

No branches or pull requests

5 participants