Skip to content

Commit 91d44f1

Browse files
Merge pull request #21191 from praveenkumar/issue_20203
Fix #20203 for release-3.10 branch
2 parents 2a0315a + 23c17b9 commit 91d44f1

File tree

1 file changed

+18
-15
lines changed

1 file changed

+18
-15
lines changed

pkg/oc/clusteradd/components/web-console-operator/web_console_operator.go

+18-15
Original file line numberDiff line numberDiff line change
@@ -93,23 +93,26 @@ func (c *WebConsoleOperatorComponentOptions) Install(dockerClient dockerhelper.I
9393
if err != nil {
9494
return err
9595
}
96-
operatorConfig, err := operatorClient.WebconsoleV1alpha1().OpenShiftWebConsoleConfigs().Get("instance", metav1.GetOptions{})
97-
if err != nil {
98-
return err
99-
}
10096

101-
masterPublicHostPort, err := getMasterPublicHostPort(c.InstallContext.BaseDir())
102-
if err != nil {
103-
return err
104-
}
105-
operatorConfig.Spec.WebConsoleConfig.ClusterInfo.ConsolePublicURL = "https://" + masterPublicHostPort + "/console/"
106-
operatorConfig.Spec.WebConsoleConfig.ClusterInfo.MasterPublicURL, err = getMasterPublicURL(c.InstallContext.BaseDir())
107-
if err != nil {
108-
return err
109-
}
11097
// we can race a controller. It's not a big deal if we're a little late, so retry on conflict. It's easier than a patch.
111-
err = retry.RetryOnConflict(retry.DefaultRetry, func() error {
112-
_, err := operatorClient.WebconsoleV1alpha1().OpenShiftWebConsoleConfigs().Update(operatorConfig)
98+
backoff := retry.DefaultBackoff
99+
backoff.Steps = 6
100+
err = retry.RetryOnConflict(backoff, func() error {
101+
operatorConfig, err := operatorClient.WebconsoleV1alpha1().OpenShiftWebConsoleConfigs().Get("instance", metav1.GetOptions{})
102+
if err != nil {
103+
return err
104+
}
105+
106+
masterPublicHostPort, err := getMasterPublicHostPort(c.InstallContext.BaseDir())
107+
if err != nil {
108+
return err
109+
}
110+
operatorConfig.Spec.WebConsoleConfig.ClusterInfo.ConsolePublicURL = "https://" + masterPublicHostPort + "/console/"
111+
operatorConfig.Spec.WebConsoleConfig.ClusterInfo.MasterPublicURL, err = getMasterPublicURL(c.InstallContext.BaseDir())
112+
if err != nil {
113+
return err
114+
}
115+
_, err = operatorClient.WebconsoleV1alpha1().OpenShiftWebConsoleConfigs().Update(operatorConfig)
113116
return err
114117
})
115118
if err != nil {

0 commit comments

Comments
 (0)