DiamondLightSource / DiamondLightSource/blueapi

Make readOnlyRootFilesystem true

Open
#1,096 2 comments 0 reactions 0 assignees View on GitHub
helm chart priority low
Dominant language
Python
Stars
13
Forks
13
Avg merge
2d 11h
Merged PRs (30d)
25

Description

It is Helm best practice to set readOnlyRootFilesystem as true, which means that the container filesystem is readOnly and only mounts may be editable, and those mounts are explicitly marked as such- this prevents unknown behaviour when the container image is modified.

This is blocked by dodal importing `ophyd.sim` which creates SynAxis devices as a side effect. This should either be modified within ophyd.sim (which may be problematic as many facilities depend on ophyd.sim for testing) or may be resolved by dodal no longer importing ophyd.sim, which currently requires the [final devices are remade in ophyd-async](https://github.com/DiamondLightSource/mx-bluesky/issues/1076)

## Acceptance Criteria
- The helm values securityContext has readOnlyRootFilesystem by default

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.