Skip to content

Commit

Permalink
Addressing review comments, remove explicit exposed port for containers
Browse files Browse the repository at this point in the history
Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
  • Loading branch information
Artur Ciocanu committed Aug 29, 2024
1 parent 57ac962 commit b5824bc
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ public class DaprKeyValueRepositoryIT {
.withDatabaseName("dapr_db_repository")
.withUsername("postgres")
.withPassword("password")
.withExposedPorts(5432)
.withNetwork(DAPR_NETWORK);

@Container
Expand All @@ -77,7 +76,6 @@ public class DaprKeyValueRepositoryIT {

@DynamicPropertySource
static void daprProperties(DynamicPropertyRegistry registry) {
DAPR_CONTAINER.start();
registry.add("dapr.grpc.port", DAPR_CONTAINER::getGrpcPort);
registry.add("dapr.http.port", DAPR_CONTAINER::getHttpPort);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,6 @@ public class MySQLDaprKeyValueTemplateIT {
.withDatabaseName("dapr_db")
.withUsername("mysql")
.withPassword("password")
.withExposedPorts(3306)
.withNetwork(DAPR_NETWORK)
.waitingFor(MYSQL_WAIT_STRATEGY);

Expand All @@ -92,7 +91,6 @@ public class MySQLDaprKeyValueTemplateIT {

@DynamicPropertySource
static void daprProperties(DynamicPropertyRegistry registry) {
DAPR_CONTAINER.start();
registry.add("dapr.grpc.port", DAPR_CONTAINER::getGrpcPort);
registry.add("dapr.http.port", DAPR_CONTAINER::getHttpPort);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,6 @@ public class PostgreSQLDaprKeyValueTemplateIT {
.withDatabaseName("dapr_db")
.withUsername("postgres")
.withPassword("password")
.withExposedPorts(5432)
.withNetwork(DAPR_NETWORK);

@Container
Expand All @@ -78,7 +77,6 @@ public class PostgreSQLDaprKeyValueTemplateIT {

@DynamicPropertySource
static void daprProperties(DynamicPropertyRegistry registry) {
DAPR_CONTAINER.start();
registry.add("dapr.grpc.port", DAPR_CONTAINER::getGrpcPort);
registry.add("dapr.http.port", DAPR_CONTAINER::getHttpPort);
}
Expand Down

0 comments on commit b5824bc

Please sign in to comment.