Project

General

Profile

Bug #12407

With root as default user, unable to have multiple RGW instances running

Added by Pavan Rallabhandi over 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Urgent
Assignee:
Target version:
-
Start date:
07/20/2015
Due date:
% Done:

0%

Source:
Community (dev)
Tags:
Backport:
hammer
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:

Description

This is with reference to the commit https://github.com/ceph/ceph/commit/f30fa4a364602fb9412babf7319140eca4c64995 and tracker http://tracker.ceph.com/issues/11453

On Hammer binaries, we are finding this fix has regressed to have multiple RGW instances to be run on a single machine. Meaning, with no user specified under ‘client.radosgw.gateway’ sections, and by having the default user to be assumed as ‘root’, we are unable to get multiple RGW daemons run on a client machine.

The start-stop-daemon complains than an instance of ‘radosgw’ is already running, by starting the first daemon in the configuration and bails out from starting further instances:

<snip>

+ start-stop-daemon --start u root -x /usr/bin/radosgw - -n client.radosgw.gateway-3
/usr/bin/radosgw already running.

<\snip>

However, by having a user specified in the relevant ‘client.radosgw.gateway’ sections, one can get around this issue. Wanted to confirm if this is indeed a regression or was it expected to behave so from the fix.


Related issues

Copied to rgw - Backport #13227: With root as default user, unable to have multiple RGW instances running Resolved 07/20/2015

Associated revisions

Revision e17c8e1a (diff)
Added by Sage Weil about 3 years ago

init-radosgw: specify pid file to start-stop-daemon

This fixes restart when multiple instances are running.

Fixes: #12407
Tested-by: Pavan Rallabhandi <>
Signed-off-by: Sage Weil <>

Revision 607904e8 (diff)
Added by Sage Weil about 3 years ago

init-radosgw: specify pid file to start-stop-daemon

This fixes restart when multiple instances are running.

Fixes: #12407
Tested-by: Pavan Rallabhandi <>
Signed-off-by: Sage Weil <>
(cherry picked from commit e17c8e1a7a06db0e2680dedc7e954021610320d1)

History

#1 Updated by Sage Weil over 3 years ago

  • Project changed from Ceph to rgw
  • Category deleted (22)
  • Priority changed from Normal to Urgent
  • Affected Versions deleted (v0.21.1)

#2 Updated by Loic Dachary over 3 years ago

  • Target version deleted (v0.94.3)

#3 Updated by Yehuda Sadeh over 3 years ago

  • Assignee set to Sage Weil

#4 Updated by Sage Weil over 3 years ago

diff --git a/src/init-radosgw b/src/init-radosgw
index 6fb8ba2..b7569a0 100644
--- a/src/init-radosgw
+++ b/src/init-radosgw
@@ -100,7 +100,7 @@ case "$1" in

             echo "Starting $name..." 
            if [ $DEBIAN -eq 1 ]; then
-               start-stop-daemon --start -u $user -x $RADOSGW -- -n $name
+               start-stop-daemon --start -u $user -x $RADOSGW -p /var/run/ceph/client-$name.pid -- -n $name
            elif [ -n "$SYSTEMD_RUN" ]; then
                 $SYSTEMD_RUN -r su "$user" -c "ulimit -n 32768; $RADOSGW -n $name" 
             else

this would probably fix it. Pavan, do you mind testing?

#5 Updated by Sage Weil over 3 years ago

Pavan, can you also let me know what version of debian you are using? Thanks!

#6 Updated by Sage Weil over 3 years ago

  • Status changed from New to Need More Info

#7 Updated by Pavan Rallabhandi over 3 years ago

Sage, apologies for the delay in response, was away from work for a while. Yes, your proposed fix works. We are running Ubuntu 14.04.

Please let me know if you need any more info from me.

Thanks,
-Pavan.

#8 Updated by Yehuda Sadeh over 3 years ago

  • Status changed from Need More Info to Verified

#9 Updated by Yehuda Sadeh over 3 years ago

  • Status changed from Verified to In Progress

#10 Updated by Sage Weil about 3 years ago

  • Status changed from In Progress to Need Review
  • Backport set to hammer

#11 Updated by Sage Weil about 3 years ago

  • Status changed from Need Review to Pending Backport

#12 Updated by Loic Dachary about 3 years ago

What commit / pull requests fixes this issue ?

#15 Updated by Loic Dachary about 3 years ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF