Skip to content

Commit

Permalink
multiple ARO managed public IPs per Load balancer during cluster upda…
Browse files Browse the repository at this point in the history
…te (#3095)

* add dynamic validation for lb profile

* add CreateOrUpdateAndWait to publicIpAddressClient

* add List to PublicIPAddressesClient

* fix fake uuid generator

* add reconcileLoadBalancerProfile to update process

* defer cleanup of ips to prevent orphaned ips

* cleanup

* cleanup

* patch effectiveOutboundIPs if it does not match actual loadbalancer ips

* cleanup

* refactor cleanup of managed ips to propogate cleanup errors

* update error description for multiple errors

* update dynamic validation to use snatPortsPerIP const
  • Loading branch information
tony-schndr authored Aug 28, 2023
1 parent bd89261 commit 3b370ae
Show file tree
Hide file tree
Showing 13 changed files with 2,226 additions and 13 deletions.
1 change: 1 addition & 0 deletions pkg/cluster/install.go
Original file line number Diff line number Diff line change
Expand Up @@ -186,6 +186,7 @@ func (m *manager) Update(ctx context.Context) error {
steps.Action(m.renewMDSDCertificate),
steps.Action(m.updateOpenShiftSecret),
steps.Action(m.updateAROSecret),
steps.Action(m.reconcileLoadBalancerProfile),
}

if m.adoptViaHive {
Expand Down
Loading

0 comments on commit 3b370ae

Please sign in to comment.