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

mimic: rgw: beast: set a default port for endpoints #27661

Merged
merged 1 commit into from May 6, 2019

Conversation

Projects
None yet
5 participants
@pdvian
Copy link

commented Apr 18, 2019

rgw: beast: set a default port for endpoints
if no port is specified we don't set one(or set to 0), and don't exactly error
out, so rgw keeps running without being bound to a port. Set the port to default
in these cases

Fixes: http://tracker.ceph.com/issues/39000
Signed-off-by: Abhishek Lekshmanan <abhishek@suse.com>
(cherry picked from commit 9e5d4bd)

@smithfarm smithfarm added this to the mimic milestone Apr 18, 2019

@smithfarm smithfarm added the rgw label Apr 18, 2019

@smithfarm smithfarm requested a review from cbodley Apr 18, 2019

@cbodley

This comment has been minimized.

Copy link
Contributor

commented Apr 18, 2019

unrelated test failure:

122/157 Test #129: unittest_fastbmap_allocator .............***Failed 6.11 sec

@cbodley

This comment has been minimized.

Copy link
Contributor

commented Apr 18, 2019

jenkins test make check

@smithfarm

This comment has been minimized.

Copy link
Contributor

commented Apr 23, 2019

jenkins re-test this please

@smithfarm

This comment has been minimized.

Copy link
Contributor

commented Apr 23, 2019

jenkins test docs

@yuriw

This comment has been minimized.

Copy link
Contributor

commented May 2, 2019

@yuriw yuriw merged commit 8c8071a into ceph:mimic May 6, 2019

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.