Synchronize ScyllaDB Manager version with config.yaml

scylladb · Sep 3, 2024 · 7e5af53 · 7e5af53
1 parent 382e626
commit 7e5af53
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 6 deletions.
diff --git a/Makefile b/Makefile
@@ -433,7 +433,7 @@ endef
 define update-scylla-helm-versions
 	$(YQ) eval-all -i -P '\
 	select(fi==0).scyllaImage.tag = ( select(fi==1) | .operator.scyllaDBVersion ) | \
-	select(fi==0).agentImage.tag = ( select(fi==1) | .operator.scyllaDBManagerVersion ) | \
+	select(fi==0).agentImage.tag = ( select(fi==1) | .operator.scyllaDBManagerAgentVersion ) | \
 	select(fi==0)' \
 	'$(1)' './assets/config/config.yaml'
 endef
@@ -442,7 +442,8 @@ endef
 define update-scylla-manager-helm-versions
 	$(YQ) eval-all -i -P '\
 	select(fi==0).scylla.scyllaImage.tag = ( select(fi==1) | .operator.scyllaDBVersion ) | \
-	select(fi==0).scylla.agentImage.tag = ( select(fi==1) | .operator.scyllaDBManagerVersion ) | \
+	select(fi==0).scylla.agentImage.tag = ( select(fi==1) | .operator.scyllaDBManagerAgentVersion ) | \
+	select(fi==0).image.tag = ( select(fi==1) | .operator.scyllaDBManagerVersion ) | \
 	select(fi==0)' \
 	'$(1)' './assets/config/config.yaml'
 endef
@@ -515,7 +516,7 @@ verify-deploy:
 define replace-scyllacluster-versions
 	$(YQ) eval-all -i -P '\
 	select(fi==0 and di==$(2)).spec.version = ( select(fi==1) | .operator.scyllaDBVersion ) | \
-	select(fi==0 and di==$(2)).spec.agentVersion = ( select(fi==1) | .operator.scyllaDBManagerVersion ) | \
+	select(fi==0 and di==$(2)).spec.agentVersion = ( select(fi==1) | .operator.scyllaDBManagerAgentVersion ) | \
 	select(fi==0)' \
 	'$(1)' './assets/config/config.yaml'
 endef

diff --git a/assets/config/config.go b/assets/config/config.go
@@ -25,6 +25,7 @@ type OperatorConfig struct {
 	ScyllaDBEnterpriseVersionNeedingConsistentClusterManagementOverride string `json:"scyllaDBEnterpriseVersionNeedingConsistentClusterManagementOverride"`
 	ScyllaDBUtilsImage                                                  string `json:"scyllaDBUtilsImage"`
 	ScyllaDBManagerVersion                                              string `json:"scyllaDBManagerVersion"`
+	ScyllaDBManagerAgentVersion                                         string `json:"scyllaDBManagerAgentVersion"`
 	BashToolsImage                                                      string `json:"bashToolsImage"`
 	GrafanaImage                                                        string `json:"grafanaImage"`
 	PrometheusVersion                                                   string `json:"prometheusVersion"`

diff --git a/assets/config/config.yaml b/assets/config/config.yaml
@@ -6,7 +6,8 @@ operator:
   scyllaDBEnterpriseVersionNeedingConsistentClusterManagementOverride: "2024.1.8"
   # TODO: scyllaDBUtils image can't be bumped until scylladb/scylladb#17787 is fixed.
   scyllaDBUtilsImage: "docker.io/scylladb/scylla:5.4.0@sha256:b9070afdb2be0d5c59b1c196e1bb66660351403cb30d5c6ba446ef8c3b0754f1"
-  scyllaDBManagerVersion: "3.3.0"
+  scyllaDBManagerVersion: "3.3.0@sha256:e8c5b62c9330f91dfca24f109b033df78113d3ffaac306edf6d3c4346e1fa0bf"
+  scyllaDBManagerAgentVersion: "3.3.0@sha256:dc2684f51e961d88da5a8eac2d9f165cb7a960cbf91f67f49332e7224317c96b"
   bashToolsImage: "registry.access.redhat.com/ubi9/ubi:9.3-1361.1699548029@sha256:6b95efc134c2af3d45472c0a2f88e6085433df058cc210abb2bb061ac4d74359"
   grafanaImage: "docker.io/grafana/grafana:9.5.12@sha256:7d2f2a8b7aebe30bf3f9ae0f190e508e571b43f65753ba3b1b1adf0800bc9256"
   prometheusVersion: "v2.44.0"

diff --git a/assets/config/config_test.go b/assets/config/config_test.go
@@ -48,6 +48,11 @@ func TestProjectConfig(t *testing.T) {
 		t.Error(err)
 	}
 
+	err = validateRequired(Project.Operator.ScyllaDBManagerAgentVersion)
+	if err != nil {
+		t.Error(err)
+	}
+
 	err = validateImage(Project.Operator.BashToolsImage)
 	if err != nil {
 		t.Error(err)

diff --git a/test/e2e/framework/framework.go b/test/e2e/framework/framework.go
@@ -135,7 +135,7 @@ func (f *Framework) CommonLabels() map[string]string {
 func (f *Framework) GetDefaultScyllaCluster() *scyllav1.ScyllaCluster {
 	renderArgs := map[string]any{
 		"scyllaDBVersion":             configassets.Project.Operator.ScyllaDBVersion,
-		"scyllaDBManagerVersion":      configassets.Project.Operator.ScyllaDBManagerVersion,
+		"scyllaDBManagerVersion":      configassets.Project.Operator.ScyllaDBManagerAgentVersion,
 		"nodeServiceType":             TestContext.ScyllaClusterOptions.ExposeOptions.NodeServiceType,
 		"nodesBroadcastAddressType":   TestContext.ScyllaClusterOptions.ExposeOptions.NodesBroadcastAddressType,
 		"clientsBroadcastAddressType": TestContext.ScyllaClusterOptions.ExposeOptions.ClientsBroadcastAddressType,
@@ -151,7 +151,7 @@ func (f *Framework) GetDefaultScyllaCluster() *scyllav1.ScyllaCluster {
 func (f *Framework) GetDefaultZonalScyllaClusterWithThreeRacks() *scyllav1.ScyllaCluster {
 	renderArgs := map[string]any{
 		"scyllaDBVersion":             configassets.Project.Operator.ScyllaDBVersion,
-		"scyllaDBManagerVersion":      configassets.Project.Operator.ScyllaDBManagerVersion,
+		"scyllaDBManagerVersion":      configassets.Project.Operator.ScyllaDBManagerAgentVersion,
 		"nodeServiceType":             TestContext.ScyllaClusterOptions.ExposeOptions.NodeServiceType,
 		"nodesBroadcastAddressType":   TestContext.ScyllaClusterOptions.ExposeOptions.NodesBroadcastAddressType,
 		"clientsBroadcastAddressType": TestContext.ScyllaClusterOptions.ExposeOptions.ClientsBroadcastAddressType,