From: Josh Durgin <josh.durgin@inktank.com>
To: Tommi Virtanen <tv@inktank.com>
Cc: Yehuda Sadeh Weinraub <yehudasa@gmail.com>,
Christian Brunner <chb@muc.de>,
ceph-devel@vger.kernel.org
Subject: Re: [PATCH 1/2 v2] rbd: allow importing from stdin
Date: Wed, 16 May 2012 16:58:59 -0700 [thread overview]
Message-ID: <4FB43F43.2080205@inktank.com> (raw)
In-Reply-To: <CADvuQRH9KYM1KswDTBKsuMPKxqMdH0CBM=sGZq3BfMvWAG7sDQ@mail.gmail.com>
On 05/16/2012 04:51 PM, Tommi Virtanen wrote:
> On Wed, May 16, 2012 at 3:25 PM, Yehuda Sadeh Weinraub
> <yehudasa@gmail.com> wrote:
>> I'd rather not pass size for stdin. We should instead make it so that
>> we don't read size at all and just importing while !eof (or do that
>> only when size is 0). We'd need to create the image with size=0 and
>> update the header in the end.
>
> Hopefully something there would prevent us from leaking objects if the
> rbd import crashed half way through. Perhaps the header needs to
> change before each object gets created?
librbd prevents you from writing beyond the bounds of an image.
If we keep that restriction, you'd need to resize the image to
have enough space for the import, and resize down to the final
size at the end.
next prev parent reply other threads:[~2012-05-16 23:59 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-16 20:32 [PATCH 1/2 v2] rbd: allow importing from stdin Christian Brunner
2012-05-16 22:25 ` Yehuda Sadeh Weinraub
2012-05-16 23:51 ` Tommi Virtanen
2012-05-16 23:58 ` Josh Durgin [this message]
2012-05-17 0:00 ` Tommi Virtanen
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=4FB43F43.2080205@inktank.com \
--to=josh.durgin@inktank.com \
--cc=ceph-devel@vger.kernel.org \
--cc=chb@muc.de \
--cc=tv@inktank.com \
--cc=yehudasa@gmail.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.