Merge ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:initial-adjustments into ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:main
Proposed by
Athos Ribeiro
Status: | Merged |
---|---|
Merge reported by: | Athos Ribeiro |
Merged at revision: | 9f46604ac59e024049a4b03d2d786fabeb1e4557 |
Proposed branch: | ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:initial-adjustments |
Merge into: | ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:main |
Diff against target: |
51 lines (+3/-6) 4 files modified
Dockerfile (+1/-2) README.md (+0/-1) data/squid.yaml (+0/-3) entrypoint.sh (+2/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Sergio Durigan Junior | Approve | ||
Bryce Harrington | Pending | ||
Canonical Server | Pending | ||
Review via email:
|
Description of the change
Do not generate default volume for configuration. Configuration should be provided by users and there is no point into storing the default, provided configurations into a volume.
Also, create missing cache directories to ensure they are available before starting the service.
To post a comment you must log in.
Thanks, Athos.
This LGTM. I was unsure whether the "squid -Nz" command would always return, but I did some tests in a container here and it seems that it does.
+1