Skip to content

fix: harden operator manager pod with readOnlyRootFilesystem and resource limits - #181

Merged
miyunari merged 1 commit into
sigstore:mainfrom
sampras343:fix/manager-security-hardening
Sep 20, 2026
Merged

miyunari merged 1 commit into
sigstore:mainfrom
sampras343:fix/manager-security-hardening

Conversation

@sampras343

@sampras343 sampras343 commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

What

Harden the operator manager deployment (config/manager/manager.yaml):

  • Set readOnlyRootFilesystem: true on the manager container
  • Set resource requests/limits: CPU 10m/500m, Memory 64Mi/256Mi

Why

The manager container's SecurityContext was missing readOnlyRootFilesystem, which fails restricted Pod Security Standards. A writable rootfs also means a compromised process (e.g., RCE via a dependency vulnerability) could drop binaries or modify DNS config inside the container.

Resource limits were {}. MVO runs a webhook in the admission path of every pod creation. A webhook with no memory limit that leaks under load can OOM-kill other pods on the node via eviction pressure. CPU limits prevent a runaway reconciliation loop from starving the kubelet.

Both are about MVO being trustworthy as a security component.

Summary

  • Add readOnlyRootFilesystem: true to the manager container SecurityContext to meet restricted Pod Security Standards and prevent filesystem tampering in a compromised-process scenario
  • Set resource requests/limits (CPU 10m/500m, Memory 64Mi/256Mi) to prevent unbounded resource consumption from webhook storms or runaway reconciliation

Closes #182

Release Note

Documentation

…urce limits

Add readOnlyRootFilesystem: true to the manager container SecurityContext
to meet restricted Pod Security Standards and prevent filesystem tampering.
Set resource requests/limits (CPU 10m/500m, Memory 64Mi/256Mi) to prevent
unbounded resource consumption.

Signed-off-by: Sachin Sampras M <sampras343@gmail.com>
@sampras343

Copy link
Copy Markdown
Contributor Author

The comment is still preserved to allow users to edit these resource limits based on requirements. But a default is set as a preventive method in case of mem leaks.

@miyunari
miyunari merged commit a0b3e08 into sigstore:main Sep 20, 2026
6 checks passed
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.

Operator manager pod fails restricted Pod Security Standards

2 participants