Author: djencks Date: Thu Jan 13 10:20:38 2005 New Revision: 125086 URL: http://svn.apache.org/viewcvs?view=rev&rev=125086 Log: Fix by dain for situation where jmx connects but waited-for module is not yet loaded Modified: geronimo/trunk/plugins/maven-geronimo-plugin/src/java/org/apache/geronimo/deployment/mavenplugin/WaitForStarted.java
Modified: geronimo/trunk/plugins/maven-geronimo-plugin/src/java/org/apache/geronimo/deployment/mavenplugin/WaitForStarted.java Url: http://svn.apache.org/viewcvs/geronimo/trunk/plugins/maven-geronimo-plugin/src/java/org/apache/geronimo/deployment/mavenplugin/WaitForStarted.java?view=diff&rev=125086&p1=geronimo/trunk/plugins/maven-geronimo-plugin/src/java/org/apache/geronimo/deployment/mavenplugin/WaitForStarted.java&r1=125085&p2=geronimo/trunk/plugins/maven-geronimo-plugin/src/java/org/apache/geronimo/deployment/mavenplugin/WaitForStarted.java&r2=125086 ============================================================================== --- geronimo/trunk/plugins/maven-geronimo-plugin/src/java/org/apache/geronimo/deployment/mavenplugin/WaitForStarted.java (original) +++ geronimo/trunk/plugins/maven-geronimo-plugin/src/java/org/apache/geronimo/deployment/mavenplugin/WaitForStarted.java Thu Jan 13 10:20:38 2005 @@ -28,6 +28,7 @@ import org.apache.geronimo.kernel.jmx.KernelDelegate; import org.apache.geronimo.kernel.jmx.KernelMBean; import org.apache.geronimo.kernel.management.State; +import org.apache.geronimo.kernel.config.NoSuchConfigException; public class WaitForStarted extends AbstractModuleCommand { @@ -77,9 +78,13 @@ } URI id = new URI(getId()); for (int tries = maxTries; tries > 0; tries--) { - int state = kernel.getConfigurationState(id); - if (state == State.RUNNING_INDEX) { - return; + try { + int state = kernel.getConfigurationState(id); + if (state == State.RUNNING_INDEX) { + return; + } + } catch (NoSuchConfigException e) { + //hasn't been loaded yet, keep trying } Thread.sleep(1000); }