From deepakcs at linux.vnet.ibm.com Fri Feb 8 12:22:44 2013 Content-Type: multipart/mixed; boundary="===============5828602471938049306==" MIME-Version: 1.0 From: Deepak C Shetty To: devel at ovirt.org Subject: Re: [Engine-devel] NPE during addStorageServer command [SOLVED} Date: Fri, 08 Feb 2013 22:47:54 +0530 Message-ID: <51153342.9050703@linux.vnet.ibm.com> In-Reply-To: 51136FAE.3050609@linux.vnet.ibm.com --===============5828602471938049306== 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. --------------030806070107090805070000 Content-Type: text/plain; charset=3DISO-8859-1; format=3Dflowed Content-Transfer-Encoding: 7bit On 02/07/2013 02:41 PM, Deepak C Shetty wrote: > On 02/07/2013 10:02 AM, Sahina Bose wrote: >> >> On 02/06/2013 05:36 PM, Shireesh Anjal wrote: >>> On 02/06/2013 04:55 PM, Sahina Bose wrote: >>>> Looking at the code, it looks like you need a Storage helper called = >>>> GLUSTERFSStorageHelper in package org.ovirt.engine.core.bll.storage. = >>> >>> Which is introduced by Sharad's this patch: http://gerrit.ovirt.org/8834 >>> I guess you don't have this patch in your local repository. >> In this patch, you will need to rename the file as it's case = >> sensitive on linux- GlusterFsStorageHelper should be = >> GLUSTERFSStorageHelper. > > > Thanks Sahina, i will give that a try. > > Shireesh, i have this patch in my local repo, but 'guess it won't = > matter, due to the issue pointed by Sahina. > > I will try the fix proposed and get back. That works.. NPE is gone with the above change. thanks all ! --------------030806070107090805070000 Content-Type: text/html; charset=3DISO-8859-1 Content-Transfer-Encoding: 7bit
On 02/07/2013 02:41 PM, Deepak C Shetty wrote:
On 02/07/2013 10:02 AM, Sahina Bose wrote:

On 02/06/2013 05:36 PM, Shireesh Anjal wrote:
On 02/06/2013 04:55 PM, Sahina Bose wrote:
Looking at the code, it looks like you need a Storage helper called GLUSTERFSStorageHelper in package org.ovirt.engine.core.bll.storage.

Which is introduced by Sharad's this patch: http://gerrit.ovirt.org/8= 834
I guess you don't have this patch in your local repository.
In this patch, you will need to rename the file as it's case sensitive on linux- GlusterFsStorageHelper should be GLUSTERFSStorageHelper.


Thanks Sahina, i will give that a try.

Shireesh, i have this patch in my local repo, but 'guess it won't matter, due to the issue pointed by Sahina.

I will try the fix proposed and get back.

