Skip to content

feat: Harden the operator Deployment - #963

Open
lfrancke wants to merge 1 commit into
mainfrom
feat/harden-operator-deployment
Open

lfrancke wants to merge 1 commit into
mainfrom
feat/harden-operator-deployment

Conversation

@lfrancke

@lfrancke lfrancke commented Oct 21, 2026 •

Copy link
Copy Markdown
Member

Description

Part of stackabletech/issues#903

Sets a restricted security context on the operator Deployment (runAsNonRoot, the RuntimeDefault seccomp profile, allowPrivilegeEscalation: false, all capabilities dropped) plus a read-only root filesystem, and updates the defaults in values.schema.json to match.

The security context is the same one I already did for commons-operator: stackabletech/commons-operator#462
It was approved over there already and more details linked there. Not repeating all of that here.

The read-only root filesystem works because file logs are the only thing an operator writes, and those moved to an emptyDir under /var/log with stackabletech/operator-templating#656.

Users can still override all of this.

Note

When this is merged I'll have CLaude open all the remaining ones.

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 21, 2026
@lfrancke lfrancke moved this to Development: Waiting for Review in Stackable Engineering Oct 21, 2026

@sbernauer sbernauer 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, but stackabletech/commons-operator#462 also has a changelog entry, which is missing here.
I think we should also set readOnlyRootFilesystem on commons-operator

@sbernauer sbernauer moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Oct 22, 2026
@lfrancke

Copy link
Copy Markdown
Member Author

Yes, commons-operator will get a followup to have the readOnlyRootFilesystem. Nick and I decided during that PR that it makes sense to tackle that.

And changelog: Good catch. I'll add it here.

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.

2 participants