Bug 1321991

Summary:	failover serverset don't work - getting null objects as addreses
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	nijin ashok <nashok>
Component:	ovirt-engine-extension-aaa-ldap	Assignee:	Martin Perina <mperina>
Status:	CLOSED ERRATA	QA Contact:	Gonza <grafuls>
Severity:	medium	Docs Contact:
Priority:	high
Version:	3.6.3	CC:	bazulay, bugs, grafuls, lsurette, mgoldboi, mperina, omachace, pbrilla, pstehlik, Rhev-m-bugs, ykaul
Target Milestone:	ovirt-3.6.5	Keywords:	ZStream
Target Release:	3.6.5
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	1313516	Environment:
Last Closed:	2016-04-20 16:39:52 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Infra	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:	1313516
Bug Blocks:

Description nijin ashok 2016-03-29 14:18:13 UTC

+++ This bug was initially created as a clone of Bug #1313516 +++

Description of problem:
There is an issue getting serverset servers/ports.
When creating serversets in Framework.java, we get list of addresses from map which contain also ".port=xxxx",
which lead to having null in list. For most of the serversets it's not an issue, but FailoverServers must not contain null as address.


Version-Release number of selected component (if applicable):
1.1.2

How reproducible:
always

--- Additional comment from Ondra Machacek on 2016-03-02 03:43:56 EST ---

Example configuration:
pool.default.serverset.type = failover
pool.default.serverset.failover.00.server = server1
pool.default.serverset.failover.01.server = server2

Exception:
WARNING [ovirt-engine-extension-aaa-ldap.authn::ipa-authn] Cannot initialize LDAP framework, deferring initialization. Error: A null object was provided where a non-null object is required (non-null index 0).  Thread stack trace:  getStackTrace(Thread.java:1589) / ensureNotNull(Validator.java:60) / <init>(SingleServerSet.java:140) / <init>(FailoverServerSet.java:289) / createConnectionPool(Framework.java:549) / createConnectionPoolEntry(Framework.java:667) / getConnectionPoolEntry(Framework.java:782) / runSequence(Framework.java:1505) / open(Framework.java:719) / ensureFramework(AuthnExtension.java:49) / doAuthenticateCredentials(AuthnExtension.java:145) / invoke(AuthnExtension.java:70) / invoke(ExtensionProxy.java:49) / invoke(ExtensionProxy.java:73) / invoke(ExtensionProxy.java:109) / execute(AAAServiceImpl.java:160) / execute(AAAServiceImpl.java:423) / run(AAAServiceImpl.java:694) / main(ExtensionsToolExecutor.java:103) / invoke0(NativeMethodAccessorImpl.java) / invoke(NativeMethodAccessorImpl.java:57) / invoke(DelegatingMethodAccessorImpl.java:43) / invoke(Method.java:606) / run(Module.java:312) / main(Main.java:473)
2016-03-02 10:40:44 SEVERE  A null object was provided where a non-null object is required (non-null index 0).  Thread stack trace:  getStackTrace(Thread.java:1589) / ensureNotNull(Validator.java:60) / <init>(SingleServerSet.java:140) / <init>(FailoverServerSet.java:289) / createConnectionPool(Framework.java:549) / createConnectionPoolEntry(Framework.java:667) / getConnectionPoolEntry(Framework.java:782) / runSequence(Framework.java:1505) / open(Framework.java:719) / ensureFramework(AuthnExtension.java:49) / doAuthenticateCredentials(AuthnExtension.java:145) / invoke(AuthnExtension.java:70) / invoke(ExtensionProxy.java:49) / invoke(ExtensionProxy.java:73) / invoke(ExtensionProxy.java:109) / execute(AAAServiceImpl.java:160) / execute(AAAServiceImpl.java:423) / run(AAAServiceImpl.java:694) / main(ExtensionsToolExecutor.java:103) / invoke0(NativeMethodAccessorImpl.java) / invoke(NativeMethodAccessorImpl.java:57) / invoke(DelegatingMethodAccessorImpl.java:43) / invoke(Method.java:606) / run(Module.java:312) / main(Main.java:473)

Workaround:
In jar file:
/usr/share/ovirt-engine-extension-aaa-ldap/modules/org/ovirt/engine-extensions/aaa/ldap/main/ovirt-engine-extension-aaa-ldap.jar
Find file:
org/ovirt/engineextensions/aaa/ldap/profile-defaults.properties
There comment/remove line:
pool.default.serverset.failover.port = 389
Remember that in this case you have to always use 389 port.

--- Additional comment from Martin Perina on 2016-03-24 11:37:55 EDT ---

Fix will be included in ovirt-engine-extension-aaa-ldap 1.1.3 package

Comment 2 Martin Perina 2016-03-30 07:12:59 UTC

Targeting to 3.6.5 same as upstream bug

Comment 4 Martin Perina 2016-03-31 15:06:19 UTC

Back to modified as ovirt-engine-extension-aaa-ldap-1.1.3 will be included in next 3.6.5 build

Comment 6 Gonza 2016-04-06 09:34:47 UTC

Verified with:
ovirt-engine-extension-aaa-ldap-1.1.3-1.el6ev.noarch

[Conf values]
pool.default.serverset.type = failover
pool.default.serverset.failover.00.server = xxxlc.xxx.rhev.lab.eng.brq.redhat.com
pool.default.serverset.failover.01.server = xxxpc.xxx.rhev.lab.eng.brq.redhat.com
pool.default.serverset.failover.domain = ${global:vars.domain}

[Log]
2016-04-06 11:26:39,525 INFO  [org.ovirt.engineextensions.aaa.ldap.Framework] (ServerService Thread Pool -- 60) [] [ovirt-engine-extension-aaa-ldap.authn::ldap-auth-simple_ad] Creating LDAP pool 'authz'
2016-04-06 11:26:43,130 INFO  [org.ovirt.engineextensions.aaa.ldap.Framework] (ServerService Thread Pool -- 60) [] [ovirt-engine-extension-aaa-ldap.authn::ldap-auth-simple_ad] LDAP pool 'authz' information: vendor='null' version='null'
2016-04-06 11:26:43,134 INFO  [org.ovirt.engineextensions.aaa.ldap.Framework] (ServerService Thread Pool -- 60) [] [ovirt-engine-extension-aaa-ldap.authn::ldap-auth-simple_ad] Creating LDAP pool 'authn'
2016-04-06 11:26:43,494 INFO  [org.ovirt.engineextensions.aaa.ldap.Framework] (ServerService Thread Pool -- 60) [] [ovirt-engine-extension-aaa-ldap.authn::ldap-auth-simple_ad] LDAP pool 'authn' information: vendor='null' version='null'
2016-04-06 11:26:43,747 INFO  [org.ovirt.engineextensions.aaa.ldap.Framework] (ServerService Thread Pool -- 60) [] [ovirt-engine-extension-aaa-ldap.authn::ldap-auth-simple_ad] Creating LDAP pool 'xxx.rhev.lab.eng.brq.redhat.com'
2016-04-06 11:26:44,939 INFO  [org.ovirt.engineextensions.aaa.ldap.Framework] (ServerService Thread Pool -- 60) [] [ovirt-engine-extension-aaa-ldap.authn::ldap-auth-simple_ad] LDAP pool 'authz.lab.eng.brq.redhat.com' information: vendor='null' version='null'

Comment 8 errata-xmlrpc 2016-04-20 16:39:52 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://rhn.redhat.com/errata/RHBA-2016-0663.html