+1 Il giorno gio 2 mag 2024 alle ore 10:57 Yeser Amer <ya...@apache.org> ha scritto:
> + 1 > I suggest opening a ticket to solve those tests before the merge, to keep > track of it and don't forget it :) > > On 2024/04/30 14:25:38 Tibor Zimányi wrote: > > Hi everyone, > > > > this is a status update about the Quarkus upgrade, that is in progress. > > There are PRs open, nearly all PR checks are green, except Jenkins job on > > kogito-apps failing on some tests (1). The same PR check is green for > > multiple runs in GitHub Actions. After some investigation, these tests > look > > like flaky tests. See e.g. test (2). It is reported failing to start a > > Keycloak container. When I run it locally, it fails on not being able to > > connect to a PostgreSQL database. When I add a Thread.sleep(5000) here > (3), > > the test passes for me. Similar for other tests. > > > > So I would like to propose to merge the PRs and eventually fix/remove the > > tests in a separate PR. > > > > After we have the PRs merged, we need a new timed snapshot of the > > libraries, so Quarkus can be updated in the kie-tools repository. > > > > Best regards, > > Tibor > > > > (1) https://github.com/apache/incubator-kie-kogito-apps/pull/2039 > > (2) > > > org.kie.kogito.index.postgresql.reporting.ProcessInstanceVariableMappingIT.testProcessInstanceVariableMapping > > (3) > > > https://github.com/apache/incubator-kie-kogito-runtimes/blob/402e4cc6e015a679cf81025db579a1ec77839448/kogito-test-utils/src/main/java/org/kie/kogito/testcontainers/KogitoPostgreSqlContainer.java#L52 > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@kie.apache.org > For additional commands, e-mail: dev-h...@kie.apache.org > >
