Thanks for spotting the issue in the first place :-) Robert
On Wed, 2015-08-05 at 17:46 +0200, Julian Sedding wrote: > Thanks for following up on this Robert! > > Regards > Julian > > On Tue, Aug 4, 2015 at 11:02 AM, Robert Munteanu <romb...@apache.org> > wrote: > > Hi Julian, > > > > On Mon, 2015-08-03 at 20:51 +0200, Julian Sedding wrote: > > > Hi Robert > > > > > > It seems that this commit causes up to ~900 tests to fail > > > (sporadically in two groups), both on Java 1.7 and 1.8. Before > > > this > > > we > > > had only few sporadically failing tests. Could you take a look if > > > your > > > commit is the true cause or if the Jenkins builds give the wrong > > > impression[0,1]. > > > > Unfortunately it's my commit, so reverted this change in r1694016. > > I > > wasn't able to fix the jackrabbit.server bundle in a short time ( > > I'll > > send a separate email about this ) so I'll have to start digging > > again > > :-) > > > > Thanks, > > > > Robert > > > > > > > > > > > > Thanks. > > > > > > Regards > > > Julian > > > > > > [0] https://builds.apache.org/job/sling-trunk-1.7/2091/ > > > [1] https://builds.apache.org/job/sling-trunk-1.8/1379/ > > > > > > > > > On Thu, Jul 30, 2015 at 9:49 AM, <romb...@apache.org> wrote: > > > > Author: rombert > > > > Date: Thu Jul 30 07:49:31 2015 > > > > New Revision: 1693392 > > > > > > > > URL: http://svn.apache.org/r1693392 > > > > Log: > > > > SLING-4556 - NPE in DiscoveryServiceImpl#activate due to > > > > OakSlingRepositoryManager restart during startup > > > > > > > > Use start level ordering to ensure that the > > > > OakSlingRepositoryManager is > > > > only activated once all dependencies are activated and > > > > configured: > > > > > > > > - Move org.apache.sling.installer.factory.configuration to the > > > > :boot > > > > feature so that components are configured as soon as they > > > > become > > > > availalble, otherwise that would've happened at start level 20 > > > > - Move the org.apache.sling.jcr.oak.server bundle to start > > > > level > > > > 16, so > > > > that all the bundles which provide its dependendencies, mostly > > > > oak > > > > -core, > > > > are properly configured when the OakSlingRepositoryManager is > > > > activated > > > > > > > > Modified: > > > > > > > > sling/trunk/launchpad/builder/src/main/provisioning/boot.txt > > > > sling/trunk/launchpad/builder/src/main/provisioning/oak.txt > > > > > > > > sling/trunk/launchpad/builder/src/main/provisioning/sling.txt > > > > > > > > Modified: > > > > sling/trunk/launchpad/builder/src/main/provisioning/boot.txt > > > > URL: > > > > http://svn.apache.org/viewvc/sling/trunk/launchpad/builder/src > > > > /main/provisioning/boot.txt?rev=1693392&r1=1693391&r2=1693392&v > > > > iew= > > > > diff > > > > =============================================================== > > > > ==== > > > > =========== > > > > --- > > > > sling/trunk/launchpad/builder/src/main/provisioning/boot.txt > > > > (original) > > > > +++ > > > > sling/trunk/launchpad/builder/src/main/provisioning/boot.txt > > > > Thu Jul 30 07:49:31 2015 > > > > @@ -43,7 +43,8 @@ > > > > > > > > org.apache.sling/org.apache.sling.launchpad.installer/1.2.2 > > > > org.apache.sling/org.apache.sling.installer.core/3.6.6 > > > > > > > > org.apache.sling/org.apache.sling.installer.provider.file/1.1. > > > > 0 > > > > + > > > > org.apache.sling/org.apache.sling.installer.factory.configurat > > > > ion/ > > > > 1.1.2 > > > > org.apache.felix/org.apache.felix.configadmin/1.8.6 > > > > org.apache.felix/org.apache.felix.eventadmin/1.4.2 > > > > # dependency with javax.inject - required for Sling Models > > > > (SLING-4710) > > > > - org.apache.geronimo.specs/geronimo-atinject_1.0_spec/1.0 > > > > + org.apache.geronimo.specs/geronimo-atinject_1.0_spec/1.0 > > > > \ No newline at end of file > > > > > > > > Modified: > > > > sling/trunk/launchpad/builder/src/main/provisioning/oak.txt > > > > URL: > > > > http://svn.apache.org/viewvc/sling/trunk/launchpad/builder/src > > > > /main/provisioning/oak.txt?rev=1693392&r1=1693391&r2=1693392&vi > > > > ew=d > > > > iff > > > > =============================================================== > > > > ==== > > > > =========== > > > > --- sling/trunk/launchpad/builder/src/main/provisioning/oak.txt > > > > (original) > > > > +++ sling/trunk/launchpad/builder/src/main/provisioning/oak.txt > > > > Thu > > > > Jul 30 07:49:31 2015 > > > > @@ -27,7 +27,6 @@ > > > > org.apache.felix/org.apache.felix.jaas/0.0.2 > > > > > > > > [artifacts startLevel=15 runModes=oak] > > > > - org.apache.sling/org.apache.sling.jcr.oak.server/0.0.2 > > > > -SNAPSHOT > > > > com.google.guava/guava/15.0 > > > > org.apache.jackrabbit/oak-core/${oak.version} > > > > org.apache.jackrabbit/oak-commons/${oak.version} > > > > @@ -37,6 +36,11 @@ > > > > [artifacts startLevel=15 runModes=oak_mongo] > > > > org.mongodb/mongo-java-driver/2.13.0 > > > > > > > > +# start the Oak server instance after all components have been > > > > configured > > > > +# and started to avoid restarting that component ( see SLING > > > > -4556 > > > > ) > > > > +[artifacts startLevel=16 runModes=oak] > > > > + org.apache.sling/org.apache.sling.jcr.oak.server/0.0.2 > > > > -SNAPSHOT > > > > + > > > > [configurations runModes=oak] > > > > org.apache.felix.jaas.Configuration.factory-GuestLoginModule > > > > jaas.controlFlag="optional" > > > > > > > > Modified: > > > > sling/trunk/launchpad/builder/src/main/provisioning/sling.txt > > > > URL: > > > > http://svn.apache.org/viewvc/sling/trunk/launchpad/builder/src > > > > /main/provisioning/sling.txt?rev=1693392&r1=1693391&r2=1693392& > > > > view > > > > =diff > > > > =============================================================== > > > > ==== > > > > =========== > > > > --- > > > > sling/trunk/launchpad/builder/src/main/provisioning/sling.txt > > > > (original) > > > > +++ > > > > sling/trunk/launchpad/builder/src/main/provisioning/sling.txt > > > > Thu Jul 30 07:49:31 2015 > > > > @@ -71,7 +71,6 @@ > > > > org.apache.sling/org.apache.sling.models.impl/1.2.0 > > > > org.apache.felix/org.apache.felix.http.whiteboard/2.3.2 > > > > org.apache.sling/org.apache.sling.installer.console/1.0.0 > > > > - > > > > org.apache.sling/org.apache.sling.installer.factory.configurat > > > > ion/ > > > > 1.1.2 > > > > > > > > org.apache.sling/org.apache.sling.installer.provider.jcr/3.1.1 > > > > 6 > > > > > > > > [artifacts startLevel=5] > > > > > > > > > >
