nicoloboschi opened a new pull request #2937: URL: https://github.com/apache/bookkeeper/pull/2937
### Motivation Integration tests under `backward-compat` do not work neither with Gradle and Maven. For Maven, they are not triggered at all For Gradle, they run and fail because in Gradle the docker images used inside the tests are not built at all. ### Changes * Build docker images under `tests/docker-images` * Fix some tests failing due to BookieId migration * Fix `tests/backward-compat/upgrade/src/test/groovy/org/apache/bookkeeper/tests/backwardcompat/TestCompatUpgrade.groovy` test which needs tests running in a specific order * Fix `MavenClassLoader`removing the need to have the current SNAPSHOT version jars in the local repository. Now it uses the distribution tar.gz * Add these tests in GH actions Some important notes: * I haven't found the date where tests were not being run anymore but I think is more than 1 year ago, at least since BP-41 * I kept Arquillian Cube as docker container runner since there's no real need to migrate to testcontainers * **These tests only work on Linux** because they relies on `Host` docker network which is only available on Linux * I haven't fixed Maven execution since we are already using Gradle in CI Master Issue: #2930 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
