Skip to content

Commit

Permalink
Merge pull request #1135 from gcs278/missing-scope-change-instructions
Browse files Browse the repository at this point in the history
OCPBUGS-39151: Add Missing Scope Change Instructions
  • Loading branch information
openshift-merge-bot[bot] authored Sep 4, 2024
2 parents 8252ac4 + a6b6bb2 commit 59a477e
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions pkg/operator/controller/ingress/load_balancer_service.go
Original file line number Diff line number Diff line change
Expand Up @@ -793,8 +793,7 @@ func loadBalancerServiceIsProgressing(ic *operatorv1.IngressController, service
}
if wantScope != haveScope {
err := fmt.Errorf("The IngressController scope was changed from %q to %q.", haveScope, wantScope)
switch platform.Type {
case configv1.AWSPlatformType, configv1.IBMCloudPlatformType:
if _, ok := platformsWithMutableScope[platform.Type]; !ok {
err = fmt.Errorf("%[1]s To effectuate this change, you must delete the service: `oc -n %[2]s delete svc/%[3]s`; the service load-balancer will then be deprovisioned and a new one created. This will most likely cause the new load-balancer to have a different host name and IP address from the old one's. Alternatively, you can revert the change to the IngressController: `oc -n openshift-ingress-operator patch ingresscontrollers/%[4]s --type=merge --patch='{\"spec\":{\"endpointPublishingStrategy\":{\"loadBalancer\":{\"scope\":\"%[5]s\"}}}}'`", err.Error(), service.Namespace, service.Name, ic.Name, haveScope)
}
errs = append(errs, err)
Expand Down

0 comments on commit 59a477e

Please sign in to comment.