Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding new example: Docker Registry #53

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Conversation

pcgeek86
Copy link
Contributor

No description provided.

ports:
- 5000
volumes:
- /denverimaging/e3ebf132-c720-4d33-afe0-7b858eb88cdd/3dc9dd9a-802f-4dee-83dc-955f32165f08:/var/lib/registry
Copy link
Contributor

@wallnerryan wallnerryan Nov 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this path be in here? Think this should be registry-mount


flocker_hub:
endpoint: https://data.flockerhub.clusterhq.com
tokenfile: /root/vhut.txt
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

token.txt

set -e

function fli () {
local zpool_name='denverimaging'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should probably be configurable, as they may not want or have this as a zpool name

docker run --rm -it --privileged -v /etc/hosts:/etc/hosts -v /root:/root -v /${zpool_name}:/${zpool_name}:shared -v /var/log/fli:/var/log/fli -v /lib/modules:/lib/modules quay.io/clusterhq_prod/fli:c6a5deac3bb68b93341c8accfdde66fd7d13fc1f "$@"
}

### Set a unique VOlumeSet and Volume name
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where are these used. Not seeing where volumeset_name, volume_name or snapshot_name is used?

@wallnerryan
Copy link
Contributor

hey @pcgeek86 let me know if you want to address the above comments and get your example in. happy to have it.

@pcgeek86
Copy link
Contributor Author

pcgeek86 commented Dec 1, 2016

@wallnerryan Yeah I will update it at some point. I think I made the changes somewhere, but never committed and/or pushed them.

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.

2 participants