From noc at nieuwland.nl Wed Nov 19 03:37:20 2014 Content-Type: multipart/mixed; boundary="===============3994094504204108490==" MIME-Version: 1.0 From: noc To: users at ovirt.org Subject: Re: [ovirt-users] Status libgfapi support in oVirt Date: Wed, 19 Nov 2014 09:36:28 +0100 Message-ID: <546C568C.7020000@nieuwland.nl> In-Reply-To: CAC1p533uzPA6m96oMFumCZobUeULVnuD5GT82wepCv1rwkfYnw@mail.gmail.com --===============3994094504204108490== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable This is a multi-part message in MIME format. --------------070607040303030304040205 Content-Type: text/plain; charset=3Dwindows-1252 Content-Transfer-Encoding: 7bit On 18-11-2014 20:57, Christopher Young wrote: > I'm replying to 'up' this as well as I'm most interested in this. I > actually thought this was implemented and working too. > > On Mon, Nov 17, 2014 at 10:01 AM, Daniel Helgenberger > > > wrote: > > Hello Joop, > > thanks for raising the issue as it is one of the things I assumed are > already implemented and working. > > Sadly I cannot provide any answer ... > > On 17.11.2014 09:39, Joop wrote: > > I have been trying to use libgfapi glusterfs support in oVirt > but can't > > get it to work. After talks on IRC it seems I should apply a patch > > (http://gerrit.ovirt.org/33768) to enable libgf BUT I can't get > it to > > work. Systems used: > > - hosts Centos7 or Fedora20 (so upto date qemu/libvirt/oVirt(3.5)) > > - glusterfs-3.6.1 > > - vdsm-4.16.0-524.gitbc618a4.el7.x86_64 (snapshot master 14-nov) > > - vdsm-4.16.7-1.gitdb83943.el7.x86_64 (official ovirt-3.5 vdsm, > seems > > newer than master snapshot?? ) > > > > Just adding the patch to vdsm-4.16.7-1.gitdb83943.el7.x86_64 doesn't > > work, vdsm doesn't start anymore due to an error in virt/vm.py. > > > > Q1: what is de exact status of libgf and oVirt. > > Q2: how do I test that patch? > I experimented a little more and found that if I create a VM in oVirt on a glusterfs storage domain and start it, it won't use libgfapi, BUT if I use virsh on the host where the VM runs and then add a disk the libgfapi way the VM will see the disk and can use it. So the underlying infra is capable of using libgf but oVirt isn't using it. Thats where the patch comes in I think but I can't get it to work. Q: how do I test that patch I'm able to get around gcc/patch/git etc so don't need a n00b step by step but a little how to get started is very welcome. Thanks in advance, Joop --------------070607040303030304040205 Content-Type: text/html; charset=3Dwindows-1252 Content-Transfer-Encoding: 8bit
On 18-11-2014 20:57, Christopher Young wrote:
I'm replying to 'up' this as well as I'm most interested in this.=C2=A0 I actually thought this was implemented a= nd working too.

On Mon, Nov 17, 2014 at 10:01 AM, Daniel Helgenberger <daniel.helgenberger(a)m-box.de> wrote:
Hello Joop,

thanks for raising the issue as it is one of the things I assumed are
already implemented and working.

Sadly I cannot provide any answer ...

