Skip to content

[SPARK-57940] Enable readOnlyRootFilesystem for operator container in Helm chart#743

Closed
dongjoon-hyun wants to merge 1 commit into
apache:mainfrom
dongjoon-hyun:SPARK-57940
Closed

[SPARK-57940] Enable readOnlyRootFilesystem for operator container in Helm chart#743
dongjoon-hyun wants to merge 1 commit into
apache:mainfrom
dongjoon-hyun:SPARK-57940

Conversation

@dongjoon-hyun

Copy link
Copy Markdown
Member

What changes were proposed in this pull request?

This PR aims to enable readOnlyRootFilesystem for the operator container in the Helm chart.

Since the operator writes temporary files under java.io.tmpdir (SPARK-57918), an emptyDir
volume is mounted at /tmp, following the existing logs-volume pattern.

Why are the changes needed?

A read-only root filesystem prevents tampering with the container image at runtime and
completes the CIS Benchmark hardening on top of the existing restricted Pod Security
Standard compliance.

Does this PR introduce any user-facing change?

Yes. The operator container root filesystem becomes read-only by default; writable paths are
provided via emptyDir volumes (/tmp, /opt/spark-operator/logs). Users can override
operatorDeployment.operatorPod.operatorContainer.securityContext if needed.

How was this patch tested?

Pass the CIs. Also manually verified with helm lint --strict and helm template.

Was this patch authored or co-authored using generative AI tooling?

Generated-by: Claude Fable 5

…in Helm chart

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@dongjoon-hyun

Copy link
Copy Markdown
Member Author

I'm going to merge this security feature to unblock the release preparation.

@dongjoon-hyun dongjoon-hyun added this to the 1.0.0 milestone Jul 6, 2026
@dongjoon-hyun

Copy link
Copy Markdown
Member Author

Merged to main

@dongjoon-hyun
dongjoon-hyun deleted the SPARK-57940 branch July 6, 2026 00:55

@peter-toth peter-toth left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Late LGTM.

@dongjoon-hyun

Copy link
Copy Markdown
Member Author

Thank you, Peter!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants