-
Notifications
You must be signed in to change notification settings - Fork 572
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
[JBPM-9178] Immutable SpringBoot Deployment / Adding controller conne… #2220
Conversation
depends on kiegroup/jbpm#1749 |
Jenkins retest this please. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some minor changes required.
I would try to cover these scenarios adding some test cases, as test coverage is quite low here.
...er-spring-boot-autoconfiguration/src/main/java/org/kie/server/springboot/EmbeddedKieJar.java
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Outdated
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Outdated
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Show resolved
Hide resolved
...ces-common/src/main/java/org/kie/server/services/impl/InmutableContainerStartupStrategy.java
Outdated
Show resolved
Hide resolved
...ices-common/src/main/resources/META-INF/services/org.kie.server.services.api.StartupStrategy
Show resolved
Hide resolved
.../main/java/org/kie/server/services/impl/storage/memory/InMemoryKieServerStateRepository.java
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Outdated
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Outdated
Show resolved
Hide resolved
d304d7b
to
4d13074
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just some previous comment not resolved yet
import org.kie.server.services.impl.storage.KieServerState; | ||
import org.slf4j.Logger; | ||
import org.slf4j.LoggerFactory; | ||
|
||
|
||
public class InmutableContainerStartupStrategy implements StartupStrategy { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this not resolved yet
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Show resolved
Hide resolved
...on/src/main/java/org/kie/server/springboot/autoconfiguration/KieServerAutoConfiguration.java
Show resolved
Hide resolved
4d13074
to
e99a279
Compare
…ction capabilities
e99a279
to
223415e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Amazing job! very easy and intuitive to use. well done
SonarCloud Quality Gate failed. 0 Bugs 3.4% Coverage The version of Java (1.8.0_202) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11. |
under test.