All of lore.kernel.org
 help / color / mirror / Atom feed
From: Smart Weblications GmbH - Florian Wiessner <f.wiessner@smart-weblications.de>
To: Tommi Virtanen <tommi.virtanen@dreamhost.com>
Cc: ceph-devel@vger.kernel.org
Subject: Re: bug when using ceph and keyring does not exist
Date: Thu, 10 Nov 2011 16:32:30 +0100	[thread overview]
Message-ID: <4EBBEE8E.5030808@smart-weblications.de> (raw)
In-Reply-To: <CAORUGqC2cDWHi37G6g8R2Q03OrkhfR-EgfB_1h5Ca6WzZThP5A@mail.gmail.com>

Am 08.11.2011 18:57, schrieb Tommi Virtanen:
> On Sun, Nov 6, 2011 at 23:27, Smart Weblications GmbH - Florian
> Wiessner<f.wiessner@smart-weblications.de>  wrote:
>> i think i found a bug:
> ...

>
> So a missing keyring make ceph command crash. I see this commit, I
> think Sam found the bug independently and fixed it already. Please try
> again!
>
> commit d4ef9215fd00b08a198986fde4af5635833555ec
> Author: Samuel Just<samuel.just@dreamhost.com>
> Date:   2011-11-07 15:04:02 -0800
>
>      common: return null if mc.init() unsuccessful
>
>      Prevents ceph.cc from segfaulting on missing keyring.
>

yes, this fixes it - but a warning on stderr or so would be better imho..


-- 

Mit freundlichen Grüßen,

Florian Wiessner

Smart Weblications GmbH
Martinsberger Str. 1
D-95119 Naila

fon.: +49 9282 9638 200
fax.: +49 9282 9638 205
24/7: +49 900 144 000 00 - 0,99 EUR/Min*
http://www.smart-weblications.de

--
Sitz der Gesellschaft: Naila
Geschäftsführer: Florian Wiessner
HRB-Nr.: HRB 3840 Amtsgericht Hof
*aus dem dt. Festnetz, ggf. abweichende Preise aus dem Mobilfunknetz
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

      reply	other threads:[~2011-11-10 15:32 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-07  7:27 bug when using ceph and keyring does not exist Smart Weblications GmbH - Florian Wiessner
2011-11-08 17:57 ` Tommi Virtanen
2011-11-10 15:32   ` Smart Weblications GmbH - Florian Wiessner [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EBBEE8E.5030808@smart-weblications.de \
    --to=f.wiessner@smart-weblications.de \
    --cc=ceph-devel@vger.kernel.org \
    --cc=tommi.virtanen@dreamhost.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.