On 17.11.2014 09:39, Joop wrote:
> I have been trying to use libgfapi glusterfs support in oVirt but can't
> get it to work. After talks on IRC it seems I should apply a patch
> (= http://gerrit.ovirt.org/33768) to enable libgf BUT I can't get it to
> work. Systems used:
> - hosts Centos7 or Fedora20 (so upto date qemu/libvirt/oVirt(3.5))
> - glusterfs-3.6.1
> - vdsm-4.16.0-524.gitbc618a4.el7.x86_64 (snapshot master 14-nov)
> - vdsm-4.16.7-1.gitdb83943.el7.x86_64 (official ovirt-3.5 vdsm, seems
> newer than master snapshot?? )
>
> Just adding the patch to vdsm-4.16.7-1.gitdb83943.el7.x86_64 doesn't
> work, vdsm doesn't start anymore due to an error in virt/vm.py.
>
> Q1: what is de exact status of libgf and oVirt.
> Q2: how do I test that patch?
I experimented a little more and found that if I create a VM in oVirt on a glusterfs storage domain and start it, it won't use libgfapi, BUT if I use virsh on the host where the VM runs and then add a disk the libgfapi way the VM will see the disk and can use it. So the underlying infra is capable of using libgf but oVirt isn't using it. Thats where the patch comes in I think but I can't get it to work.

Q: how do I test that patch

I'm able to get around gcc/patch/git etc so don't need a n00b step by step but a little how to get started is very welcome.

Thanks in advance,

Joop

--------------070607040303030304040205-- --===============3994094504204108490== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4KLS0tLS0tLS0tLS0t LS0wNzA2MDcwNDAzMDMwMzAzMDQwNDAyMDUKQ29udGVudC1UeXBlOiB0ZXh0L3BsYWluOyBjaGFy c2V0PXdpbmRvd3MtMTI1MgpDb250ZW50LVRyYW5zZmVyLUVuY29kaW5nOiA3Yml0CgpPbiAxOC0x MS0yMDE0IDIwOjU3LCBDaHJpc3RvcGhlciBZb3VuZyB3cm90ZToKPiBJJ20gcmVwbHlpbmcgdG8g J3VwJyB0aGlzIGFzIHdlbGwgYXMgSSdtIG1vc3QgaW50ZXJlc3RlZCBpbiB0aGlzLiAgSQo+IGFj dHVhbGx5IHRob3VnaHQgdGhpcyB3YXMgaW1wbGVtZW50ZWQgYW5kIHdvcmtpbmcgdG9vLgo+Cj4g T24gTW9uLCBOb3YgMTcsIDIwMTQgYXQgMTA6MDEgQU0sIERhbmllbCBIZWxnZW5iZXJnZXIKPiA8 ZGFuaWVsLmhlbGdlbmJlcmdlckBtLWJveC5kZSA8bWFpbHRvOmRhbmllbC5oZWxnZW5iZXJnZXJA bS1ib3guZGU+Pgo+IHdyb3RlOgo+Cj4gICAgIEhlbGxvIEpvb3AsCj4KPiAgICAgdGhhbmtzIGZv ciByYWlzaW5nIHRoZSBpc3N1ZSBhcyBpdCBpcyBvbmUgb2YgdGhlIHRoaW5ncyBJIGFzc3VtZWQg YXJlCj4gICAgIGFscmVhZHkgaW1wbGVtZW50ZWQgYW5kIHdvcmtpbmcuCj4KPiAgICAgU2FkbHkg SSBjYW5ub3QgcHJvdmlkZSBhbnkgYW5zd2VyIC4uLgo+Cj4gICAgIE9uIDE3LjExLjIwMTQgMDk6 MzksIEpvb3Agd3JvdGU6Cj4gICAgID4gSSBoYXZlIGJlZW4gdHJ5aW5nIHRvIHVzZSBsaWJnZmFw aSBnbHVzdGVyZnMgc3VwcG9ydCBpbiBvVmlydAo+ICAgICBidXQgY2FuJ3QKPiAgICAgPiBnZXQg aXQgdG8gd29yay4gQWZ0ZXIgdGFsa3Mgb24gSVJDIGl0IHNlZW1zIEkgc2hvdWxkIGFwcGx5IGEg cGF0Y2gKPiAgICAgPiAoaHR0cDovL2dlcnJpdC5vdmlydC5vcmcvMzM3NjgpIHRvIGVuYWJsZSBs aWJnZiBCVVQgSSBjYW4ndCBnZXQKPiAgICAgaXQgdG8KPiAgICAgPiB3b3JrLiBTeXN0ZW1zIHVz ZWQ6Cj4gICAgID4gLSBob3N0cyBDZW50b3M3IG9yIEZlZG9yYTIwIChzbyB1cHRvIGRhdGUgcWVt dS9saWJ2aXJ0L29WaXJ0KDMuNSkpCj4gICAgID4gLSBnbHVzdGVyZnMtMy42LjEKPiAgICAgPiAt IHZkc20tNC4xNi4wLTUyNC5naXRiYzYxOGE0LmVsNy54ODZfNjQgKHNuYXBzaG90IG1hc3RlciAx NC1ub3YpCj4gICAgID4gLSB2ZHNtLTQuMTYuNy0xLmdpdGRiODM5NDMuZWw3Lng4Nl82NCAob2Zm aWNpYWwgb3ZpcnQtMy41IHZkc20sCj4gICAgIHNlZW1zCj4gICAgID4gbmV3ZXIgdGhhbiBtYXN0 ZXIgc25hcHNob3Q/PyApCj4gICAgID4KPiAgICAgPiBKdXN0IGFkZGluZyB0aGUgcGF0Y2ggdG8g dmRzbS00LjE2LjctMS5naXRkYjgzOTQzLmVsNy54ODZfNjQgZG9lc24ndAo+ICAgICA+IHdvcmss IHZkc20gZG9lc24ndCBzdGFydCBhbnltb3JlIGR1ZSB0byBhbiBlcnJvciBpbiB2aXJ0L3ZtLnB5 Lgo+ICAgICA+Cj4gICAgID4gUTE6IHdoYXQgaXMgZGUgZXhhY3Qgc3RhdHVzIG9mIGxpYmdmIGFu ZCBvVmlydC4KPiAgICAgPiBRMjogaG93IGRvIEkgdGVzdCB0aGF0IHBhdGNoPwo+CkkgZXhwZXJp bWVudGVkIGEgbGl0dGxlIG1vcmUgYW5kIGZvdW5kIHRoYXQgaWYgSSBjcmVhdGUgYSBWTSBpbiBv VmlydCBvbgphIGdsdXN0ZXJmcyBzdG9yYWdlIGRvbWFpbiBhbmQgc3RhcnQgaXQsIGl0IHdvbid0 IHVzZSBsaWJnZmFwaSwgQlVUIGlmIEkKdXNlIHZpcnNoIG9uIHRoZSBob3N0IHdoZXJlIHRoZSBW TSBydW5zIGFuZCB0aGVuIGFkZCBhIGRpc2sgdGhlIGxpYmdmYXBpCndheSB0aGUgVk0gd2lsbCBz ZWUgdGhlIGRpc2sgYW5kIGNhbiB1c2UgaXQuIFNvIHRoZSB1bmRlcmx5aW5nIGluZnJhIGlzCmNh cGFibGUgb2YgdXNpbmcgbGliZ2YgYnV0IG9WaXJ0IGlzbid0IHVzaW5nIGl0LiBUaGF0cyB3aGVy ZSB0aGUgcGF0Y2gKY29tZXMgaW4gSSB0aGluayBidXQgSSBjYW4ndCBnZXQgaXQgdG8gd29yay4K ClE6IGhvdyBkbyBJIHRlc3QgdGhhdCBwYXRjaAoKSSdtIGFibGUgdG8gZ2V0IGFyb3VuZCBnY2Mv cGF0Y2gvZ2l0IGV0YyBzbyBkb24ndCBuZWVkIGEgbjAwYiBzdGVwIGJ5CnN0ZXAgYnV0IGEgbGl0 dGxlIGhvdyB0byBnZXQgc3RhcnRlZCBpcyB2ZXJ5IHdlbGNvbWUuCgpUaGFua3MgaW4gYWR2YW5j ZSwKCkpvb3AKCgotLS0tLS0tLS0tLS0tLTA3MDYwNzA0MDMwMzAzMDMwNDA0MDIwNQpDb250ZW50 LVR5cGU6IHRleHQvaHRtbDsgY2hhcnNldD13aW5kb3dzLTEyNTIKQ29udGVudC1UcmFuc2Zlci1F bmNvZGluZzogOGJpdAoKPGh0bWw+CiAgPGhlYWQ+CiAgICA8bWV0YSBjb250ZW50PSJ0ZXh0L2h0 bWw7IGNoYXJzZXQ9d2luZG93cy0xMjUyIgogICAgICBodHRwLWVxdWl2PSJDb250ZW50LVR5cGUi PgogIDwvaGVhZD4KICA8Ym9keSBiZ2NvbG9yPSIjRkZGRkZGIiB0ZXh0PSIjMDAwMDAwIj4KICAg IDxkaXYgY2xhc3M9Im1vei1jaXRlLXByZWZpeCI+T24gMTgtMTEtMjAxNCAyMDo1NywgQ2hyaXN0 b3BoZXIgWW91bmcKICAgICAgd3JvdGU6PGJyPgogICAgPC9kaXY+CiAgICA8YmxvY2txdW90ZQpj aXRlPSJtaWQ6Q0FDMXA1MzN1elBBNm05Nm9NRnVtQ1pvYlVlVUxWbnVENUdUODJ3ZXBDdjFyd2tm WW53QG1haWwuZ21haWwuY29tIgogICAgICB0eXBlPSJjaXRlIj4KICAgICAgPGRpdiBkaXI9Imx0 ciI+SSdtIHJlcGx5aW5nIHRvICd1cCcgdGhpcyBhcyB3ZWxsIGFzIEknbSBtb3N0CiAgICAgICAg aW50ZXJlc3RlZCBpbiB0aGlzLqAgSSBhY3R1YWxseSB0aG91Z2h0IHRoaXMgd2FzIGltcGxlbWVu dGVkIGFuZAogICAgICAgIHdvcmtpbmcgdG9vLjwvZGl2PgogICAgICA8ZGl2IGNsYXNzPSJnbWFp bF9leHRyYSI+PGJyPgogICAgICAgIDxkaXYgY2xhc3M9ImdtYWlsX3F1b3RlIj5PbiBNb24sIE5v diAxNywgMjAxNCBhdCAxMDowMSBBTSwKICAgICAgICAgIERhbmllbCBIZWxnZW5iZXJnZXIgPHNw YW4gZGlyPSJsdHIiPiZsdDs8YQogICAgICAgICAgICAgIG1vei1kby1ub3Qtc2VuZD0idHJ1ZSIK ICAgICAgICAgICAgICBocmVmPSJtYWlsdG86ZGFuaWVsLmhlbGdlbmJlcmdlckBtLWJveC5kZSIg dGFyZ2V0PSJfYmxhbmsiPmRhbmllbC5oZWxnZW5iZXJnZXJAbS1ib3guZGU8L2E+Jmd0Ozwvc3Bh bj4KICAgICAgICAgIHdyb3RlOjxicj4KICAgICAgICAgIDxibG9ja3F1b3RlIGNsYXNzPSJnbWFp bF9xdW90ZSIgc3R5bGU9Im1hcmdpbjowIDAgMAogICAgICAgICAgICAuOGV4O2JvcmRlci1sZWZ0 OjFweCAjY2NjIHNvbGlkO3BhZGRpbmctbGVmdDoxZXgiPkhlbGxvCiAgICAgICAgICAgIEpvb3As PGJyPgogICAgICAgICAgICA8YnI+CiAgICAgICAgICAgIHRoYW5rcyBmb3IgcmFpc2luZyB0aGUg aXNzdWUgYXMgaXQgaXMgb25lIG9mIHRoZSB0aGluZ3MgSQogICAgICAgICAgICBhc3N1bWVkIGFy ZTxicj4KICAgICAgICAgICAgYWxyZWFkeSBpbXBsZW1lbnRlZCBhbmQgd29ya2luZy48YnI+CiAg ICAgICAgICAgIDxicj4KICAgICAgICAgICAgU2FkbHkgSSBjYW5ub3QgcHJvdmlkZSBhbnkgYW5z d2VyIC4uLjxicj4KICAgICAgICAgICAgPGRpdj4KICAgICAgICAgICAgICA8ZGl2IGNsYXNzPSJo NSI+PGJyPgogICAgICAgICAgICAgICAgT24gMTcuMTEuMjAxNCAwOTozOSwgSm9vcCB3cm90ZTo8 YnI+CiAgICAgICAgICAgICAgICAmZ3Q7IEkgaGF2ZSBiZWVuIHRyeWluZyB0byB1c2UgbGliZ2Zh cGkgZ2x1c3RlcmZzCiAgICAgICAgICAgICAgICBzdXBwb3J0IGluIG9WaXJ0IGJ1dCBjYW4ndDxi cj4KICAgICAgICAgICAgICAgICZndDsgZ2V0IGl0IHRvIHdvcmsuIEFmdGVyIHRhbGtzIG9uIElS QyBpdCBzZWVtcyBJCiAgICAgICAgICAgICAgICBzaG91bGQgYXBwbHkgYSBwYXRjaDxicj4KICAg ICAgICAgICAgICAgICZndDsgKDxhIG1vei1kby1ub3Qtc2VuZD0idHJ1ZSIKICAgICAgICAgICAg ICAgICAgaHJlZj0iaHR0cDovL2dlcnJpdC5vdmlydC5vcmcvMzM3NjgiIHRhcmdldD0iX2JsYW5r Ij5odHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy8zMzc2ODwvYT4pCiAgICAgICAgICAgICAgICB0byBl bmFibGUgbGliZ2YgQlVUIEkgY2FuJ3QgZ2V0IGl0IHRvPGJyPgogICAgICAgICAgICAgICAgJmd0 OyB3b3JrLiBTeXN0ZW1zIHVzZWQ6PGJyPgogICAgICAgICAgICAgICAgJmd0OyAtIGhvc3RzIENl bnRvczcgb3IgRmVkb3JhMjAgKHNvIHVwdG8gZGF0ZQogICAgICAgICAgICAgICAgcWVtdS9saWJ2 aXJ0L29WaXJ0KDMuNSkpPGJyPgogICAgICAgICAgICAgICAgJmd0OyAtIGdsdXN0ZXJmcy0zLjYu MTxicj4KICAgICAgICAgICAgICAgICZndDsgLSB2ZHNtLTQuMTYuMC01MjQuZ2l0YmM2MThhNC5l bDcueDg2XzY0IChzbmFwc2hvdAogICAgICAgICAgICAgICAgbWFzdGVyIDE0LW5vdik8YnI+CiAg ICAgICAgICAgICAgICAmZ3Q7IC0gdmRzbS00LjE2LjctMS5naXRkYjgzOTQzLmVsNy54ODZfNjQg KG9mZmljaWFsCiAgICAgICAgICAgICAgICBvdmlydC0zLjUgdmRzbSwgc2VlbXM8YnI+CiAgICAg ICAgICAgICAgICAmZ3Q7IG5ld2VyIHRoYW4gbWFzdGVyIHNuYXBzaG90Pz8gKTxicj4KICAgICAg ICAgICAgICAgICZndDs8YnI+CiAgICAgICAgICAgICAgICAmZ3Q7IEp1c3QgYWRkaW5nIHRoZSBw YXRjaCB0bwogICAgICAgICAgICAgICAgdmRzbS00LjE2LjctMS5naXRkYjgzOTQzLmVsNy54ODZf NjQgZG9lc24ndDxicj4KICAgICAgICAgICAgICAgICZndDsgd29yaywgdmRzbSBkb2Vzbid0IHN0 YXJ0IGFueW1vcmUgZHVlIHRvIGFuIGVycm9yIGluCiAgICAgICAgICAgICAgICB2aXJ0L3ZtLnB5 Ljxicj4KICAgICAgICAgICAgICAgICZndDs8YnI+CiAgICAgICAgICAgICAgICAmZ3Q7IFExOiB3 aGF0IGlzIGRlIGV4YWN0IHN0YXR1cyBvZiBsaWJnZiBhbmQgb1ZpcnQuPGJyPgogICAgICAgICAg ICAgICAgJmd0OyBRMjogaG93IGRvIEkgdGVzdCB0aGF0IHBhdGNoPzxicj4KICAgICAgICAgICAg ICA8L2Rpdj4KICAgICAgICAgICAgPC9kaXY+CiAgICAgICAgICA8L2Jsb2NrcXVvdGU+CiAgICAg ICAgPC9kaXY+CiAgICAgIDwvZGl2PgogICAgPC9ibG9ja3F1b3RlPgogICAgSSBleHBlcmltZW50 ZWQgYSBsaXR0bGUgbW9yZSBhbmQgZm91bmQgdGhhdCBpZiBJIGNyZWF0ZSBhIFZNIGluCiAgICBv VmlydCBvbiBhIGdsdXN0ZXJmcyBzdG9yYWdlIGRvbWFpbiBhbmQgc3RhcnQgaXQsIGl0IHdvbid0 IHVzZQogICAgbGliZ2ZhcGksIEJVVCBpZiBJIHVzZSB2aXJzaCBvbiB0aGUgaG9zdCB3aGVyZSB0 aGUgVk0gcnVucyBhbmQgdGhlbgogICAgYWRkIGEgZGlzayB0aGUgbGliZ2ZhcGkgd2F5IHRoZSBW TSB3aWxsIHNlZSB0aGUgZGlzayBhbmQgY2FuIHVzZSBpdC4KICAgIFNvIHRoZSB1bmRlcmx5aW5n IGluZnJhIGlzIGNhcGFibGUgb2YgdXNpbmcgbGliZ2YgYnV0IG9WaXJ0IGlzbid0CiAgICB1c2lu ZyBpdC4gVGhhdHMgd2hlcmUgdGhlIHBhdGNoIGNvbWVzIGluIEkgdGhpbmsgYnV0IEkgY2FuJ3Qg Z2V0IGl0CiAgICB0byB3b3JrLjxicj4KICAgIDxicj4KICAgIFE6IGhvdyBkbyBJIHRlc3QgdGhh dCBwYXRjaDxicj4KICAgIDxicj4KICAgIEknbSBhYmxlIHRvIGdldCBhcm91bmQgZ2NjL3BhdGNo L2dpdCBldGMgc28gZG9uJ3QgbmVlZCBhIG4wMGIgc3RlcAogICAgYnkgc3RlcCBidXQgYSBsaXR0 bGUgaG93IHRvIGdldCBzdGFydGVkIGlzIHZlcnkgd2VsY29tZS48YnI+CiAgICA8YnI+CiAgICBU aGFua3MgaW4gYWR2YW5jZSw8YnI+CiAgICA8YnI+CiAgICBKb29wPGJyPgogICAgPGJyPgogIDwv Ym9keT4KPC9odG1sPgoKLS0tLS0tLS0tLS0tLS0wNzA2MDcwNDAzMDMwMzAzMDQwNDAyMDUtLQo= --===============3994094504204108490==--