chore(trino): Add 483, deprecate 481, remove 479

CHANGELOG.md

@@ -10,6 +10,7 @@ All notable changes to this project will be documented in this file.
   ([#944]).
 - Add `/ready` endpoint to the operator Deployment, which reports the CRD installation status ([#949]).
 - Coordinators now have a default affinity to the OPA Pods when OPA authorization is configured ([#924]).
+- Add support for 483 ([#956]).
 
 ### Changed
 
@@ -59,6 +60,10 @@ All notable changes to this project will be documented in this file.
   `replicas: null`, but counting it as zero left `discovery.uri` out of `config.properties`
   altogether, so no pod could find the coordinator ([#945]).
 
+### Removed
+
+- Remove support for 479 ([#956]).
+
 [#909]: https://github.com/stackabletech/trino-operator/pull/909
 [#913]: https://github.com/stackabletech/trino-operator/pull/913
 [#918]: https://github.com/stackabletech/trino-operator/pull/918
@@ -74,6 +79,7 @@ All notable changes to this project will be documented in this file.
 [#945]: https://github.com/stackabletech/trino-operator/pull/945
 [#949]: https://github.com/stackabletech/trino-operator/pull/949
 [#953]: https://github.com/stackabletech/trino-operator/pull/953
+[#956]: https://github.com/stackabletech/trino-operator/pull/956
 
 ## [26.7.0] - 2026-07-21
 

docs/modules/trino/examples/getting_started/code/trino.yaml

@@ -5,7 +5,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "481"
+    productVersion: "483"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:

docs/modules/trino/examples/getting_started/code/trino.yaml.j2

@@ -5,7 +5,7 @@ metadata:
   name: simple-trino
 spec:
   image:
-    productVersion: "481"
+    productVersion: "483"
   clusterConfig:
     catalogLabelSelector:
       matchLabels:

docs/modules/trino/partials/supported-versions.adoc

@@ -3,6 +3,6 @@
 // Stackable Platform documentation.
 
 // These versions must be sorted in descending order (highest to lowest).
-- 481 (Please note that we use the https://github.com/snowlift/trino-storage[trino-storage] connector version 477, as no newer version was available at the time of release)
-- 479 (deprecated; Please note that we use the https://github.com/snowlift/trino-storage[trino-storage] connector version 477, as no newer version was available at the time of release)
+- 483 (Note, the trino-storage connector is no longer shipped)
+- 481 (Deprecated)
 - 477 (LTS)

rust/operator-binary/src/config/jvm.rs

@@ -127,10 +127,8 @@ fn recommended_trino_jvm_args(product_version: u16) -> Result<Vec<String>, Error
             "-XX:+EnableDynamicAgentLoading".to_owned(),
         ]),
         // Copied from:
-        // - https://trino.io/docs/479/installation/deployment.html#jvm-config.
-        //   However, the docs are wrong: https://github.com/trinodb/trino/commit/1ddb0f9976fcd9917aaf0b689ca0acc8635e24f1.
-        //   According to the commit we need to add "--add-modules=jdk.incubator.vector"
-        479 => Ok(vec![
+        // - https://trino.io/docs/481/installation/deployment.html#jvm-config
+        481 => Ok(vec![
             "-XX:InitialRAMPercentage=80".to_owned(),
             "-XX:MaxRAMPercentage=80".to_owned(),
             "-XX:G1HeapRegionSize=32M".to_owned(),
@@ -143,13 +141,11 @@ fn recommended_trino_jvm_args(product_version: u16) -> Result<Vec<String>, Error
             "-XX:PerBytecodeRecompilationCutoff=10000".to_owned(),
             "-Djdk.attach.allowAttachSelf=true".to_owned(),
             "-Djdk.nio.maxCachedBufferSize=2000000".to_owned(),
-            "-Dfile.encoding=UTF-8".to_owned(),
-            "-XX:+EnableDynamicAgentLoading".to_owned(),
             "--add-modules=jdk.incubator.vector".to_owned(),
         ]),
         // Copied from:
-        // - https://trino.io/docs/481/installation/deployment.html#jvm-config
-        481 => Ok(vec![
+        // - https://trino.io/docs/483/installation/deployment.html#jvm-config
+        483 => Ok(vec![
             "-XX:InitialRAMPercentage=80".to_owned(),
             "-XX:MaxRAMPercentage=80".to_owned(),
             "-XX:G1HeapRegionSize=32M".to_owned(),

tests/templates/kuttl/smoke/01-create-s3-connection.yaml

@@ -0,0 +1,5 @@
+---
+apiVersion: kuttl.dev/v1beta1
+kind: TestStep
+commands:
+  - script: envsubst '$NAMESPACE' < 01_s3-connection.yaml | kubectl apply -n $NAMESPACE -f -

tests/templates/kuttl/smoke/01_s3-connection.yaml

@@ -0,0 +1,43 @@
+---
+apiVersion: s3.stackable.tech/v1alpha1
+kind: S3Connection
+metadata:
+  name: garage
+spec:
+  # Must be the FQDN, that is the only name the "tls" SecretClass puts into the
+  # certificate for the service=garage scope used in garage.yaml.
+  host: "garage.${NAMESPACE}.svc.cluster.local"
+  port: 9000
+  # The region must match the s3_region in garage.yaml.
+  region:
+    name: region-1
+  accessStyle: Path
+  credentials:
+    secretClass: s3-credentials-class
+  tls:
+    verification:
+      server:
+        caCert:
+          secretClass: tls
+---
+apiVersion: secrets.stackable.tech/v1alpha1
+kind: SecretClass
+metadata:
+  name: s3-credentials-class
+spec:
+  backend:
+    k8sSearch:
+      searchNamespace:
+        pod: {}
+---
+apiVersion: v1
+kind: Secret
+metadata:
+  name: garage-credentials
+  labels:
+    secrets.stackable.tech/class: s3-credentials-class
+stringData:
+  # Garage requires the GK<24 hex> key id format and a 64 hex char secret.
+  # Must match the GARAGE_DEFAULT_* variables in garage.yaml.
+  accessKey: GK31c0ffee31c0ffee31c0ffee
+  secretKey: deadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeefdeadbeef

tests/templates/kuttl/smoke/02-assert.yaml

@@ -0,0 +1,12 @@
+---
+apiVersion: kuttl.dev/v1beta1
+kind: TestAssert
+timeout: 600
+---
+apiVersion: apps/v1
+kind: Deployment
+metadata:
+  name: garage
+status:
+  readyReplicas: 1
+  replicas: 1

tests/templates/kuttl/smoke/02-install-garage.yaml

@@ -0,0 +1,5 @@
+---
+apiVersion: kuttl.dev/v1beta1
+kind: TestStep
+commands:
+  - script: kubectl -n $NAMESPACE apply -f 02_garage.yaml