Skip to content

fix(operator): Add missing SecurityContextBuilder::build function - #1271

Merged
Techassi merged 5 commits into
mainfrom
fix/stackable-operator-security-ctx-builder-build-function
Sep 7, 2026
Merged

fix(operator): Add missing SecurityContextBuilder::build function#1271
Techassi merged 5 commits into
mainfrom
fix/stackable-operator-security-ctx-builder-build-function

Conversation

@Techassi

@Techassi Techassi commented Sep 4, 2026

Copy link
Copy Markdown
Member

This was discovered when I implemented stackabletech/stackablectl#453 and had to deal with the SecurityContextBuilder changes from #1205.

@Techassi Techassi self-assigned this Sep 4, 2026
@Techassi Techassi moved this to Development: In Progress in Stackable Engineering Sep 4, 2026
@Techassi Techassi moved this from Development: In Progress to Development: Waiting for Review in Stackable Engineering Sep 4, 2026
Comment thread crates/stackable-operator/src/builder/pod/security.rs
@Techassi
Techassi requested a review from lfrancke September 4, 2026 15:05
@Techassi Techassi moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Sep 4, 2026

@NickLarsenNZ NickLarsenNZ 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

@NickLarsenNZ NickLarsenNZ moved this from Development: In Review to Development: Done in Stackable Engineering Sep 7, 2026
@NickLarsenNZ NickLarsenNZ moved this from Development: Done to Development: In Review in Stackable Engineering Sep 7, 2026
@Techassi
Techassi added this pull request to the merge queue Sep 7, 2026
Merged via the queue into main with commit 79e2819 Sep 7, 2026
5 checks passed
@Techassi
Techassi deleted the fix/stackable-operator-security-ctx-builder-build-function branch September 7, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: In Review

Development

Successfully merging this pull request may close these issues.

3 participants