Skip to content

feat: Harden the operator Deployment - #807

Merged
lfrancke merged 2 commits into
mainfrom
feat/harden-operator-deployment
Oct 22, 2026
Merged

lfrancke merged 2 commits into
mainfrom
feat/harden-operator-deployment

Conversation

@lfrancke

Copy link
Copy Markdown
Member

Description

Part of stackabletech/issues#903, same change as stackabletech/trino-operator#963. See there for the description and testing.

Definition of Done Checklist

Author

  • Changes are OpenShift compatible
  • Helm chart can be installed and deployed operator works
  • Integration tests passed (for non trivial changes)

Reviewer

  • Code contains useful comments
  • Changelog updated

Acceptance

  • Proper release label has been added
  • Release note snippet added

@lfrancke lfrancke self-assigned this Oct 22, 2026
@lfrancke lfrancke moved this to Development: Waiting for Review in Stackable Engineering Oct 22, 2026
@adwk67
adwk67 self-requested a review October 22, 2026 13:29
@adwk67 adwk67 moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Oct 22, 2026

@adwk67 adwk67 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

--- PASS: kuttl (166.29s)
    --- PASS: kuttl/harness (0.00s)
        --- PASS: kuttl/harness/smoke_superset-6.1.0_openshift-true (165.88s)
PASS

@lfrancke
lfrancke added this pull request to the merge queue Oct 22, 2026
@lfrancke lfrancke moved this from Development: In Review to Development: Done in Stackable Engineering Oct 22, 2026
Merged via the queue into main with commit 06eab6c Oct 22, 2026
13 checks passed
@lfrancke
lfrancke deleted the feat/harden-operator-deployment branch October 22, 2026 13:49
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Oct 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

2 participants