Skip to content

read_only and volumes added for #3061#3110

Closed
wdhif wants to merge 1 commit intogoharbor:masterfrom
wdhif:read_only
Closed

read_only and volumes added for #3061#3110
wdhif wants to merge 1 commit intogoharbor:masterfrom
wdhif:read_only

Conversation

@wdhif
Copy link
Copy Markdown
Contributor

@wdhif wdhif commented Aug 23, 2017

This add read_only flag for the images of Harbor, as described in #3061

  • read_only and volumes (in Dockerfiles) for harbor
  • read_only and volumes (in Compose file) for notary
  • read_only and volumes (in Dockerfiles) for clair

The images will need to be rebuild and push with the new volumes, or the compose will fail.

- read_only and volumes (in Dockerfiles) for harbor
- read_only and volumes (in Compose file) for notary
- read_only and volumes (in Dockerfiles) for clair
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.03%) to 54.808% when pulling f8c2186 on wdhif:read_only into 0659ede on vmware:master.

@reasonerjt
Copy link
Copy Markdown
Contributor

@wdhif thanks for the contribution, but per discussion we want to disable the "read_only" by default, and we will refine the docker files such that the containers will work when user enable this flag in docker-compose template.

@reasonerjt
Copy link
Copy Markdown
Contributor

closing as the requirement is addressed in other PRs.

@reasonerjt reasonerjt closed this Nov 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants