Skip to content
This repository was archived by the owner on Nov 10, 2025. It is now read-only.

Add a volume for /app/tmp/cache in Dockerfile#762

Merged
AP-Hunt merged 1 commit intomainfrom
cache-dir-volume-hint
May 23, 2025
Merged

Add a volume for /app/tmp/cache in Dockerfile#762
AP-Hunt merged 1 commit intomainfrom
cache-dir-volume-hint

Conversation

@AP-Hunt
Copy link
Copy Markdown
Contributor

@AP-Hunt AP-Hunt commented May 23, 2025

What problem does this pull request solve?

Trello card: https://trello.com/c/KoHvaEUA/681-aws-m112-ecs-read-only-root-filesystem-configuration

So that the application can write to it when the filesystem is otherwise readonly

Things to consider when reviewing

  • Ensure that you consider the wider context.
  • Does it work when run on your machine?
  • Is it clear what the code is doing?
  • Do the commit messages explain why the changes were made?
  • Are there all the unit tests needed?
  • Has all relevant documentation been updated?

So that the application can write to it when the filesystem is otherwise readonly
@sonarqubecloud
Copy link
Copy Markdown

@AP-Hunt AP-Hunt merged commit 8bb184b into main May 23, 2025
5 checks passed
@AP-Hunt AP-Hunt deleted the cache-dir-volume-hint branch May 23, 2025 13:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant