From: ronnie sahlberg <ronniesahlberg@gmail.com>
To: Stefan Hajnoczi <stefanha@gmail.com>
Cc: Kevin Wolf <kwolf@redhat.com>, Anand Avati <aavati@redhat.com>,
Vijay Bellur <vbellur@redhat.com>,
Amar Tumballi <amarts@redhat.com>,
qemu-devel@nongnu.org, Markus Armbruster <armbru@redhat.com>,
bharata@linux.vnet.ibm.com
Subject: Re: [Qemu-devel] [RFC PATCH 0/2] GlusterFS support in QEMU - v2
Date: Mon, 23 Jul 2012 19:34:20 +1000 [thread overview]
Message-ID: <CAN05THQk4BcPoec4zsD=2m6fuefuHrvAL3k3t5XeVQ_uNHcOCg@mail.gmail.com> (raw)
In-Reply-To: <CAJSP0QXdLCV8FDAMLesZS4Y2tYZYLADUFj3-=5AyzviKjRAV3A@mail.gmail.com>
Stefan,
in iscsi, i just specify those extra arguments that are required that
are not part of the url itself as just command line options :
qemu-system-i386 -iscsi initiator-name=iqn.qemu.test:my-initiator \
-boot d -drive file=iscsi://127.0.0.1/iqn.qemu.test/1 \
-cdrom iscsi://127.0.0.1/iqn.qemu.test/2
Here initiator-name is a custom option to the iscsi layer to tell it
which name to use when identifying/logging in to the target.
Similar concept could be used by glusterfs ?
regards
ronnie sahlberg
On Mon, Jul 23, 2012 at 7:20 PM, Stefan Hajnoczi <stefanha@gmail.com> wrote:
> On Mon, Jul 23, 2012 at 9:50 AM, Bharata B Rao
> <bharata@linux.vnet.ibm.com> wrote:
>> On Sun, Jul 22, 2012 at 03:42:28PM +0100, Stefan Hajnoczi wrote:
>>> On Sat, Jul 21, 2012 at 9:29 AM, Bharata B Rao
>>> <bharata@linux.vnet.ibm.com> wrote:
>>> > -drive file=gluster:server@port:volname:image
>>> >
>>> > - Here 'gluster' is the protocol.
>>> > - 'server@port' specifies the server where the volume file specification for
>>> > the given volume resides. 'port' is the port number on which gluster
>>> > management daemon (glusterd) is listening. This is optional and if not
>>> > specified, QEMU will send 0 which will make libgfapi to use the default
>>> > port.
>>>
>>> 'server@port' is weird notation. Normally it is 'server:port' (e.g.
>>> URLs). Can you change it?
>>
>> I don't like but, but settled for it since port was optional and : was
>> being used as separator here.
>>
>>>
>>> What about the other transports supported by libgfapi: UNIX domain
>>> sockets and RDMA? My reading of glfs.h is that there are 3 connection
>>> options:
>>> 1. 'transport': 'socket' (default), 'unix', 'rdma'
>>> 2. 'host': server hostname for 'socket', path to UNIX domain socket
>>> for 'unix', or something else for 'rdma'
>>> 3. 'port': TCP port when 'socket' is used. Ignored otherwise.
>>>
>>> Unfortunately QEMU block drivers cannot take custom options yet. That
>>> would make it possible to cleanly map these connection options and
>>> save you from inventing syntax which doesn't expose all options.
>>>
>>> In the meantime it would be nice if the syntax exposed all options.
>>
>> So without the capability to pass custom options to block drivers, am I forced
>> to keep extending the file= with more and more options ?
>>
>> file=gluster:transport:server:port:volname:image ?
>>
>> Looks ugly and not easy to make any particular option optional. If needed I can
>> support this from GlusterFS backend.
>
> Kevin, Markus: Any thoughts on passing options to block drivers?
> Encoding GlusterFS options into a "filename" string is pretty
> cumbersome.
>
next prev parent reply other threads:[~2012-07-23 9:34 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-21 8:29 [Qemu-devel] [RFC PATCH 0/2] GlusterFS support in QEMU - v2 Bharata B Rao
2012-07-21 8:30 ` [Qemu-devel] [RFC PATCH 1/2] qemu: Add a config option for GlusterFS as block backend Bharata B Rao
2012-07-21 8:31 ` [Qemu-devel] [RFC PATCH 2/2] block: gluster " Bharata B Rao
2012-07-22 15:38 ` Stefan Hajnoczi
2012-07-23 8:32 ` Bharata B Rao
2012-07-23 9:06 ` Stefan Hajnoczi
2012-07-21 12:22 ` [Qemu-devel] [RFC PATCH 0/2] GlusterFS support in QEMU - v2 Vijay Bellur
2012-07-21 13:04 ` Bharata B Rao
2012-07-22 14:42 ` Stefan Hajnoczi
2012-07-23 8:50 ` Bharata B Rao
2012-07-23 9:20 ` Stefan Hajnoczi
2012-07-23 9:34 ` ronnie sahlberg [this message]
2012-07-23 9:35 ` Stefan Hajnoczi
2012-07-23 14:34 ` Eric Blake
2012-07-24 3:34 ` Bharata B Rao
2012-07-24 10:24 ` Kevin Wolf
2012-07-24 11:30 ` Markus Armbruster
2012-07-23 9:36 ` Vijay Bellur
2012-07-23 9:16 ` Daniel P. Berrange
2012-07-23 9:28 ` ronnie sahlberg
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='CAN05THQk4BcPoec4zsD=2m6fuefuHrvAL3k3t5XeVQ_uNHcOCg@mail.gmail.com' \
--to=ronniesahlberg@gmail.com \
--cc=aavati@redhat.com \
--cc=amarts@redhat.com \
--cc=armbru@redhat.com \
--cc=bharata@linux.vnet.ibm.com \
--cc=kwolf@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@gmail.com \
--cc=vbellur@redhat.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;
as well as URLs for NNTP newsgroup(s).