Merge ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:main-wip into ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:main
Proposed by
Athos Ribeiro
Status: | Merged |
---|---|
Merged at revision: | c44d0c1bc71e27c92f4757cae59fd59534c8f8de |
Proposed branch: | ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:main-wip |
Merge into: | ~ubuntu-docker-images/ubuntu-docker-images/+git/squid:main |
Diff against target: |
102 lines (+36/-11) 4 files modified
Dockerfile (+2/-1) README.md (+4/-2) data/squid.yaml (+12/-7) entrypoint.sh (+18/-1) |
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
As per the review suggestions in our mailing list, this MP updates the OCI documentation and allows for parameters to be passed through CMD. The latter was done in a much simpler way than the one present in redis image due to how the parameters are being passed to the image. Moreover, in the most basic use case the users will not need to pass any parameters, but mount the configuration file instead.
To post a comment you must log in.
Thanks, Athos. LGTM.