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
DBZ-2193 Investigate the replacement of LB services with port redirects in OpenShift test-suite #3774
Conversation
Good job as a PoC. Two things which I would like to see
|
...em/src/main/java/io/debezium/testing/system/tools/databases/AbstractOcpDatabaseDeployer.java
Outdated
Show resolved
Hide resolved
...em/src/main/java/io/debezium/testing/system/tools/databases/AbstractOcpDatabaseDeployer.java
Outdated
Show resolved
Hide resolved
.../src/main/java/io/debezium/testing/system/tools/databases/AbstractOcpDatabaseController.java
Outdated
Show resolved
Hide resolved
cb0a77e
to
646e699
Compare
7cd28e9
to
7cd43b9
Compare
@MartinMedek pleas rebase this. It's about time we get this merged. Apology for the delay. |
7cd43b9
to
a2851c7
Compare
279d875
to
b830da9
Compare
b830da9
to
5d41413
Compare
5d41413
to
82475b1
Compare
@obabec should be good to merge once @MartinMedek resolves issues with DB2 |
@jcechace Martin will also add kafka external listener with route and ssl to this PR. |
5ed5984
to
5a687c7
Compare
...ium-testing-system/src/main/java/io/debezium/testing/system/tools/kafka/KafkaController.java
Outdated
Show resolved
Hide resolved
...-testing-system/src/main/java/io/debezium/testing/system/tools/kafka/OcpKafkaController.java
Outdated
Show resolved
Hide resolved
...zium-testing-system/src/test/java/io/debezium/testing/system/fixtures/kafka/DockerKafka.java
Outdated
Show resolved
Hide resolved
...bezium-testing-system/src/test/java/io/debezium/testing/system/tests/MongoConnectorTest.java
Outdated
Show resolved
Hide resolved
.../src/main/java/io/debezium/testing/system/tools/databases/AbstractOcpDatabaseController.java
Show resolved
Hide resolved
096cbc2
to
6efd76b
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.
LGTM
6efd76b
to
3d52d26
Compare
…alanced services to access DBs
…r, moved the methods to be called explicitly in test setup and cleanup Fixed mongo parameters in ConnectorFactories
…local port is now chosen as first free random port.
3d52d26
to
1ca290c
Compare
…dant test classes, refactor fixed default sqlserver test parameters in pom
1ca290c
to
f71762e
Compare
No description provided.