Merge pull request #52293 from shiftstack/fix-lb-setting-kuryr

maxwelldb · web-flow · commit 575aac0ead01 · 2022-11-03T08:58:13.000-04:00
Bug OCPBUGS-2886: Add note about unnecessary load-balancer setting on CP
diff --git a/modules/installation-osp-setting-cloud-provider-options.adoc b/modules/installation-osp-setting-cloud-provider-options.adoc
@@ -31,7 +31,7 @@ $ vi openshift/manifests/cloud-provider-config.yaml
 
 . Modify the options based on link:https://v1-18.docs.kubernetes.io/docs/concepts/cluster-administration/cloud-providers/#cloud-conf[the `cloud.conf` specification].
 +
-Configuring Octavia for load balancing is a common case. For example:
+Configuring Octavia for load balancing is a common case for clusters that do not use Kuryr. For example:
 +
 [source,text]
 ----
@@ -64,6 +64,11 @@ Prior to saving your changes, verify that the file is structured correctly. Clus
 ====
 You must set the value of the `create-monitor` property to `True` if you use services that have the value of the `.spec.endpointTrafficPolicy` property set to `Local`. The OVN Octavia provider in {rh-openstack} 16.1 and 16.2 does not support health monitors. Therefore, services that have `ETP` parameter values set to `Local` might not respond when the `lb-provider` value is set to `"ovn"`.
 ====
++
+[IMPORTANT]
+====
+For installations that use Kuryr, Kuryr handles relevant services. There is no need to configure Octavia load balancing in the cloud provider.
+====
 
 . Save the changes to the file and proceed with installation.
 +

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ $ vi openshift/manifests/cloud-provider-config.yaml`
`31`	`31`
`32`	`32`	. Modify the options based on link:https://v1-18.docs.kubernetes.io/docs/concepts/cluster-administration/cloud-providers/#cloud-conf[the `cloud.conf` specification].
`33`	`33`	`+`
`34`		`-Configuring Octavia for load balancing is a common case. For example:`
	`34`	`+Configuring Octavia for load balancing is a common case for clusters that do not use Kuryr. For example:`
`35`	`35`	`+`
`36`	`36`	`[source,text]`
`37`	`37`	`----`
`@@ -64,6 +64,11 @@ Prior to saving your changes, verify that the file is structured correctly. Clus`
`64`	`64`	`====`
`65`	`65`	You must set the value of the `create-monitor` property to `True` if you use services that have the value of the `.spec.endpointTrafficPolicy` property set to `Local`. The OVN Octavia provider in {rh-openstack} 16.1 and 16.2 does not support health monitors. Therefore, services that have `ETP` parameter values set to `Local` might not respond when the `lb-provider` value is set to `"ovn"`.
`66`	`66`	`====`
	`67`	`++`
	`68`	`+[IMPORTANT]`
	`69`	`+====`
	`70`	`+For installations that use Kuryr, Kuryr handles relevant services. There is no need to configure Octavia load balancing in the cloud provider.`
	`71`	`+====`
`67`	`72`
`68`	`73`	`. Save the changes to the file and proceed with installation.`
`69`	`74`	`+`