Skip to content

[Backport] move integration tests from ZooKeeper 3.4.x to 3.5.x#10829

Merged
jihoonson merged 1 commit intoapache:0.21.0from
jihoonson:backport-10786-to-0.21.0
Feb 3, 2021
Merged

[Backport] move integration tests from ZooKeeper 3.4.x to 3.5.x#10829
jihoonson merged 1 commit intoapache:0.21.0from
jihoonson:backport-10786-to-0.21.0

Conversation

@jihoonson
Copy link
Copy Markdown
Contributor

Backport of #10786 to 0.21.0.

* move integration tests from ZooKeeper 3.4.x to 3.5.x
* run a subset of our integration tests with ZK 3.4 for backwards compatibility testing.
* remove need to build separate docker-base image
- use multi-stage build for the base image
- use openjdk base image instead of building our own JDK base
- workaround Debian not including MySQL by using MariaDB
- download mysql connector directly instead of using distro version
* fix incorrect openssl command failing on Debian
* keep mysql connector version in sync with pom version
@jihoonson jihoonson added this to the 0.21.0 milestone Feb 2, 2021
@jihoonson
Copy link
Copy Markdown
Contributor Author

Generating report for /home/travis/build/apache/druid
Generating report for /home/travis/build/apache/druid/extensions-core/kafka-indexing-service
Encountered error [Command 'mvn -Ddependency.locations.enabled=false -Ddependency.details.enabled=false project-info-reports:dependencies' returned non-zero exit status 1] when generating report for /home/travis/build/apache/druid/extensions-core/kafka-indexing-service
Generating report for /home/travis/build/apache/druid/extensions-core/kubernetes-extensions
Encountered error [Command 'mvn -Ddependency.locations.enabled=false -Ddependency.details.enabled=false project-info-reports:dependencies' returned non-zero exit status 1] when generating report for /home/travis/build/apache/druid
...

I can't understand this license check failure. I ran the same test manually and it passed. I will merge this PR shortly.

@jihoonson jihoonson merged commit 1c3928f into apache:0.21.0 Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants