apache / pulsar-client-go

Apache Pulsar Go Client Library
https://pulsar.apache.org/
Apache License 2.0
659 stars 336 forks source link

[Issue 1282][test] use pulsar-client-go-test docker to test TestBlueGreenMigrationTestSuite case #1287

Closed geniusjoe closed 1 month ago

geniusjoe commented 1 month ago

Contribution Checklist

Issue #1282

Motivation

Use pulsar-client-go-test docker to test TestBlueGreenMigrationTestSuite case, so that we can get rid of local environment go pkg dependencies.

Modifications

  1. Use integration-tests/extensible-load-manager/docker-compose.yml docker network in integration-tests/blue-green/docker-compose.yml green cluster, to make sure two clusters are in the same network environment.
  2. Use new unique service names in integration-tests/blue-green/docker-compose.yml green cluster to avoid service name duplication.
  3. Use service name instead of service forwarding port in pulsar/blue_green_migration_test.go.

Verifying this change

This change added tests and can be verified as follows:

Does this pull request potentially affect one of the following parts:

If yes was chosen, please highlight the changes

Documentation

geniusjoe commented 1 month ago

@nodece Thanks for reviewing this pr, could you help me to merge this pr as well? 😂