From deepakcs at linux.vnet.ibm.com Thu Feb 7 04:16:05 2013 Content-Type: multipart/mixed; boundary="===============4446000296132400035==" MIME-Version: 1.0 From: Deepak C Shetty To: devel at ovirt.org Subject: Re: [Engine-devel] NPE during addStorageServer command Date: Thu, 07 Feb 2013 14:41:10 +0530 Message-ID: <51136FAE.3050609@linux.vnet.ibm.com> In-Reply-To: 51132E67.4030707@redhat.com --===============4446000296132400035== 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. --------------040502050501060904020109 Content-Type: text/plain; charset=3DISO-8859-1; format=3Dflowed Content-Transfer-Encoding: 7bit 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. --------------040502050501060904020109 Content-Type: text/html; charset=3DISO-8859-1 Content-Transfer-Encoding: 7bit
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/883= 4
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.
--------------040502050501060904020109-- --===============4446000296132400035== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4KLS0tLS0tLS0tLS0t LS0wNDA1MDIwNTA1MDEwNjA5MDQwMjAxMDkKQ29udGVudC1UeXBlOiB0ZXh0L3BsYWluOyBjaGFy c2V0PUlTTy04ODU5LTE7IGZvcm1hdD1mbG93ZWQKQ29udGVudC1UcmFuc2Zlci1FbmNvZGluZzog N2JpdAoKT24gMDIvMDcvMjAxMyAxMDowMiBBTSwgU2FoaW5hIEJvc2Ugd3JvdGU6Cj4KPiBPbiAw Mi8wNi8yMDEzIDA1OjM2IFBNLCBTaGlyZWVzaCBBbmphbCB3cm90ZToKPj4gT24gMDIvMDYvMjAx MyAwNDo1NSBQTSwgU2FoaW5hIEJvc2Ugd3JvdGU6Cj4+PiBMb29raW5nIGF0IHRoZSBjb2RlLCBp dCBsb29rcyBsaWtlIHlvdSBuZWVkIGEgU3RvcmFnZSBoZWxwZXIgY2FsbGVkIAo+Pj4gR0xVU1RF UkZTU3RvcmFnZUhlbHBlciBpbiBwYWNrYWdlIG9yZy5vdmlydC5lbmdpbmUuY29yZS5ibGwuc3Rv cmFnZS4gCj4+Cj4+IFdoaWNoIGlzIGludHJvZHVjZWQgYnkgU2hhcmFkJ3MgdGhpcyBwYXRjaDog aHR0cDovL2dlcnJpdC5vdmlydC5vcmcvODgzNAo+PiBJIGd1ZXNzIHlvdSBkb24ndCBoYXZlIHRo aXMgcGF0Y2ggaW4geW91ciBsb2NhbCByZXBvc2l0b3J5Lgo+IEluIHRoaXMgcGF0Y2gsIHlvdSB3 aWxsIG5lZWQgdG8gcmVuYW1lIHRoZSBmaWxlIGFzIGl0J3MgY2FzZSBzZW5zaXRpdmUgCj4gb24g bGludXgtIEdsdXN0ZXJGc1N0b3JhZ2VIZWxwZXIgc2hvdWxkIGJlIEdMVVNURVJGU1N0b3JhZ2VI ZWxwZXIuCgoKVGhhbmtzIFNhaGluYSwgaSB3aWxsIGdpdmUgdGhhdCBhIHRyeS4KClNoaXJlZXNo LCBpIGhhdmUgdGhpcyBwYXRjaCBpbiBteSBsb2NhbCByZXBvLCBidXQgJ2d1ZXNzIGl0IHdvbid0 IAptYXR0ZXIsIGR1ZSB0byB0aGUgaXNzdWUgcG9pbnRlZCBieSBTYWhpbmEuCgpJIHdpbGwgdHJ5 IHRoZSBmaXggcHJvcG9zZWQgYW5kIGdldCBiYWNrLgoKLS0tLS0tLS0tLS0tLS0wNDA1MDIwNTA1 MDEwNjA5MDQwMjAxMDkKQ29udGVudC1UeXBlOiB0ZXh0L2h0bWw7IGNoYXJzZXQ9SVNPLTg4NTkt MQpDb250ZW50LVRyYW5zZmVyLUVuY29kaW5nOiA3Yml0Cgo8aHRtbD4KICA8aGVhZD4KICAgIDxt ZXRhIGNvbnRlbnQ9InRleHQvaHRtbDsgY2hhcnNldD1JU08tODg1OS0xIgogICAgICBodHRwLWVx dWl2PSJDb250ZW50LVR5cGUiPgogIDwvaGVhZD4KICA8Ym9keSBiZ2NvbG9yPSIjRkZGRkZGIiB0 ZXh0PSIjMDAwMDAwIj4KICAgIDxkaXYgY2xhc3M9Im1vei1jaXRlLXByZWZpeCI+T24gMDIvMDcv MjAxMyAxMDowMiBBTSwgU2FoaW5hIEJvc2UKICAgICAgd3JvdGU6PGJyPgogICAgPC9kaXY+CiAg ICA8YmxvY2txdW90ZSBjaXRlPSJtaWQ6NTExMzJFNjcuNDAzMDcwN0ByZWRoYXQuY29tIiB0eXBl PSJjaXRlIj4KICAgICAgPG1ldGEgY29udGVudD0idGV4dC9odG1sOyBjaGFyc2V0PUlTTy04ODU5 LTEiCiAgICAgICAgaHR0cC1lcXVpdj0iQ29udGVudC1UeXBlIj4KICAgICAgPGJyPgogICAgICA8 ZGl2IGNsYXNzPSJtb3otY2l0ZS1wcmVmaXgiPk9uIDAyLzA2LzIwMTMgMDU6MzYgUE0sIFNoaXJl ZXNoCiAgICAgICAgQW5qYWwgd3JvdGU6PGJyPgogICAgICA8L2Rpdj4KICAgICAgPGJsb2NrcXVv dGUgY2l0ZT0ibWlkOjUxMTI0NzRCLjEwOTA5MDlAcmVkaGF0LmNvbSIgdHlwZT0iY2l0ZSI+CiAg ICAgICAgPG1ldGEgY29udGVudD0idGV4dC9odG1sOyBjaGFyc2V0PUlTTy04ODU5LTEiCiAgICAg ICAgICBodHRwLWVxdWl2PSJDb250ZW50LVR5cGUiPgogICAgICAgIDxkaXYgY2xhc3M9Im1vei1j aXRlLXByZWZpeCI+T24gMDIvMDYvMjAxMyAwNDo1NSBQTSwgU2FoaW5hIEJvc2UKICAgICAgICAg IHdyb3RlOjxicj4KICAgICAgICA8L2Rpdj4KICAgICAgICA8YmxvY2txdW90ZSBjaXRlPSJtaWQ6 NTExMjNEOTEuMzAyMDVAcmVkaGF0LmNvbSIgdHlwZT0iY2l0ZSI+CiAgICAgICAgICA8bWV0YSBj b250ZW50PSJ0ZXh0L2h0bWw7IGNoYXJzZXQ9SVNPLTg4NTktMSIKICAgICAgICAgICAgaHR0cC1l cXVpdj0iQ29udGVudC1UeXBlIj4KICAgICAgICAgIExvb2tpbmcgYXQgdGhlIGNvZGUsIGl0IGxv b2tzIGxpa2UgeW91IG5lZWQgYSBTdG9yYWdlIGhlbHBlcgogICAgICAgICAgY2FsbGVkIEdMVVNU RVJGU1N0b3JhZ2VIZWxwZXIgaW4gcGFja2FnZQogICAgICAgICAgb3JnLm92aXJ0LmVuZ2luZS5j b3JlLmJsbC5zdG9yYWdlLiA8L2Jsb2NrcXVvdGU+CiAgICAgICAgPGJyPgogICAgICAgIFdoaWNo IGlzIGludHJvZHVjZWQgYnkgU2hhcmFkJ3MgdGhpcyBwYXRjaDogPGEKICAgICAgICAgIG1vei1k by1ub3Qtc2VuZD0idHJ1ZSIgY2xhc3M9Im1vei10eHQtbGluay1mcmVldGV4dCIKICAgICAgICAg IGhyZWY9Imh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLzg4MzQiPmh0dHA6Ly9nZXJyaXQub3ZpcnQu b3JnLzg4MzQ8L2E+PGJyPgogICAgICAgIEkgZ3Vlc3MgeW91IGRvbid0IGhhdmUgdGhpcyBwYXRj aCBpbiB5b3VyIGxvY2FsIHJlcG9zaXRvcnkuPGJyPgogICAgICA8L2Jsb2NrcXVvdGU+CiAgICAg IEluIHRoaXMgcGF0Y2gsIHlvdSB3aWxsIG5lZWQgdG8gcmVuYW1lIHRoZSBmaWxlIGFzIGl0J3Mg Y2FzZQogICAgICBzZW5zaXRpdmUgb24gbGludXgtIEdsdXN0ZXJGc1N0b3JhZ2VIZWxwZXIgc2hv dWxkIGJlCiAgICAgIEdMVVNURVJGU1N0b3JhZ2VIZWxwZXIuPGJyPgogICAgPC9ibG9ja3F1b3Rl PgogICAgPGJyPgogICAgPGJyPgogICAgVGhhbmtzIFNhaGluYSwgaSB3aWxsIGdpdmUgdGhhdCBh IHRyeS48YnI+CiAgICA8YnI+CiAgICBTaGlyZWVzaCwgaSBoYXZlIHRoaXMgcGF0Y2ggaW4gbXkg bG9jYWwgcmVwbywgYnV0ICdndWVzcyBpdCB3b24ndAogICAgbWF0dGVyLCBkdWUgdG8gdGhlIGlz c3VlIHBvaW50ZWQgYnkgU2FoaW5hLjxicj4KICAgIDxicj4KICAgIEkgd2lsbCB0cnkgdGhlIGZp eCBwcm9wb3NlZCBhbmQgZ2V0IGJhY2suPGJyPgogIDwvYm9keT4KPC9odG1sPgoKLS0tLS0tLS0t LS0tLS0wNDA1MDIwNTA1MDEwNjA5MDQwMjAxMDktLQoK --===============4446000296132400035==--