Skip to content

Set readOnlyRootFilesystem on controller-manager container - #761

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
steveb:norootman
Aug 5, 2026
Merged

Set readOnlyRootFilesystem on controller-manager container#761
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
steveb:norootman

Conversation

@steveb

@steveb steveb commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

Add readOnlyRootFilesystem: true to the manager container's securityContext to complete the restricted Pod Security Standards profile. Mount an emptyDir at /tmp to support controller-runtime metrics cert auto-generation and Go stdlib temporary file needs.

Jira: OSPRH-33373

Checklist before requesting a review

  • I have performed a self-review of my code and confirmed it passes tests
  • Performed pre-commit run --all
  • Tested operator image in a test/dev environment. It can be CRC via install_yamls or a hotstack instance (optional)
  • Verified that no failures present in logs(optional):
    • ironic-operator-build-deploy-kuttl
    • podified-multinode-ironic-deployment

Add readOnlyRootFilesystem: true to the manager container's
securityContext to complete the restricted Pod Security Standards
profile. Mount an emptyDir at /tmp to support controller-runtime
metrics cert auto-generation and Go stdlib temporary file needs.

Jira: OSPRH-33373
Signed-off-by: Steve Baker <sbaker@redhat.com>
@openshift-ci
openshift-ci Bot requested review from hjensas and stuggi July 31, 2026 04:47
@stuggi

stuggi commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

fyi, for the operators deployed via the openstack-operator, we'd have to make the change there, like openstack-k8s-operators/openstack-operator#2008

@steveb

steveb commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

/retest

@steveb

steveb commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator Author

I don't think this one needs to be backported to stable. Landing it gives us kuttl coverage that the operator runs with readOnlyRootFilesystem: true, and the product readOnlyRootFilesystem: true is handled by openstack-k8s-operators/openstack-operator#2008

@steveb
steveb requested review from juliakreger and mumesan August 4, 2026 01:55
@openshift-ci

openshift-ci Bot commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mumesan, steveb

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot
openshift-merge-bot Bot merged commit e7764d1 into openstack-k8s-operators:main Aug 5, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants