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
doc: rgw: mention the civetweb support for binding to multiple ports #17141
Conversation
jenkins render docs |
Doc render available at http://docs.ceph.com/ceph-prs/17141/ |
@mattbenjamin @mdw-at-linuxbox does this look okay? |
doc/install/install-ceph-gateway.rst
Outdated
.. versionadded :: Luminous | ||
|
||
Furthermore, civetweb can be made to bind to multiple ports, multiple ports | ||
needs to be separated by ``+`` in the configuration. This allows for use cases where |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
needs
-> need
. though it may read better as can be made to bind to multiple ports by separating them with ``+`` in the configuration.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe I should also throw in that this has the advantage that the threadpool for civetweb gets shared in this case?
We support binding multiple ports in civetweb after the upgrade to 1.8, clarify this in the documentation as well Fixes: http://tracker.ceph.com/issues/20942 Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
2c4a4e0
to
01740c6
Compare
@cbodley updated |
@cbodley ok to merge? |
We support binding multiple ports in civetweb after the upgrade to 1.8,
clarify this in the documentation as well
Fixes: http://tracker.ceph.com/issues/20942
Signed-off-by: Abhishek Lekshmanan abhishek@suse.com