Skip to content

Commit 750623a

Browse files
danielmeintclaude
andcommitted
Surface required solrMajorVersion in helm/solr chart
Adds a new `solrMajorVersion` value (default 9, matching the default image.tag of 9.10.0) and renders it under spec.solrMajorVersion. Uses Helm's `required` so the chart fails fast with a clear message if a user explicitly unsets it. Documents the new value in the chart README and adds a v0.10.0 upgrade note covering both the CRD field and the new chart value. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
1 parent 7b49624 commit 750623a

4 files changed

Lines changed: 12 additions & 0 deletions

File tree

docs/upgrade-notes.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,11 @@ _Note that the Helm chart version does not contain a `v` prefix, which the downl
126126
## Upgrade Warnings and Notes
127127

128128
### v0.10.0
129+
- **`SolrCloud.spec.solrMajorVersion` is now required**
130+
A new required field `spec.solrMajorVersion` (int, 8–10) drives version-specific operator behavior for Solr 10 support.
131+
Existing SolrCloud CRs must add this field (e.g. `solrMajorVersion: 9`) before upgrading the operator; CRD validation rejects CRs that omit it.
132+
The `helm/solr` chart exposes a new `solrMajorVersion` value (default `9`) that must be bumped to `10` when running a Solr 10.x image.
133+
129134
- **Logging now defaults to JSON format**
130135
The new default for CLI flag `--zap-devel` is now `false`, causing log encoding to be `json` and log level to be `info`.
131136
There is a new helm value `development` that can be set to `true` to switch to `console` encoding and `debug` level

helm/solr/README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -85,6 +85,7 @@ Descriptions on how to use these options can be found in the [SolrCloud document
8585
| image.tag | string | `"9.10.0"` | The tag/version of Solr to run |
8686
| image.pullPolicy | string | | PullPolicy for the Solr image, defaults to the empty Pod behavior |
8787
| image.imagePullSecret | string | | PullSecret for the Solr image |
88+
| solrMajorVersion | int | `9` | Solr major version (8, 9, or 10). Required by the operator; must match `image.tag`'s major version. When pointing `image.tag` at a Solr 10.x image, set this to `10`. |
8889
| busyBoxImage.repository | string | `"busybox"` | The repository of the BusyBox image |
8990
| busyBoxImage.tag | string | `"1.28.0-glibc"` | The tag/version of BusyBox to run |
9091
| busyBoxImage.pullPolicy | string | | PullPolicy for the BusyBox image, defaults to the empty Pod behavior |

helm/solr/templates/solrcloud.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ spec:
2424
{{- if (quote .Values.replicas) }}
2525
replicas: {{ .Values.replicas }}
2626
{{- end }}
27+
solrMajorVersion: {{ required "solrMajorVersion is required (8, 9, or 10) and must match image.tag's major version" .Values.solrMajorVersion }}
2728
{{- if .Values.image }}
2829
solrImage:
2930
{{- if .Values.image.repository }}

helm/solr/values.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,6 +46,11 @@ image:
4646
pullPolicy: ""
4747
imagePullSecret: ""
4848

49+
# Solr major version (8, 9, or 10). Required by the operator; must match the
50+
# major version of image.tag. Defaults to 9 to match the default image.tag.
51+
# When pointing image.tag at a Solr 10.x image, set this to 10.
52+
solrMajorVersion: 9
53+
4954
busyBoxImage: {}
5055
# repository: "busybox"
5156
# tag: "1.28.0-glibc"

0 commit comments

Comments
 (0)