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

Review ProcessService inject into management addon to avoid UnsatisfiedResolutionException when a project doesn't have a bpmn/serverless workflow file #3473

Closed
ricardozanini opened this issue Apr 12, 2024 · 3 comments · Fixed by #3474
Assignees

Comments

@ricardozanini
Copy link
Member

          Folks, this PR broke the SonataFlow builder/devmode images:
2024-04-11 19:38:43,504 3161eb1b5bd6 ERROR [io.quarkus.deployment.dev.IsolatedDevModeMain:156] (main) Failed to start quarkus: java.lang.RuntimeException: io.quarkus.builder.BuildException: Build failure: Build failed due to errors
        [error]: Build step io.quarkus.arc.deployment.ArcProcessor#validate threw an exception: jakarta.enterprise.inject.spi.DeploymentException: jakarta.enterprise.inject.UnsatisfiedResolutionException: Unsatisfied dependency for type org.kie.kogito.process.ProcessService and qualifiers [@Default]
        - java member: org.kie.kogito.process.management.ProcessInstanceManagementResource#processService
        - declared on CLASS bean [types=[java.lang.Object, org.kie.kogito.process.management.ProcessInstanceManagementResource], qualifiers=[@Default, @Any], target=org.kie.kogito.process.management.ProcessInstanceManagementResource]

The culprit is this change: quarkus/addons/process-management/runtime/src/main/java/org/kie/kogito/process/management/ProcessInstanceManagementResource.java

The ProcessService bean is not available. Since these images start with an empty project, I think this injection dependency is not found. I might be wrong, I haven't dug enough.

Can we make this injection optional?

Originally posted by @ricardozanini in #3403 (comment)

@ricardozanini
Copy link
Member Author

See the proposal implementation here: #3403 (comment)

@ricardozanini
Copy link
Member Author

@elguardian why did you reopen it? Is the bug stills there? 🤔

@elguardian
Copy link
Contributor

@ricardozanini There are more things linked to this and I don't know the context; I am not the owner so I leave to your criteria to close it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants