Project

General

Profile

Actions

Bug #13709

closed

init script reload doesn't work on EL7

Added by Herve Rousseau over 8 years ago. Updated about 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
-
% Done:

0%

Source:
Community (user)
Tags:
Backport:
hammer, infernalis
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Checked on ceph-radosgw-0.94.5-0.el7.centos.x86_64:

# /etc/init.d/ceph-radosgw reload
Reloading ceph-radosgw configuration (via systemctl):  Job for ceph-radosgw.service failed. See 'systemctl status ceph-radosgw.service' and 'journalctl -xn' for details.
                                                           [FAILED]

This is because in the reload case, killproc $RADOSGW -SIGHUP is handled by sh's built-in kill which doesn't understand -SIGHUP

  • Output of the init script with -x
    Nov 06 09:36:33 cephrgwd01.cern.ch ceph-radosgw[31731]: + kill -SIGHUP 31250
    Nov 06 09:36:33 cephrgwd01.cern.ch ceph-radosgw[31731]: + RC=1
    Nov 06 09:36:33 cephrgwd01.cern.ch ceph-radosgw[31731]: + '[' 1 -eq 0 ']'
    Nov 06 09:36:33 cephrgwd01.cern.ch ceph-radosgw[31731]: + failure 'radosgw -SIGHUP'
    Nov 06 09:36:33 cephrgwd01.cern.ch ceph-radosgw[31731]: + local rc=1
    
  • Manual confirmation of the issue:
    sh-4.2# kill -SIGHUP `pidof radosgw`
    sh: kill: SIGHUP: invalid signal specification
    sh-4.2# kill -HUP `pidof radosgw`
    sh-4.2# echo $?
    0
    sh-4.2#
    

There's a pull request coming to fix this.


Related issues 2 (0 open2 closed)

Copied to rgw - Backport #13830: infernalis: init script reload doesn't work on EL7ResolvedAbhishek VarshneyActions
Copied to rgw - Backport #13831: hammer: init script reload doesn't work on EL7ResolvedLoïc DacharyActions
Actions #2

Updated by Nathan Cutler over 8 years ago

  • Status changed from New to Fix Under Review
Actions #3

Updated by Yehuda Sadeh over 8 years ago

  • Assignee set to Ken Dreyer
Actions #4

Updated by Ken Dreyer over 8 years ago

  • Assignee changed from Ken Dreyer to Boris Ranto
Actions #5

Updated by Boris Ranto over 8 years ago

  • Status changed from Fix Under Review to Pending Backport
  • Backport changed from hammer to hammer, infernalis

Merged upstream (master). We should probably back-port this to infernalis as well.

Actions #6

Updated by Abhishek Varshney over 8 years ago

  • Copied to Backport #13830: infernalis: init script reload doesn't work on EL7 added
Actions #7

Updated by Abhishek Varshney over 8 years ago

  • Copied to Backport #13831: hammer: init script reload doesn't work on EL7 added
Actions #8

Updated by Nathan Cutler about 8 years ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF