Skip to content

Commit

Permalink
fix: registry deployment debug port should match the spec
Browse files Browse the repository at this point in the history
Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com>
  • Loading branch information
sagikazarmark authored and bitsf committed Apr 21, 2022
1 parent 95a56f3 commit 75575ac
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions controllers/goharbor/registry/deployments.go
Expand Up @@ -52,8 +52,7 @@ var (
)

const (
apiPort = 5000 // https://github.com/docker/distribution/blob/749f6afb4572201e3c37325d0ffedb6f32be8950/contrib/compose/docker-compose.yml#L15
metricsPort = 5001 // https://github.com/docker/distribution/blob/b12bd4004afc203f1cbd2072317c8fda30b89710/cmd/registry/config-dev.yml#L34
apiPort = 5000 // https://github.com/docker/distribution/blob/749f6afb4572201e3c37325d0ffedb6f32be8950/contrib/compose/docker-compose.yml#L15
// registry controller port.
httpsPort = 8443
httpPort = 8080
Expand Down Expand Up @@ -298,7 +297,7 @@ func (r *Reconciler) GetDeployment(ctx context.Context, registry *goharborv1.Reg
Name: harbormetav1.RegistryAPIPortName,
Protocol: corev1.ProtocolTCP,
}, {
ContainerPort: metricsPort,
ContainerPort: registry.Spec.HTTP.Debug.Port,
Name: harbormetav1.RegistryMetricsPortName,
Protocol: corev1.ProtocolTCP,
}},
Expand Down

0 comments on commit 75575ac

Please sign in to comment.