All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wido den Hollander <wido@widodh.nl>
To: Tommi Virtanen <tv@inktank.com>
Cc: ceph-devel <ceph-devel@vger.kernel.org>
Subject: Re: [PATCH] docs: Add CloudStack documentation
Date: Thu, 06 Sep 2012 00:39:23 +0200	[thread overview]
Message-ID: <5047D49B.3060105@widodh.nl> (raw)
In-Reply-To: <CADvuQREoGMAL+fKTMpYDk73-8f5yWVNHTAjpiHrLFEO6noFJ4w@mail.gmail.com>



On 09/05/2012 08:22 PM, Tommi Virtanen wrote:
> On Wed, Sep 5, 2012 at 10:05 AM, Wido den Hollander <wido@widodh.nl> wrote:
>>> For example, rbd:?id=admin&secret=s3kr1t&mon=1.2.3.4&mon=5.6.7.8&pool=rbd
>>> is perfectly legal.
>>>
>>> Whether some Java library fails to implement generic URIs is another
>>> concern..
>> It is indeed a Java library in this case:
>> http://docs.oracle.com/javase/6/docs/api/java/net/URI.html
>
> I don't see anything on that page to prevent you from doing
> u.getQuery() and using the fields from that (and then living with Java
> lack of good stdlib and using something like
> http://stackoverflow.com/questions/1667278/parsing-query-strings-in-java
> to actually parse the string into key=value pairs).
>

The main problem with that is how CloudStack internally stores the data. 
At the storage driver the URI doesn't arrive in plain format, it gets 
splitted with getHost(), getAuthUsername(), getPath() and arrives in 
these separate variables at the point where libvirt is being called.

Wido

> Or, use *exactly* what the QEmu RBD code is doing, by getting the data
> our of the URI with getSchemeSpecificPart -- that'll work as long as
> you don't start the part with a slash ("rbd:/").
>
> I see no reason here why the
> rbd:poolname/devicename[@snapshotname][:option1=value1[:option2=value2...]]
> syntax from QEmu wouldn't work, just as is.
>

  reply	other threads:[~2012-09-05 22:39 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-08 21:51 [PATCH] docs: Add CloudStack documentation Wido den Hollander
2012-09-04 23:49 ` Sage Weil
2012-09-05 15:21 ` Calvin Morrow
2012-09-05 15:28   ` Wido den Hollander
2012-09-05 15:42     ` Sage Weil
2012-09-05 16:14     ` Tommi Virtanen
2012-09-05 17:05       ` Wido den Hollander
2012-09-05 18:22         ` Tommi Virtanen
2012-09-05 22:39           ` Wido den Hollander [this message]
2012-09-05 22:52             ` 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=5047D49B.3060105@widodh.nl \
    --to=wido@widodh.nl \
    --cc=ceph-devel@vger.kernel.org \
    --cc=tv@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 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.