From: Alex Elder <elder@inktank.com>
To: Josh Durgin <josh.durgin@inktank.com>
Cc: Yehuda Sadeh <yehuda@inktank.com>,
Gregory Farnum <greg@inktank.com>,
"ceph-devel@vger.kernel.org" <ceph-devel@vger.kernel.org>
Subject: Re: rbd volume upgrades
Date: Fri, 09 Nov 2012 14:09:15 -0600 [thread overview]
Message-ID: <509D62EB.1080508@inktank.com> (raw)
In-Reply-To: <509D617E.1050000@inktank.com>
On 11/09/2012 02:03 PM, Josh Durgin wrote:
> On 11/09/2012 11:44 AM, Yehuda Sadeh wrote:
>> On Fri, Nov 9, 2012 at 11:30 AM, Josh Durgin <josh.durgin@inktank.com>
>> wrote:
>>> On 11/09/2012 11:08 AM, Yehuda Sadeh wrote:
>>>>
>>>> On Fri, Nov 9, 2012 at 11:04 AM, Josh Durgin <josh.durgin@inktank.com>
>>>> wrote:
>>>>>
>>>>> On 11/09/2012 11:01 AM, Gregory Farnum wrote:
>>>>>>
>>>>>>
>>>>>> I was asked today if there's a way to upgrade RBD volumes from v1 to
>>>>>> v2. I didn't think so, but wanted
>>>>>> 1) to make sure I'm right,
>>>>>> 2) to ask how hard it would be,
>>>>>> 3) to ask if we haven't done it because it didn't occur to us or
>>>>>> because it's too hard.
>>>>>> -Greg
>>>>>>
>>>>>
>>>>> This was addressed in the original discussions about format 2.
>>>>>
>>>>> You need to export and then import the volume as format 2. Format 2
>>>>> uses
>>>>> different names for objects, so providing an 'upgrade' path would
>>>>> still
>>>>> require copying all the data around.
>>>>>
>>>> Couldn't we just set a flag in the header specifying the object naming
>>>> version, which would then only require updating the header?
>>>>
>>>> Yehuda
>>>
>>>
>>> The header was separated from the id object to allow renames to work
>>> while the image was in use or with cloning. The whole header format
>>> changed and moved to a different object as a result. It would be
>>> messy to implement this kind of upgrade, and doesn't provide much
>>> benefit when there's an easy way to convert already. If someone really
>>> wanted it, it could be implemented, but otherwise I don't think it's
>>> worth adding. It would have to be added to the upcoming kernel
>>> layering support too.
>>>
>>
>> The assumption is that when you upgrade you don't go back, so the fact
>> that the header was separated from the id object doesn't change much.
>> An upgrade process would be the same as creating a new v2 image,
>> having object names (prefix?) that set as the original object names,
>> and with a version field that specifies that these are a v1 names.
>>
>> The problem that I see with converting v1 to v2 through copy is that
>> (besides the cumbersome and potentially very long process) we will end
>> up turning sparse data objects into fully written data objects, which
>> will affect the data consumption.
>
> That's a good point about export. It would be good to make export create
> sparse files as well, but since it doesn't yet, the in-place upgrade
> would be better for space usage.
Plus! It looks like you don't even need a flag.
I think if you simply recorded the old-format object prefix in the
new format header, all would be fine. The format of the object
id has not changed between v1 and v2, just the object prefix.
-Alex
next prev parent reply other threads:[~2012-11-09 20:09 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-09 19:01 rbd volume upgrades Gregory Farnum
2012-11-09 19:04 ` Josh Durgin
2012-11-09 19:06 ` Gregory Farnum
2012-11-09 19:08 ` Yehuda Sadeh
2012-11-09 19:27 ` Alex Elder
2012-11-09 19:30 ` Josh Durgin
2012-11-09 19:44 ` Yehuda Sadeh
2012-11-09 19:52 ` Alex Elder
2012-11-09 20:03 ` Josh Durgin
2012-11-09 20:09 ` Alex Elder [this message]
2012-11-09 20:26 ` Josh Durgin
2012-11-09 20:30 ` Yehuda Sadeh
2012-11-09 20:31 ` Yehuda Sadeh
2012-11-09 20:34 ` Josh Durgin
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=509D62EB.1080508@inktank.com \
--to=elder@inktank.com \
--cc=ceph-devel@vger.kernel.org \
--cc=greg@inktank.com \
--cc=josh.durgin@inktank.com \
--cc=yehuda@inktank.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox