-
Notifications
You must be signed in to change notification settings - Fork 145
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
[Fix #2177] Merge ProcessDefinitionEvent metadata #2178
Conversation
PR job Reproducerbuild-chain build full_downstream -f 'https://raw.githubusercontent.com/${AUTHOR:apache}/incubator-kie-kogito-pipelines/${BRANCH:main}/.ci/buildchain-config-pr-cdb.yaml' -o 'bc' -p apache/incubator-kie-kogito-apps -u #2178 --skipParallelCheckout NOTE: To install the build-chain tool, please refer to https://github.com/kiegroup/github-action-build-chain#local-execution Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/main/job/pullrequest_jobs/job/kogito-apps-pr/job/PR-2178/3/display/redirect Test results:
Those are the test failures: org.kie.kogito.index.inmemory.KogitoDevServiceTest.testDataIndexDevServiceio.quarkus.builder.BuildException: Build failure: Build failed due to errors[error]: Build step org.kie.kogito.quarkus.serverless.workflow.deployment.ServerlessWorkflowDevServicesProcessor#startDataIndexDevService threw an exception: java.lang.RuntimeException: Failed to start Kogito Data Index Dev Services at org.kie.kogito.quarkus.workflow.deployment.AbstractDevServicesProcessor.startDataIndexDevService(AbstractDevServicesProcessor.java:129) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) at java.base/java.lang.reflect.Method.invoke(Method.java:568) at io.quarkus.deployment.ExtensionLoader$3.execute(ExtensionLoader.java:849) at io.quarkus.builder.BuildContext.run(BuildContext.java:256) at org.jboss.threads.ContextHandler$1.runWith(ContextHandler.java:18) at org.jboss.threads.EnhancedQueueExecutor$Task.run(EnhancedQueueExecutor.java:2513) at org.jboss.threads.EnhancedQueueExecutor$ThreadBody.run(EnhancedQueueExecutor.java:1538) at java.base/java.lang.Thread.run(Thread.java:840) at org.jboss.threads.JBossThread.run(JBossThread.java:501) Caused by: java.lang.RuntimeException: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image org.kie.kogito/data-index-service-inmemory:999-SNAPSHOT at org.kie.kogito.quarkus.workflow.deployment.AbstractDevServicesProcessor.lambda$startDataIndex$2(AbstractDevServicesProcessor.java:203) at java.base/java.util.Optional.orElseGet(Optional.java:364) at org.kie.kogito.quarkus.workflow.deployment.AbstractDevServicesProcessor.startDataIndex(AbstractDevServicesProcessor.java:208) at org.kie.kogito.quarkus.workflow.deployment.AbstractDevServicesProcessor.startDataIndexDevService(AbstractDevServicesProcessor.java:119) ... 11 more Caused by: org.testcontainers.containers.ContainerLaunchException: Container startup failed for image org.kie.kogito/data-index-service-inmemory:999-SNAPSHOT at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:359) at org.testcontainers.containers.GenericContainer.start(GenericContainer.java:330) at org.kie.kogito.quarkus.workflow.deployment.AbstractDevServicesProcessor.lambda$startDataIndex$2(AbstractDevServicesProcessor.java:199) ... 14 more Caused by: org.rnorth.ducttape.RetryCountExceededException: Retry limit hit with exception at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:88) at org.testcontainers.containers.GenericContainer.doStart(GenericContainer.java:344) ... 16 more Caused by: org.testcontainers.containers.ContainerLaunchException: Could not create/start container at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:563) at org.testcontainers.containers.GenericContainer.lambda$doStart$0(GenericContainer.java:354) at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:81) ... 17 more Caused by: java.lang.IllegalStateException: Wait strategy failed. Container exited with code 1 at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:533) ... 19 more Caused by: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for URL to be accessible (http://172.18.0.1:55296/q/health/ready should return HTTP [200]) at org.testcontainers.containers.wait.strategy.HttpWaitStrategy.waitUntilReady(HttpWaitStrategy.java:320) at org.testcontainers.containers.wait.strategy.AbstractWaitStrategy.waitUntilReady(AbstractWaitStrategy.java:52) at org.testcontainers.containers.GenericContainer.waitUntilContainerStarted(GenericContainer.java:909) at org.testcontainers.containers.GenericContainer.tryStart(GenericContainer.java:500) ... 19 more Caused by: org.rnorth.ducttape.TimeoutException: Timeout waiting for result with exception at org.rnorth.ducttape.unreliables.Unreliables.retryUntilSuccess(Unreliables.java:54) at org.testcontainers.containers.wait.strategy.HttpWaitStrategy.waitUntilReady(HttpWaitStrategy.java:252) ... 22 more Caused by: java.lang.RuntimeException: java.net.ConnectException: Connection refused at org.testcontainers.containers.wait.strategy.HttpWaitStrategy.lambda$null$6(HttpWaitStrategy.java:312) at org.rnorth.ducttape.ratelimits.RateLimiter.doWhenReady(RateLimiter.java:27) at org.testcontainers.containers.wait.strategy.HttpWaitStrategy.lambda$waitUntilReady$7(HttpWaitStrategy.java:257) at org.rnorth.ducttape.unreliables.Unreliables.lambda$retryUntilSuccess$0(Unreliables.java:43) at java.base/java.util.concurrent.FutureTask.run(FutureTask.java:264) at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) at java.base/java.lang.Thread.run(Thread.java:840) Caused by: java.net.ConnectException: Connection refused at java.base/sun.nio.ch.Net.connect0(Native Method) at java.base/sun.nio.ch.Net.connect(Net.java:579) at java.base/sun.nio.ch.Net.connect(Net.java:568) at java.base/sun.nio.ch.NioSocketImpl.connect(NioSocketImpl.java:593) at java.base/java.net.Socket.connect(Socket.java:633) at java.base/java.net.Socket.connect(Socket.java:583) at java.base/sun.net.NetworkClient.doConnect(NetworkClient.java:183) at java.base/sun.net.www.http.HttpClient.openServer(HttpClient.java:533) at java.base/sun.net.www.http.HttpClient.openServer(HttpClient.java:638) at java.base/sun.net.www.http.HttpClient.(HttpClient.java:281) at java.base/sun.net.www.http.HttpClient.New(HttpClient.java:386) at java.base/sun.net.www.http.HttpClient.New(HttpClient.java:408) at java.base/sun.net.www.protocol.http.HttpURLConnection.getNewHttpClient(HttpURLConnection.java:1309) at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect0(HttpURLConnection.java:1242) at java.base/sun.net.www.protocol.http.HttpURLConnection.plainConnect(HttpURLConnection.java:1128) at java.base/sun.net.www.protocol.http.HttpURLConnection.connect(HttpURLConnection.java:1057) at org.testcontainers.containers.wait.strategy.HttpWaitStrategy.lambda$null$6(HttpWaitStrategy.java:274) ... 7 more org.kie.kogito.index.inmemory.ProcessDataIndexInMemoryIT.testProcessInstanceEventsjava.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.index.test.quarkus.DataIndexInMemoryQuarkusTestResourceorg.kie.kogito.index.postgresql.ProcessDataIndexPostgreSqlHttpIT.testProcessInstanceEventsjava.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.index.test.quarkus.http.DataIndexPostgreSqlHttpQuarkusTestResourceorg.kie.kogito.index.service.InmemoryPostgreSqlIndexingServiceIT.(?)java.lang.ExceptionInInitializerErrororg.kie.kogito.index.service.KeycloakInmemoryPostgreSqlIndexingServiceIT.testUnauthorizedUserAccessjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.service.KeycloakPostgreSqlIndexingServiceIT.testUnauthorizedUserAccessjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.service.PostgreSqlIndexingServiceIT.(?)java.lang.ExceptionInInitializerErrororg.kie.kogito.index.service.messaging.PostgreSqlMessagingKafkaConsumerIT.testProcessInstanceEventjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.mapper.JobEntityMapperIT.testMapToModeljava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.H2JobEntityQueryIT.testJobQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.H2ProcessDefinitionEntityQueryIT.testProcessDefinitionQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.H2ProcessInstanceEntityQueryIT.testProcessInstanceQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.H2UserTaskInstanceEntityQueryIT.testUserTaskInstanceQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.PostgreSQLJobEntityQueryIT.testJobQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.PostgreSQLProcessDefinitionEntityQueryIT.testProcessDefinitionQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.PostgreSQLProcessInstanceEntityQueryIT.testProcessInstanceQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.query.PostgreSQLUserTaskInstanceEntityQueryIT.testUserTaskInstanceQueryjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.storage.H2JobStorageIT.testJobEntityjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.storage.PostgreSQLJobStorageIT.testJobEntityjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.storage.PostgreSQLProcessDefinitionStorageIT.testProcessDefinitionEntityjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.storage.PostgreSQLProcessInstanceStorageIT.testProcessInstanceErrorEventjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.jdbc.storage.PostgreSQLUserTaskInstanceStorageIT.testUserTaskInstanceEntityjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.postgresql.reporting.ProcessInstanceVariableMappingIT.testProcessInstanceVariableMappingjava.lang.ExceptionInInitializerErrororg.kie.kogito.index.postgresql.mapper.JobEntityMapperIT.testMapToModeljava.lang.ExceptionInInitializerErrorPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.InfinispanQuarkusAddonDataIndexPersistenceIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for detailsPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.JPAQuarkusAddonDataIndexPersistenceTest.testDataIndexAddonjava.lang.ExceptionInInitializerErrorPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.JPAQuarkusAddonDataIndexPersistenceTest.testDataIndexAddonjava.lang.ExceptionInInitializerErrorPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.PostgreSQLQuarkusAddonDataIndexPersistenceIT.testDataIndexAddonjava.util.concurrent.CompletionException: java.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.index.test.quarkus.http.DataIndexPostgreSqlHttpQuarkusTestResourcePR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.PostgreSQLQuarkusAddonDataIndexPersistenceIT.testDataIndexAddonjava.lang.RuntimeException: Unable to start Quarkus test resource class org.kie.kogito.index.test.quarkus.http.DataIndexPostgreSqlHttpQuarkusTestResourcePR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.InfinispanQuarkusAddonDataIndexIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for detailsPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.InMemoryQuarkusAddonDataIndexIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for detailsPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.InMemoryQuarkusAddonDataIndexIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for detailsPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.JPAQuarkusAddonDataIndexIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for detailsPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.JPAQuarkusAddonDataIndexIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for detailsPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.PostgreSQLQuarkusAddonDataIndexIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for detailsPR check / Build projects / org.kie.kogito.addons.quarkus.data.index.it.PostgreSQLQuarkusAddonDataIndexIT.testDataIndexAddonjava.lang.IllegalStateException: Unable to determine the status of the running process. See the above logs for details |
52aa88c
to
a33ccfd
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.
Looks good to me, just a single comment to document merges with null value.
Feel free to incorporate these tests for better coverage fjtirado#2
Great work @fjtirado !
data-index/data-index-common/src/main/java/org/kie/kogito/index/CommonUtils.java
Show resolved
Hide resolved
* [Fix apache#2177] Merge metadata in ProcessEventDefinition Fix apache#2177 * [Fix apache#2177] Partial rollback
Fix #2177