[Engine-devel] Adding support for OpenLDAP to oVirt

[Uwe Grawert]

Am 21.03.2013 um 18:06 schrieb Yair Zaslavsky:

> 
> 
> ----- Original Message -----
>> From: "Yair Zaslavsky" <yzaslavs at redhat.com>
>> To: "Juan Hernandez" <jhernand at redhat.com>
>> Cc: engine-devel at ovirt.org
>> Sent: Thursday, March 21, 2013 7:03:06 PM
>> Subject: Re: [Engine-devel] Adding support for OpenLDAP to oVirt
>> 
>> 
>> ----- Original Message -----
>>> From: "Juan Hernandez" <jhernand at redhat.com>
>>> To: "Uwe Grawert" <grawert at b1-systems.de>
>>> Cc: "Yair Zaslavsky" <yzaslavs at redhat.com>, engine-devel at ovirt.org
>>> Sent: Thursday, March 21, 2013 6:59:40 PM
>>> Subject: Re: [Engine-devel] Adding support for OpenLDAP to oVirt
>>> 
>>> On 03/21/2013 03:49 PM, Uwe Grawert wrote:
>>>> Hello Yair,
>>>> 
>>>> Am 20.03.2013 um 08:24 schrieb Yair Zaslavsky:
>>>> 
>>>>> Hi Uwe,
>>>>> Bare in mind we have some plans for refactoring/changing the
>>>>> authentication module.
>>>>> However, for the meanwhile - we will be very happy if you can
>>>>> assist in adding this support.
>>>>> I would like to introduce you to a Wiki that Oved Ourfali has
>>>>> written about the subject -
>>>>> 
>>>> good to know. Refactoring will probably take some time, so I hope
>>>> it's still worth the effort.
>>>> 
>>>>> http://www.ovirt.org/DomainInfrastructure
>>>>> Please let us know if this helps you with your questions.
>>>> 
>>>> So far I am simply copying what has been done for ITDS and
>>>> adopting
>>>> things to OpenLdap. I am using the git tag "ovirt-engine-3.2.1",
>>>> since I am getting the following error when using HEAD:
>>>> 
>>>> [root at ovirt-nightly ~]# engine-manage-domains -action=list
>>>> Exception in thread "main"
>>>> java.lang.ArrayIndexOutOfBoundsException: 1
>>>> 	at
>>>> 	org.ovirt.engine.core.domains.DomainsConfigurationEntry.<init>(DomainsConfigurationEntry.java:23)
>>>> 	at
>>>> 	org.ovirt.engine.core.domains.ManageDomains.getConfiguration(ManageDomains.java:423)
>>>> 	at
>>>> 	org.ovirt.engine.core.domains.ManageDomains.runCommand(ManageDomains.java:293)
>>>> 	at
>>>> 	org.ovirt.engine.core.domains.ManageDomains.main(ManageDomains.java:180)
>>>> 	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
>>>> 	at
>>>> 	sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
>>>> 	at
>>>> 	sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>>>> 	at java.lang.reflect.Method.invoke(Method.java:601)
>>>> 	at org.jboss.modules.Module.run(Module.java:260)
>>>> 	at org.jboss.modules.Main.main(Main.java:291)
>>>> 
>>>> I created only ovirt-engine RPMs and installed them with
>>>> --nodeps.
>>>> Might be that I am missing some dependency here. Anyways,
>>>> ovirt-engine-3.2.1 release tag is working, and diffs will very
>>>> likely apply to HEAD as well.
>>> 
>>> Note that we recently fixed an issue with how engine-config uses
>>> the
>>> log
>>> and the standard output which had this same symptom. You may want
>>> to
>>> apply this patch (already merged in the master branch) and then try
>>> again:
>>> 
>>> http://gerrit.ovirt.org/12987
>> 
>> Actually, I think Juan is right, and I saw this error myself when the
>> patch he mentioned was not applied.
> 
> Juan's patch was introduced *after* the commit hash you mentioned was introduced.
> Please rebase again or at least apply his patch.

Right. I was quite old. Now I'm current and it works well.

[root at ovirt-nightly ovirt-engine]# git log -1
commit c743bf9f0e7a20b8c69423aef80079c6e7ffbf80
Author: Libor Spevak <lspevak at redhat.com>
Date:   Thu Mar 21 12:35:33 2013 +0100

[root at ovirt-nightly ovirt-engine]# engine-manage-domains -action=list
Manage Domains completed successfully