Re: Build failed in Jenkins: build-master-slowbuild #1887

2015-07-24 Thread Rajani Karuturi
yes daan. it need not be static as its singleton. PR in progress. ~Rajani On Fri, Jul 24, 2015 at 2:16 PM, Daan Hoogland wrote: > Rajani, applicationCtx is static but being set by the override > setApplicationContext(..) which is not static. Does is need to be > static? If so an extra wrapper m

Re: Build failed in Jenkins: build-master-slowbuild #1887

2015-07-24 Thread Daan Hoogland
Rajani, applicationCtx is static but being set by the override setApplicationContext(..) which is not static. Does is need to be static? If so an extra wrapper method will kill the warning (not the issue). As the LdapUserManagerFactory is a singleton I don't think the appctx needs to be static. Ag

Build failed in Jenkins: build-master-slowbuild #1887

2015-07-24 Thread jenkins
See Changes: [Rajani Karuturi] CLOUDSTACK-8596 ability to query nested groups for Microsoft AD [Rajani Karuturi] reverting gmaven version to 1.3 [Rajani Karuturi] CLOUDSTACK-8596 addressed review comments [koushik] CLOUD