Skip to content

Commit

Permalink
Merge pull request #5104 from patrickdillon/azurestack-private-zone
Browse files Browse the repository at this point in the history
pkg/asset/manifests/dns: don't create private zone in Azure Stack
  • Loading branch information
openshift-merge-robot committed Jul 23, 2021
2 parents 8593c8e + 298c750 commit e3301dc
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions pkg/asset/manifests/dns.go
Expand Up @@ -113,8 +113,10 @@ func (d *DNS) Generate(dependencies asset.Parents) error {
ID: dnsConfig.GetDNSZoneID(installConfig.Config.Azure.BaseDomainResourceGroupName, installConfig.Config.BaseDomain),
}
}
config.Spec.PrivateZone = &configv1.DNSZone{
ID: dnsConfig.GetPrivateDNSZoneID(installConfig.Config.Azure.ClusterResourceGroupName(clusterID.InfraID), installConfig.Config.ClusterDomain()),
if installConfig.Azure.CloudName != azuretypes.StackCloud {
config.Spec.PrivateZone = &configv1.DNSZone{
ID: dnsConfig.GetPrivateDNSZoneID(installConfig.Config.Azure.ClusterResourceGroupName(clusterID.InfraID), installConfig.Config.ClusterDomain()),
}
}
case gcptypes.Name:
if installConfig.Config.Publish == types.ExternalPublishingStrategy {
Expand Down

0 comments on commit e3301dc

Please sign in to comment.