Skip to content

Commit

Permalink
Merge pull request #15585 from rogeriomm/master-fix-registry
Browse files Browse the repository at this point in the history
Fix addon registry doesn't follow Minikube DNS domain name configuration (--dns-domain)
  • Loading branch information
medyagh authored Sep 6, 2023
2 parents ccbd6cf + 431a296 commit 3276aa7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
2 changes: 1 addition & 1 deletion deploy/addons/registry/registry-proxy.yaml.tmpl
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,6 @@ spec:
hostPort: 5000
env:
- name: REGISTRY_HOST
value: registry.kube-system.svc.cluster.local
value: registry.kube-system.svc.{{.NetworkInfo.DNSDomain}}
- name: REGISTRY_PORT
value: "80"
1 change: 1 addition & 0 deletions pkg/minikube/assets/addons.go
Original file line number Diff line number Diff line change
Expand Up @@ -935,6 +935,7 @@ func GenerateTemplateData(addon *Addon, cc *config.ClusterConfig, netInfo Networ
// Network info for generating template
opts.NetworkInfo["ControlPlaneNodeIP"] = netInfo.ControlPlaneNodeIP
opts.NetworkInfo["ControlPlaneNodePort"] = fmt.Sprint(netInfo.ControlPlaneNodePort)
opts.NetworkInfo["DNSDomain"] = cfg.DNSDomain

// Append postfix "/" to registries
for k, v := range opts.Registries {
Expand Down

0 comments on commit 3276aa7

Please sign in to comment.