Skip to content

Conversation

@sbernauer
Copy link
Member

Part of stackabletech/issues#770
According to decision https://github.com/stackabletech/decisions/issues/7

Also got rid of the unmaintained keycloak-opa-poc stack

@sbernauer sbernauer self-assigned this Oct 7, 2025
@sbernauer sbernauer moved this to Development: Waiting for Review in Stackable Engineering Oct 7, 2025
@adwk67 adwk67 self-requested a review October 10, 2025 10:16
@adwk67 adwk67 moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Oct 10, 2025
Copy link
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good (tested enough stacks to cover all product changes) - just the one trino.yaml where the listener class is still under clusterConfig.

@sbernauer sbernauer requested a review from adwk67 October 20, 2025 14:41
Copy link
Member

@adwk67 adwk67 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@sbernauer sbernauer enabled auto-merge (squash) October 22, 2025 06:35
@sbernauer sbernauer disabled auto-merge October 22, 2025 06:35
@sbernauer sbernauer merged commit 3e97494 into main Oct 22, 2025
2 checks passed
@sbernauer sbernauer deleted the chore/update-listener-classes branch October 22, 2025 06:35
@sbernauer sbernauer moved this from Development: In Review to Development: Done in Stackable Engineering Oct 22, 2025
spec:
image:
productVersion: "476"
productVersion: "451" # Trino 469 and greater require TLS for S3
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At some point we're going to need to embrace this. 451 won't be around forever 😅

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: Done

Development

Successfully merging this pull request may close these issues.

4 participants