HoustonPutman commented on code in PR #530:
URL: https://github.com/apache/solr-operator/pull/530#discussion_r1156054051
##########
docs/solr-cloud/managed-updates.md:
##########
@@ -89,3 +89,25 @@ spec:
annotations:
manualrestart: "2021-10-20T08:37:00Z"
```
+
+## Pod Readiness during updates
+
+The Solr Operator sets up at least two Services for every SolrCloud.
+- Always
+ - A clusterIP service for all solr nodes (What we call the "common service")
+- Either
+ - A [Headless
Service](https://kubernetes.io/docs/concepts/services-networking/service/#headless-services)
for individual Solr Node endpoints that are not exposed via an Ingress.
+ - Individual clusterIP services for Solr Nodes that are exposed via an
Ingress
+
+Only the common service uses the `publishNotReadyAddresses: false` option,
since the common service should load balance between all available nodes.
+The other services have individual endpoints for each node, so there is no
reason to de-list pods that are not available.
+
+When doing a rolling upgrade, or taking down a pod for anytime, we want to
first stop all requests to this pod.
Review Comment:
```suggestion
When doing a rolling upgrade, or taking down a pod for any time, we want to
first stop all requests to this pod.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
