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

[DNM] Rgw socket backlog for hammer #9912

Closed
wants to merge 1 commit into from
Closed

[DNM] Rgw socket backlog for hammer #9912

wants to merge 1 commit into from

Conversation

diluga
Copy link
Contributor

@diluga diluga commented Jun 24, 2016

(cherry picked from commit d56bdf93ced6b80b07397d57e3fa68fe68304432)

RGW: add socket backlog setting for via ceph.conf
http://tracker.ceph.com/issues/16406

RGW: add socket backlog setting for via ceph.conf
http://tracker.ceph.com/issues/16406
Signed-off-by: Feng Guo <diluga@gmail.com>
@smithfarm smithfarm changed the title Rgw socket backlog for hammer [DNM] Rgw socket backlog for hammer Jun 24, 2016
@smithfarm
Copy link
Contributor

@diluga No, this is not correct. First, wait until #9891 is merged into master. Then the backport will be done by cherry-picking the commit from #9891 into a wip branch based on jewel.

Anyway, it's too early to do a backport because we don't know when, or in what form, the master commit will be merged.

@smithfarm smithfarm closed this Jun 24, 2016
@diluga
Copy link
Contributor Author

diluga commented Jun 24, 2016

sorry,but this commit is only for hammer(not jewel),so i cherry-picking with branch (hammer 0.94.7).

@smithfarm
Copy link
Contributor

I understand. But we don't cherry-pick the branch. We make a new branch based on the hammer branch and then cherry-pick the master commit (3e4df83 in this case) into the working branch and then open a PR against hammer. But we don't do this until the master PR has been merged, OK?

@diluga
Copy link
Contributor Author

diluga commented Jun 24, 2016

get it! thx!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants