Skip to content

Commit

Permalink
fix lb tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Cecile Robert-Michon committed Oct 10, 2020
1 parent 74254f3 commit 286fd9a
Show file tree
Hide file tree
Showing 4 changed files with 128 additions and 246 deletions.
1 change: 0 additions & 1 deletion cloud/scope/cluster.go
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,6 @@ func (s *ClusterScope) LBSpecs() []azure.LBSpec {
// Control Plane LB
Name: s.APIServerLB().Name,
SubnetName: s.ControlPlaneSubnet().Name,
SubnetCidrs: s.ControlPlaneSubnet().CIDRBlocks,
FrontendIPConfigs: s.APIServerLB().FrontendIPs,
APIServerPort: s.APIServerPort(),
Type: s.APIServerLB().Type,
Expand Down
2 changes: 1 addition & 1 deletion cloud/services/loadbalancers/loadbalancers.go
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ func (s *Service) Reconcile(ctx context.Context) error {
}

lb := network.LoadBalancer{
Name: to.StringPtr(lbSpec.Name),
Sku: &network.LoadBalancerSku{Name: network.LoadBalancerSkuNameStandard},
Location: to.StringPtr(s.Scope.Location()),
Tags: converters.TagsToMap(infrav1.Build(infrav1.BuildParams{
Expand Down Expand Up @@ -110,6 +109,7 @@ func (s *Service) Reconcile(ctx context.Context) error {
},
},
}
// TODO: check if this is needed for outbound access
if lbSpec.Type == infrav1.Internal {
lb.LoadBalancerPropertiesFormat.OutboundRules = nil
}
Expand Down
Loading

0 comments on commit 286fd9a

Please sign in to comment.