Enable AKS to use Outbound type of userDefinedRouting #163
Conversation
|
@alexander-demicev kindly review |
| @@ -97,7 +106,7 @@ func createManagedCluster(ctx context.Context, cred *Credentials, workplacesClie | |||
| logrus.Warnf("loadBalancerSKU 'basic' is not supported") | |||
| networkProfile.LoadBalancerSku = containerservice.Basic | |||
| case "": | |||
| networkProfile.LoadBalancerSku = containerservice.Standard | |||
| networkProfile.LoadBalancerSku = "" | |||
There was a problem hiding this comment.
No major comment, but curious why we need to remove this?
There was a problem hiding this comment.
@furkatgofurov7 ,
Operator creates a standard loadbalancer (egress) by default, by removing this, I will not have egress created by uses pre-defined route table for egress.
There was a problem hiding this comment.
I am not sure about that, but I am going to test it
|
@mjura you're the most experienced, please also take a look. |
| @@ -97,7 +106,7 @@ func createManagedCluster(ctx context.Context, cred *Credentials, workplacesClie | |||
| logrus.Warnf("loadBalancerSKU 'basic' is not supported") | |||
| networkProfile.LoadBalancerSku = containerservice.Basic | |||
| case "": | |||
| networkProfile.LoadBalancerSku = containerservice.Standard | |||
| networkProfile.LoadBalancerSku = "" | |||
There was a problem hiding this comment.
I am not sure about that, but I am going to test it
|
Testing in progress .... |
|
After some few test runs I see that example-configuration isn't complete: |
| spec: | ||
| resourceLocation: "germanywestcentral" | ||
| resourceGroup: "my-group" | ||
| clusterName: "my-cluster" |
There was a problem hiding this comment.
it is missing dnsPrefix: "example-string"
|
Will cherry-pick and add fixes |
|
We will continue there #189 |
|
PR #189 was merged |
With UDR setup, there is no need to provision loadbalancer for
egresscommunication. Egress will be routed to existing route table in the VNET/Subnet.This will be useful in the setup where creation of public loadbalancer is prohibited.