gerlowskija commented on code in PR #735: URL: https://github.com/apache/solr-operator/pull/735#discussion_r1882824081
########## helm/solr/Chart.yaml: ########## @@ -42,15 +42,12 @@ annotations: # Allowed syntax is described at: https://artifacthub.io/docs/topics/annotations/helm/#example artifacthub.io/changes: | - kind: added - description: Addition 1 + description: SolrClouds now support namespace sharing among pod containers in a pod. Review Comment: Every time I come back to this project after a lull, I have trouble deciding which chart should hold a given change-entry. Hopefully I've picked correctly here - I picked the 'solr' chart because the newly-exposed option affects the Solr pods (and not the operator pod itself) -- 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]
