Skip to content

Commit 43d3426

Browse files
committed
Forward Redis port when Docker host is remote
1 parent 2ae8506 commit 43d3426

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

internal/services/redis/docker.go

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ import (
1515
"time"
1616
)
1717

18+
const PORT = "6379"
19+
1820
type dockerCreator struct {
1921
logger *zap.Logger
2022
dockerClient *client.Client
@@ -55,9 +57,11 @@ func (r *dockerCreator) CreateRedisServer() services.RedisServerBase {
5557
panic(err)
5658
}
5759

60+
port := utils.NewPortDecision(r.dockerClient, PORT)
61+
5862
cont, err := r.dockerClient.ContainerCreate(context.Background(), &container.Config{
5963
Image: dockerImage,
60-
}, nil, &network.NetworkingConfig{
64+
}, &container.HostConfig{PortBindings: port.Map()}, &network.NetworkingConfig{
6165
EndpointsConfig: map[string]*network.EndpointSettings{
6266
networkName: {
6367
NetworkID: r.dockerNetworkId,
@@ -88,8 +92,8 @@ func (r *dockerCreator) CreateRedisServer() services.RedisServerBase {
8892

8993
s := &dockerServer{
9094
info: info{
91-
host: utils.MustString(utils.DockerContainerAddress(context.Background(), r.dockerClient, cont.ID)),
92-
port: "6379",
95+
host: port.Address(context.Background(), r.dockerClient, cont.ID),
96+
port: port.Port(),
9397
},
9498
redisDocker: r,
9599
logger: logger,

0 commit comments

Comments
 (0)