That works.. NPE is gone with the above change. thanks all !
--------------030806070107090805070000-- --===============5828602471938049306== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4KLS0tLS0tLS0tLS0t LS0wMzA4MDYwNzAxMDcwOTA4MDUwNzAwMDAKQ29udGVudC1UeXBlOiB0ZXh0L3BsYWluOyBjaGFy c2V0PUlTTy04ODU5LTE7IGZvcm1hdD1mbG93ZWQKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzog N2JpdAoKT24gMDIvMDcvMjAxMyAwMjo0MSBQTSwgRGVlcGFrIEMgU2hldHR5IHdyb3RlOgo+IE9u IDAyLzA3LzIwMTMgMTA6MDIgQU0sIFNhaGluYSBCb3NlIHdyb3RlOgo+Pgo+PiBPbiAwMi8wNi8y MDEzIDA1OjM2IFBNLCBTaGlyZWVzaCBBbmphbCB3cm90ZToKPj4+IE9uIDAyLzA2LzIwMTMgMDQ6 NTUgUE0sIFNhaGluYSBCb3NlIHdyb3RlOgo+Pj4+IExvb2tpbmcgYXQgdGhlIGNvZGUsIGl0IGxv b2tzIGxpa2UgeW91IG5lZWQgYSBTdG9yYWdlIGhlbHBlciBjYWxsZWQgCj4+Pj4gR0xVU1RFUkZT U3RvcmFnZUhlbHBlciBpbiBwYWNrYWdlIG9yZy5vdmlydC5lbmdpbmUuY29yZS5ibGwuc3RvcmFn ZS4gCj4+Pgo+Pj4gV2hpY2ggaXMgaW50cm9kdWNlZCBieSBTaGFyYWQncyB0aGlzIHBhdGNoOiBo dHRwOi8vZ2Vycml0Lm92aXJ0Lm9yZy84ODM0Cj4+PiBJIGd1ZXNzIHlvdSBkb24ndCBoYXZlIHRo aXMgcGF0Y2ggaW4geW91ciBsb2NhbCByZXBvc2l0b3J5Lgo+PiBJbiB0aGlzIHBhdGNoLCB5b3Ug d2lsbCBuZWVkIHRvIHJlbmFtZSB0aGUgZmlsZSBhcyBpdCdzIGNhc2UgCj4+IHNlbnNpdGl2ZSBv biBsaW51eC0gR2x1c3RlckZzU3RvcmFnZUhlbHBlciBzaG91bGQgYmUgCj4+IEdMVVNURVJGU1N0 b3JhZ2VIZWxwZXIuCj4KPgo+IFRoYW5rcyBTYWhpbmEsIGkgd2lsbCBnaXZlIHRoYXQgYSB0cnku Cj4KPiBTaGlyZWVzaCwgaSBoYXZlIHRoaXMgcGF0Y2ggaW4gbXkgbG9jYWwgcmVwbywgYnV0ICdn dWVzcyBpdCB3b24ndCAKPiBtYXR0ZXIsIGR1ZSB0byB0aGUgaXNzdWUgcG9pbnRlZCBieSBTYWhp bmEuCj4KPiBJIHdpbGwgdHJ5IHRoZSBmaXggcHJvcG9zZWQgYW5kIGdldCBiYWNrLgoKVGhhdCB3 b3Jrcy4uIE5QRSBpcyBnb25lIHdpdGggdGhlIGFib3ZlIGNoYW5nZS4gdGhhbmtzIGFsbCAhCgot LS0tLS0tLS0tLS0tLTAzMDgwNjA3MDEwNzA5MDgwNTA3MDAwMApDb250ZW50LVR5cGU6IHRleHQv aHRtbDsgY2hhcnNldD1JU08tODg1OS0xCkNvbnRlbnQtVHJhbnNmZXItRW5jb2Rpbmc6IDdiaXQK CjxodG1sPgogIDxoZWFkPgogICAgPG1ldGEgY29udGVudD0idGV4dC9odG1sOyBjaGFyc2V0PUlT Ty04ODU5LTEiCiAgICAgIGh0dHAtZXF1aXY9IkNvbnRlbnQtVHlwZSI+CiAgPC9oZWFkPgogIDxi b2R5IGJnY29sb3I9IiNGRkZGRkYiIHRleHQ9IiMwMDAwMDAiPgogICAgPGRpdiBjbGFzcz0ibW96 LWNpdGUtcHJlZml4Ij5PbiAwMi8wNy8yMDEzIDAyOjQxIFBNLCBEZWVwYWsgQyBTaGV0dHkKICAg ICAgd3JvdGU6PGJyPgogICAgPC9kaXY+CiAgICA8YmxvY2txdW90ZSBjaXRlPSJtaWQ6NTExMzZG QUUuMzA1MDYwOUBsaW51eC52bmV0LmlibS5jb20iCiAgICAgIHR5cGU9ImNpdGUiPgogICAgICA8 bWV0YSBjb250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9SVNPLTg4NTktMSIKICAgICAgICBodHRw LWVxdWl2PSJDb250ZW50LVR5cGUiPgogICAgICA8ZGl2IGNsYXNzPSJtb3otY2l0ZS1wcmVmaXgi Pk9uIDAyLzA3LzIwMTMgMTA6MDIgQU0sIFNhaGluYSBCb3NlCiAgICAgICAgd3JvdGU6PGJyPgog ICAgICA8L2Rpdj4KICAgICAgPGJsb2NrcXVvdGUgY2l0ZT0ibWlkOjUxMTMyRTY3LjQwMzA3MDdA cmVkaGF0LmNvbSIgdHlwZT0iY2l0ZSI+CiAgICAgICAgPG1ldGEgY29udGVudD0idGV4dC9odG1s OyBjaGFyc2V0PUlTTy04ODU5LTEiCiAgICAgICAgICBodHRwLWVxdWl2PSJDb250ZW50LVR5cGUi PgogICAgICAgIDxicj4KICAgICAgICA8ZGl2IGNsYXNzPSJtb3otY2l0ZS1wcmVmaXgiPk9uIDAy LzA2LzIwMTMgMDU6MzYgUE0sIFNoaXJlZXNoCiAgICAgICAgICBBbmphbCB3cm90ZTo8YnI+CiAg ICAgICAgPC9kaXY+CiAgICAgICAgPGJsb2NrcXVvdGUgY2l0ZT0ibWlkOjUxMTI0NzRCLjEwOTA5 MDlAcmVkaGF0LmNvbSIgdHlwZT0iY2l0ZSI+CiAgICAgICAgICA8bWV0YSBjb250ZW50PSJ0ZXh0 L2h0bWw7IGNoYXJzZXQ9SVNPLTg4NTktMSIKICAgICAgICAgICAgaHR0cC1lcXVpdj0iQ29udGVu dC1UeXBlIj4KICAgICAgICAgIDxkaXYgY2xhc3M9Im1vei1jaXRlLXByZWZpeCI+T24gMDIvMDYv MjAxMyAwNDo1NSBQTSwgU2FoaW5hCiAgICAgICAgICAgIEJvc2Ugd3JvdGU6PGJyPgogICAgICAg ICAgPC9kaXY+CiAgICAgICAgICA8YmxvY2txdW90ZSBjaXRlPSJtaWQ6NTExMjNEOTEuMzAyMDVA cmVkaGF0LmNvbSIgdHlwZT0iY2l0ZSI+CiAgICAgICAgICAgIDxtZXRhIGNvbnRlbnQ9InRleHQv aHRtbDsgY2hhcnNldD1JU08tODg1OS0xIgogICAgICAgICAgICAgIGh0dHAtZXF1aXY9IkNvbnRl bnQtVHlwZSI+CiAgICAgICAgICAgIExvb2tpbmcgYXQgdGhlIGNvZGUsIGl0IGxvb2tzIGxpa2Ug eW91IG5lZWQgYSBTdG9yYWdlIGhlbHBlcgogICAgICAgICAgICBjYWxsZWQgR0xVU1RFUkZTU3Rv cmFnZUhlbHBlciBpbiBwYWNrYWdlCiAgICAgICAgICAgIG9yZy5vdmlydC5lbmdpbmUuY29yZS5i bGwuc3RvcmFnZS4gPC9ibG9ja3F1b3RlPgogICAgICAgICAgPGJyPgogICAgICAgICAgV2hpY2gg aXMgaW50cm9kdWNlZCBieSBTaGFyYWQncyB0aGlzIHBhdGNoOiA8YQogICAgICAgICAgICBtb3ot ZG8tbm90LXNlbmQ9InRydWUiIGNsYXNzPSJtb3otdHh0LWxpbmstZnJlZXRleHQiCiAgICAgICAg ICAgIGhyZWY9Imh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLzg4MzQiPmh0dHA6Ly9nZXJyaXQub3Zp cnQub3JnLzg4MzQ8L2E+PGJyPgogICAgICAgICAgSSBndWVzcyB5b3UgZG9uJ3QgaGF2ZSB0aGlz IHBhdGNoIGluIHlvdXIgbG9jYWwgcmVwb3NpdG9yeS48YnI+CiAgICAgICAgPC9ibG9ja3F1b3Rl PgogICAgICAgIEluIHRoaXMgcGF0Y2gsIHlvdSB3aWxsIG5lZWQgdG8gcmVuYW1lIHRoZSBmaWxl IGFzIGl0J3MgY2FzZQogICAgICAgIHNlbnNpdGl2ZSBvbiBsaW51eC0gR2x1c3RlckZzU3RvcmFn ZUhlbHBlciBzaG91bGQgYmUKICAgICAgICBHTFVTVEVSRlNTdG9yYWdlSGVscGVyLjxicj4KICAg ICAgPC9ibG9ja3F1b3RlPgogICAgICA8YnI+CiAgICAgIDxicj4KICAgICAgVGhhbmtzIFNhaGlu YSwgaSB3aWxsIGdpdmUgdGhhdCBhIHRyeS48YnI+CiAgICAgIDxicj4KICAgICAgU2hpcmVlc2gs IGkgaGF2ZSB0aGlzIHBhdGNoIGluIG15IGxvY2FsIHJlcG8sIGJ1dCAnZ3Vlc3MgaXQgd29uJ3QK ICAgICAgbWF0dGVyLCBkdWUgdG8gdGhlIGlzc3VlIHBvaW50ZWQgYnkgU2FoaW5hLjxicj4KICAg ICAgPGJyPgogICAgICBJIHdpbGwgdHJ5IHRoZSBmaXggcHJvcG9zZWQgYW5kIGdldCBiYWNrLjxi cj4KICAgIDwvYmxvY2txdW90ZT4KICAgIDxicj4KICAgIFRoYXQgd29ya3MuLiBOUEUgaXMgZ29u ZSB3aXRoIHRoZSBhYm92ZSBjaGFuZ2UuIHRoYW5rcyBhbGwgITxicj4KICA8L2JvZHk+CjwvaHRt bD4KCi0tLS0tLS0tLS0tLS0tMDMwODA2MDcwMTA3MDkwODA1MDcwMDAwLS0KCg== --===============5828602471938049306==--