danielmeint commented on code in PR #827: URL: https://github.com/apache/solr-operator/pull/827#discussion_r4414905917
########## docs/modules/upgrade-notes/pages/upgrade-notes.adoc: ########## @@ -136,6 +137,35 @@ _Note that the Helm chart version does not contain a `v` prefix, which the downl === v0.10.0 +* *Solr 10 support, and Solr image tags must start with the Solr version* ++ +The Solr Operator now supports Solr 10. +Solr 10 needs a different `solr.xml`, different env vars and different `bin/solr` commands than Solr 9, so the operator reads the Solr version from `spec.solrImage.tag`. +The tag must therefore start with the Solr version, e.g. `9.10.0`, `10.0.0`, `9.10`, `8.11`, `10`, `9.10.0-slim`, `10.0.0-my-custom-build` or `10.0.0@sha256:<digest>`. +Tags without a version, such as `latest` or `slim`, and custom tags that don't start with the Solr version, are not accepted. +The operator does not reconcile a SolrCloud with such a tag, or with an unsupported Solr major version; it emits an `UnsupportedSolrVersion` Warning event instead. +*Before upgrading the operator*, check the image tags of your SolrClouds (including the `image.tag` value of the `solr` Helm chart) and change them where needed. +SolrClouds that never set a tag got the operator's default tag at the time they were created (e.g. `8.11`), which is accepted. +SolrClouds running Solr 8.11 or 9.x with an accepted tag are not changed by this upgrade, so their pods are not restarted. +Changing a tag, e.g. from `latest` to `9.10.0`, changes the pod template and therefore restarts all Solr pods, even if the new tag points to the same image. +If you pass `-Dsolr.allowPaths=...` via `solrOpts`, use `-Dsolr.security.allow.paths=...` when moving to Solr 10, which doesn't read the old camelCase name anymore. Review Comment: added a comment below; SOLR_ALLOW_PATHS is only read in v 9.6+ I found ########## docs/modules/upgrade-notes/pages/upgrade-notes.adoc: ########## @@ -136,6 +137,35 @@ _Note that the Helm chart version does not contain a `v` prefix, which the downl === v0.10.0 +* *Solr 10 support, and Solr image tags must start with the Solr version* ++ +The Solr Operator now supports Solr 10. +Solr 10 needs a different `solr.xml`, different env vars and different `bin/solr` commands than Solr 9, so the operator reads the Solr version from `spec.solrImage.tag`. +The tag must therefore start with the Solr version, e.g. `9.10.0`, `10.0.0`, `9.10`, `8.11`, `10`, `9.10.0-slim`, `10.0.0-my-custom-build` or `10.0.0@sha256:<digest>`. +Tags without a version, such as `latest` or `slim`, and custom tags that don't start with the Solr version, are not accepted. +The operator does not reconcile a SolrCloud with such a tag, or with an unsupported Solr major version; it emits an `UnsupportedSolrVersion` Warning event instead. +*Before upgrading the operator*, check the image tags of your SolrClouds (including the `image.tag` value of the `solr` Helm chart) and change them where needed. +SolrClouds that never set a tag got the operator's default tag at the time they were created (e.g. `8.11`), which is accepted. +SolrClouds running Solr 8.11 or 9.x with an accepted tag are not changed by this upgrade, so their pods are not restarted. +Changing a tag, e.g. from `latest` to `9.10.0`, changes the pod template and therefore restarts all Solr pods, even if the new tag points to the same image. +If you pass `-Dsolr.allowPaths=...` via `solrOpts`, use `-Dsolr.security.allow.paths=...` when moving to Solr 10, which doesn't read the old camelCase name anymore. Review Comment: added a comment below; `SOLR_ALLOW_PATHS` is only read in v 9.6+ I found -- 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]
