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

hammer: build/ops: init script reload doesn't work on EL7 #7187

Merged
1 commit merged into from Jan 29, 2016
Merged

hammer: build/ops: init script reload doesn't work on EL7 #7187

1 commit merged into from Jan 29, 2016

Conversation

ghost
Copy link

@ghost ghost commented Jan 11, 2016

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>
(cherry picked from commit 1b000ab)
@ghost ghost self-assigned this Jan 11, 2016
@ghost ghost added this to the hammer milestone Jan 11, 2016
@ghost ghost added bug-fix rgw labels Jan 11, 2016
ghost pushed a commit that referenced this pull request Jan 11, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jan 11, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jan 15, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Jan 29, 2016
hammer: init script reload doesn't work on EL7

Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost merged commit b6b8ee4 into ceph:hammer Jan 29, 2016
@ghost ghost changed the title hammer: init script reload doesn't work on EL7 hammer: build/ops: init script reload doesn't work on EL7 Feb 18, 2016
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant