HoustonPutman commented on PR #892: URL: https://github.com/apache/solr-operator/pull/892#issuecomment-6776063733
> That shuold be possible I suppose, do some introspection somewhere and default to true if supported. So looking through and testing things, there is actually no issue by just turning it on always. It will give an empty string for non-enabled Kubernetes clusters. And solr will handle that gracefully. I think we should just always use it, and remove the CRD option. > Related to this, I also thought about enabling affinity placement plugin as default, with disk warning set to 85% disk used (operator knows pvc size), see https://github.com/apache/solr-operator/pull/894. But that can be a followup issue once each of the features are included. I do like this. There may be a problem with the plugin not allowing multiple of the same replica on the same node, but we can adjust the integration tests to make sure we never get into that state. -- 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]
