Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As discussed here: #281
This is NOT intended to be merged yet - I would just like to initiate the discussion about the readonly containers with some sample.
In the discussion linked above I can see that you are somewhat sceptical about including the changes for the readonly containers as you worry it will complicate the chart significantly without being a requirement for most people.
I would like to convince you that this ain't the case by showing you what was required to get the db container to work. You can see that it merely required some directories to be specified as emptyDir (which will put a writeable memory drive in place of these folders that will be discarded when the container stops). From my point of view this is a very small change that will do no harm to anybody not running the containers readonly and benefit everyone who likes to do so. My impression is the real problem with this requirement is the time required to find the issues and fix them and not the resulting solution. But feel free to share your thoughts.
I am currently working on the web container which is slightly more complicated. However I think the final solution will be comparable to this one here once I figure out the cause for #283