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

Update ceph_rgw_docker_extra_env to add bind ip #1839

Merged
merged 2 commits into from Aug 31, 2017

Conversation

leseb
Copy link
Member

@leseb leseb commented Aug 31, 2017

This patch adds passing the RGW_CIVETWEB_IP to the docker
container. This IP defaults to the value of radosgw_civetweb_bind_ip.
radosgw_civetweb_bind_ip default to ipv4.default

Without this value, the RGW containter will bind to 0.0.0.0

leseb and others added 2 commits August 31, 2017 15:50
There is no need for a privileged rgw container

Signed-off-by: Sébastien Han <seb@redhat.com>
This patch adds passing the RGW_CIVETWEB_IP to the docker
container. This IP defaults to the value of radosgw_civetweb_bind_ip.
radosgw_civetweb_bind_ip default to ipv4.default

Without this value, the RGW containter will bind to 0.0.0.0
@leseb leseb force-pushed the colonwq-update-docker-rgw-exec branch from 7a4b3cf to eaccc12 Compare August 31, 2017 13:50
@leseb
Copy link
Member Author

leseb commented Aug 31, 2017

luminous-ansible2.3-docker_cluster successfully passed, it's all that matters.

@leseb leseb merged commit 673938e into master Aug 31, 2017
@leseb leseb deleted the colonwq-update-docker-rgw-exec branch August 31, 2017 17:47
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.

None yet

2 participants