HoustonPutman commented on code in PR #827:
URL: https://github.com/apache/solr-operator/pull/827#discussion_r4381583748
##########
api/v1beta1/solrcloud_types.go:
##########
@@ -1327,6 +1328,49 @@ func (zkInfo ZookeeperConnectionInfo)
ZkConnectionString() string {
return zkInfo.InternalConnectionString + zkInfo.ChRoot
}
+// solrImageTagVersionRegex matches Solr image tags, which must start with the
Solr version, "<major>[.<minor>[.<patch>...]]".
+// Anything after the version must start with neither a digit nor a dot,
+// e.g. "10.0.0", "10.1.0-SNAPSHOT", "10.0.0.1", "10.0.0@sha256:...",
"9.10-slim" or "8.11".
+// Short versions are accepted because released Solr Operator versions
defaulted to tags such as "8.11",
+// and they cannot change their major version.
+var solrImageTagVersionRegex =
regexp.MustCompile(`^(?P<major>\d+)(?:\.\d+)*(?:[^\d.].*)?$`)
+
+const (
+ // The range of Solr major versions that the Solr Operator supports.
+ MinSupportedSolrMajorVersion = 8
Review Comment:
If we make the minimum version 9, then does some of this get easier like the
solr modules stuff?
--
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]
