Skip to content

Commit

Permalink
DSE 6.8.4 support (#257)
Browse files Browse the repository at this point in the history
Co-authored-by: Madhavan Sridharan <madhavan.sridharan@datastax.com>
  • Loading branch information
msmygit and msmygit authored Sep 28, 2020
1 parent 7de00ac commit d4d4e3a
Show file tree
Hide file tree
Showing 37 changed files with 53 additions and 37 deletions.
2 changes: 1 addition & 1 deletion buildsettings.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,6 @@ jenkins:
- simple-theme-plugin
dev:
images:
- datastax/dse-server:6.8.3
- datastax/dse-server:6.8.4
- datastax/cass-config-builder:1.0.3
- datastax/cassandra-mgmtapi-3_11_7:v0.1.13
Original file line number Diff line number Diff line change
Expand Up @@ -6038,6 +6038,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
2 changes: 1 addition & 1 deletion docs/ingress/kong/sni-ingress/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ When leveraging a single endpoint ingress / load balancer we lose the ability to
k3d load image --cluster k3s-default \
datastax/cass-operator:1.3.0 \
datastax/cass-config-builder:1.0.0-ubi7 \
datastax/dse-server:6.8.0-ubi7 \
datastax/dse-server:6.8.4-ubi7 \
datastax/cassandra:3.11.6-ubi7
```

Expand Down
6 changes: 3 additions & 3 deletions docs/user/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ for your `CassandraDatacenter` resource, you can use the `serverType`, `serverVe
spec properties.

`serverType` is required and must be either `dse` or `cassandra`. `serverVersion` is also required,
and the supported versions for DSE are `6.8.0` through `6.8.3`, and for Cassandra it is `3.11.6` through `3.11.7`. More versions
and the supported versions for DSE are `6.8.0` through `6.8.4`, and for Cassandra it is `3.11.6` through `3.11.7`. More versions
will be supported in the future.

If `serverImage` is not specified, a default image for the provided `serverType` and
Expand All @@ -333,7 +333,7 @@ metadata:
name: dtcntr
spec:
serverType: dse
serverVersion: 6.8.3
serverVersion: 6.8.4

```

Expand All @@ -359,7 +359,7 @@ metadata:
name: dtcntr
spec:
serverType: dse
serverVersion: 6.8.3
serverVersion: 6.8.4
serverImage: private-docker-registry.example.com/dse-img/dse:5f6e7d8c
```
Expand Down
1 change: 1 addition & 0 deletions docs/user/cass-operator-manifests-v1.13.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6037,6 +6037,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
1 change: 1 addition & 0 deletions docs/user/cass-operator-manifests-v1.14.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6037,6 +6037,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
1 change: 1 addition & 0 deletions docs/user/cass-operator-manifests-v1.15.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6037,6 +6037,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
1 change: 1 addition & 0 deletions docs/user/cass-operator-manifests-v1.16.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6049,6 +6049,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
1 change: 1 addition & 0 deletions docs/user/cass-operator-manifests-v1.17.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6049,6 +6049,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
1 change: 1 addition & 0 deletions docs/user/cass-operator-manifests-v1.18.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6047,6 +6047,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6028,6 +6028,7 @@ spec:
- 6.8.1
- 6.8.2
- 6.8.3
- 6.8.4
- 3.11.6
- 3.11.7
- 4.0.0
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,7 @@ spec:
serverType: "dse"

# Which server version to use. Required.
serverVersion: "6.8.3"
serverVersion: "6.8.4"

# Use the serverImage configuration to override Cass Operator's logic to map
# the serverType plus serverVersion into a public container image on Docker
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ metadata:
spec:
clusterName: cluster2
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 3
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ metadata:
spec:
clusterName: cluster2
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 3
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ type CassandraDatacenterSpec struct {

// Version string for config builder,
// used to generate Cassandra server configuration
// +kubebuilder:validation:Enum="6.8.0";"6.8.1";"6.8.2";"6.8.3";"3.11.6";"3.11.7";"4.0.0"
// +kubebuilder:validation:Enum="6.8.0";"6.8.1";"6.8.2";"6.8.3";"6.8.4";"3.11.6";"3.11.7";"4.0.0"
ServerVersion string `json:"serverVersion"`

// Cassandra server image name.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,13 @@ func Test_makeImage(t *testing.T) {
errString: "server 'dse' and version '6.7.0' do not work together",
},
{
name: "test 6.8.3",
name: "test 6.8.4",
args: args{
serverImage: "",
serverType: "dse",
serverVersion: "6.8.3",
serverVersion: "6.8.4",
},
want: "datastax/dse-server:6.8.3",
want: "datastax/dse-server:6.8.4",
errString: "",
},
}
Expand Down
2 changes: 2 additions & 0 deletions operator/pkg/apis/cassandra/v1beta1/webhook.go
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,8 @@ func ValidateSingleDatacenter(dc CassandraDatacenter) error {
err = nil
case "6.8.3":
err = nil
case "6.8.4":
err = nil
default:
err = attemptedTo("use unsupported DSE version '%s'", dc.Spec.ServerVersion)
}
Expand Down
14 changes: 7 additions & 7 deletions operator/pkg/apis/cassandra/v1beta1/webhook_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -34,14 +34,14 @@ func Test_ValidateSingleDatacenter(t *testing.T) {
errString: "",
},
{
name: "Dse 6.8.3 Valid",
name: "Dse 6.8.4 Valid",
dc: &CassandraDatacenter{
ObjectMeta: metav1.ObjectMeta{
Name: "exampleDC",
},
Spec: CassandraDatacenterSpec{
ServerType: "dse",
ServerVersion: "6.8.3",
ServerVersion: "6.8.4",
},
},
errString: "",
Expand Down Expand Up @@ -122,7 +122,7 @@ func Test_ValidateSingleDatacenter(t *testing.T) {
},
Spec: CassandraDatacenterSpec{
ServerType: "dse",
ServerVersion: "6.8.3",
ServerVersion: "6.8.4",
DseWorkloads: &DseWorkloads{
AnalyticsEnabled: true,
},
Expand Down Expand Up @@ -180,7 +180,7 @@ func Test_ValidateSingleDatacenter(t *testing.T) {
},
Spec: CassandraDatacenterSpec{
ServerType: "dse",
ServerVersion: "6.8.3",
ServerVersion: "6.8.4",
Config: json.RawMessage(`
{
"cassandra-yaml": {},
Expand All @@ -191,7 +191,7 @@ func Test_ValidateSingleDatacenter(t *testing.T) {
`),
},
},
errString: "attempted to define config jvm-options with dse-6.8.3",
errString: "attempted to define config jvm-options with dse-6.8.4",
},
{
name: "Allow multiple nodes per worker requires resource requests",
Expand All @@ -201,7 +201,7 @@ func Test_ValidateSingleDatacenter(t *testing.T) {
},
Spec: CassandraDatacenterSpec{
ServerType: "dse",
ServerVersion: "6.8.3",
ServerVersion: "6.8.4",
Config: json.RawMessage(`{}`),
AllowMultipleNodesPerWorker: true,
Resources: corev1.ResourceRequirements{
Expand All @@ -226,7 +226,7 @@ func Test_ValidateSingleDatacenter(t *testing.T) {
},
Spec: CassandraDatacenterSpec{
ServerType: "dse",
ServerVersion: "6.8.3",
ServerVersion: "6.8.4",
Config: json.RawMessage(`{}`),
AllowMultipleNodesPerWorker: true,
},
Expand Down
6 changes: 6 additions & 0 deletions operator/pkg/images/images.go
Original file line number Diff line number Diff line change
Expand Up @@ -48,11 +48,13 @@ const (
DSE_6_8_1
DSE_6_8_2
DSE_6_8_3
DSE_6_8_4

UBIDSE_6_8_0
UBIDSE_6_8_1
UBIDSE_6_8_2
UBIDSE_6_8_3
UBIDSE_6_8_4

ConfigBuilder
UBIConfigBuilder
Expand All @@ -79,11 +81,13 @@ var imageLookupMap map[Image]string = map[Image]string {
DSE_6_8_1: "datastax/dse-server:6.8.1",
DSE_6_8_2: "datastax/dse-server:6.8.2",
DSE_6_8_3: "datastax/dse-server:6.8.3",
DSE_6_8_4: "datastax/dse-server:6.8.4",

UBIDSE_6_8_0: "datastax/dse-server:6.8.0-ubi7",
UBIDSE_6_8_1: "datastax/dse-server:6.8.1-ubi7",
UBIDSE_6_8_2: "datastax/dse-server:6.8.2-ubi7",
UBIDSE_6_8_3: "datastax/dse-server:6.8.3-ubi7",
UBIDSE_6_8_4: "datastax/dse-server:6.8.4-ubi7",

ConfigBuilder: "datastax/cass-config-builder:1.0.3",
UBIConfigBuilder: "datastax/cass-config-builder:1.0.3-ubi7",
Expand All @@ -109,13 +113,15 @@ var versionToDSE map[string]Image = map[string]Image {
"6.8.1": DSE_6_8_1,
"6.8.2": DSE_6_8_2,
"6.8.3": DSE_6_8_3,
"6.8.4": DSE_6_8_4,
}

var versionToUBIDSE map[string]Image = map[string]Image {
"6.8.0": UBIDSE_6_8_0,
"6.8.1": UBIDSE_6_8_1,
"6.8.2": UBIDSE_6_8_2,
"6.8.3": UBIDSE_6_8_3,
"6.8.4": UBIDSE_6_8_4,
}

var log = logf.Log.WithName("images")
Expand Down
2 changes: 1 addition & 1 deletion operator/pkg/reconciliation/testing.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ func CreateMockReconciliationContext(
Size: size,
ClusterName: clusterName,
ServerType: "dse",
ServerVersion: "6.8.3",
ServerVersion: "6.8.4",
StorageConfig: storageConfig,
},
}
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/additional-seeds-two-rack-four-node-dc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster1
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 4
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/cluster-wide-install-dc1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster1
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 1
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/cluster-wide-install-dc2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster2
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 1
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster2
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 2
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/default-single-rack-2-node-dc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster2
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 2
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/default-single-rack-single-node-dc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster2
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 1
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster2
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 1
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/default-three-rack-four-node-dc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster1
serverType: dse
serverVersion: "6.8.2"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 4
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster1
serverType: dse
serverVersion: "6.8.2"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 4
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/default-three-rack-three-node-dc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster1
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
insecure: {}
size: 3
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/dse-one-node-dc-with-mtls.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster1
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
managementApiAuth:
manual:
clientSecretName: mgmt-api-client-credentials
Expand Down
2 changes: 1 addition & 1 deletion tests/testdata/graph-dc.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ metadata:
spec:
clusterName: cluster1
serverType: dse
serverVersion: "6.8.3"
serverVersion: "6.8.4"
dseWorkloads:
graphEnabled: true
managementApiAuth:
Expand Down
Loading

0 comments on commit d4d4e3a

Please sign in to comment.