Skip to content

Commit 3bd1b69

Browse files
committed
Remove SecurityContextBuilder::run_as_root
1 parent 8f8f449 commit 3bd1b69

2 files changed

Lines changed: 1 addition & 9 deletions

File tree

crates/stackable-operator/CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ All notable changes to this project will be documented in this file.
1212
Currently this is `runAsNonRoot: true`, which might cause product Pods to crash and require changes.
1313
- BREAKING: `PodSecurityContextBuilder::run_as_non_root` now takes a `bool` instead of assuming consumers always want to set it to `true` ([#1205]).
1414
This is needed to allow users setting it to `false` in case the new `with_stackable_defaults` function sets it to `true`.
15-
- BREAKING: `SecurityContextBuilder::run_as_root` is now called on the builder, not as a builder-creation function ([#1205]).
15+
- BREAKING: `SecurityContextBuilder::run_as_root` has been removed ([#1205]).
1616

1717
[#1205]: https://github.com/stackabletech/operator-rs/pull/1205
1818

crates/stackable-operator/src/builder/pod/security.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -27,14 +27,6 @@ impl SecurityContextBuilder {
2727
builder
2828
}
2929

30-
/// Convenience function for a wide use-case.
31-
///
32-
/// Please only use this is really needed.
33-
pub fn run_as_root(&mut self) {
34-
self.run_as_user(0);
35-
self.run_as_non_root(false);
36-
}
37-
3830
pub fn allow_privilege_escalation(&mut self, value: bool) -> &mut Self {
3931
self.security_context.allow_privilege_escalation = Some(value);
4032
self

0 commit comments

Comments
 (0)