radu-gheorghe opened a new pull request, #625:
URL: https://github.com/apache/solr-operator/pull/625

   Timid attempt to fix #615 
   
   It works on my machine, it seems to fix the problem, but:
   * I have no tests. Yet. I have a feeling this is the hard part 🙂 
   * I guess I'll need to update 
[managed-updates.md](https://github.com/apache/solr-operator/blob/main/docs/solr-cloud/managed-updates.md)
 a bit as well.
   * There are a couple of TODOes in my patch where I'm not sure if things are 
right - any feedback there would be great!
   * Last but not least, with or without my patch, I always got a DOWN shard at 
the end of my rolling update. That's a separate issue, right? Still, it's 
always reproducible in my environment, maybe due to the fact that I'm testing 
with some docs in my collection and I'm having my IDE (with Solr Operator) talk 
to Kubernetes via a port forward to the "common" service, which sometimes fails 
because the underlying pod gets restarted, so I get a few "connection refused" 
errors until I put it back. Still, it should be something that the Operator 
handles gracefully, IMO.
   
   Any feedback is welcome, I'd be glad to push this forward!


-- 
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: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to