apache / solr-operator

Official Kubernetes operator for Apache Solr
https://solr.apache.org/operator
Apache License 2.0
242 stars 112 forks source link

Resources limits and requests configuration not set on SolrCloud pod #676

Closed alexangas closed 5 months ago

alexangas commented 5 months ago

I've been attempting to set config such as podOptions.resources.requests.cpu in the Helm chart for SolrCloud. However none of the changes in the podOptions.resources settings appear to be applying.

I had a look at the code - and I am totally not an expert so may be way off base - but thought we should be seeing this set when the container is initialised in https://github.com/apache/solr-operator/blob/8e986e60a82fcd577fe55e882e23a5491f4d3014/controllers/util/solr_util.go#L456. At least, that's how the other containers in the file are initialised. However I can also see this https://github.com/apache/solr-operator/blob/main/controllers/util/solr_util.go#L586 so 🤷🏻

I am using Solr Operator 0.7.1 as it's not possible for us to upgrade to Solr 8.11.

alexangas commented 5 months ago

Never mind. It is a bug in the tooling that i'm using.