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

rgw: fix reload on non Debian systems. #6482

Merged
merged 1 commit into from Nov 18, 2015
Merged

rgw: fix reload on non Debian systems. #6482

merged 1 commit into from Nov 18, 2015

Conversation

herver
Copy link
Contributor

@herver herver commented Nov 6, 2015

When using reload in non-debian systems, /bin/sh's kill is used to send the HUP signal to the radosgw process.
This kill version doesn't understand -SIGHUP as a valid signal, using -HUP does work.

Fix: #13709
Backport: hammer
Signed-off-by: Hervé Rousseau hroussea@cern.ch

When using reload in non-debian systems, /bin/sh's kill is used to send the HUP signal to the radosgw process.
This kill version doesn't understand -SIGHUP as a valid signal, using -HUP does work.

Fix: #13709
Backport: hammer
Signed-off-by: Hervé Rousseau <hroussea@cern.ch>
@smithfarm
Copy link
Contributor

LGTM

Reviewed-by: Nathan Cutler <ncutler@suse.com>

@ktdreyer
Copy link
Member

works for me

@b-ranto
Copy link
Contributor

b-ranto commented Nov 18, 2015

Good catch, lgtm, I'll go ahead and merge this.

b-ranto added a commit that referenced this pull request Nov 18, 2015
rgw: fix reload on non Debian systems.

Reviewed-by: Nathan Cutler <ncutler@suse.com>
Reviewed-by: Ken Dreyer <kdreyer@redhat.com>
Reviewed-by: Boris Ranto <branto@redhat.com>
@b-ranto b-ranto merged commit 16ef1da into ceph:master Nov 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants