Make the classes that get started by jsvc implement Daemon.

spark404 · spark404 · commit dcc288c8bebe · 2013-05-13T16:53:27.000+02:00
Set the daemon version centrally in the pom
diff --git a/agent/pom.xml b/agent/pom.xml
@@ -36,6 +36,11 @@
       <artifactId>cloud-utils</artifactId>
       <version>${project.version}</version>
     </dependency>
+    <dependency>
+        <groupId>commons-daemon</groupId>
+        <artifactId>commons-daemon</artifactId>
+        <version>${cs.daemon.version}</version>
+    </dependency>    
   </dependencies>
   <build>
     <defaultGoal>install</defaultGoal>
diff --git a/agent/src/com/cloud/agent/AgentShell.java b/agent/src/com/cloud/agent/AgentShell.java
@@ -38,6 +38,9 @@
 
 import javax.naming.ConfigurationException;
 
+import org.apache.commons.daemon.Daemon;
+import org.apache.commons.daemon.DaemonContext;
+import org.apache.commons.daemon.DaemonInitException;
 import org.apache.commons.httpclient.HttpClient;
 import org.apache.commons.httpclient.MultiThreadedHttpConnectionManager;
 import org.apache.commons.httpclient.methods.GetMethod;
@@ -58,7 +61,7 @@
 import com.cloud.utils.exception.CloudRuntimeException;
 import com.cloud.utils.script.Script;
 
-public class AgentShell implements IAgentShell {
+public class AgentShell implements IAgentShell, Daemon {
     private static final Logger s_logger = Logger.getLogger(AgentShell.class
             .getName());
     private static final MultiThreadedHttpConnectionManager s_httpClientManager = new MultiThreadedHttpConnectionManager();
@@ -376,7 +379,17 @@ protected boolean parseCommand(final String[] args)
 
         return true;
     }
-
+    
+    @Override
+    public void init(DaemonContext dc) throws DaemonInitException {
+        s_logger.debug("Initializing AgentShell from JSVC");
+        try {
+            init(dc.getArguments());
+        } catch (ConfigurationException ex) {
+            throw new DaemonInitException("Initialization failed", ex);
+        }
+    }
+    
     public void init(String[] args) throws ConfigurationException {
 
     	// PropertiesUtil is used both in management server and agent packages,
@@ -629,11 +642,13 @@ public void destroy() {
 
     public static void main(String[] args) {
         try {
+            s_logger.debug("Initializing AgentShell from main");
             AgentShell shell = new AgentShell();
             shell.init(args);
             shell.start();
         } catch (ConfigurationException e) {
             System.out.println(e.getMessage());
         }
     }
+
 }
diff --git a/pom.xml b/pom.xml
@@ -84,6 +84,7 @@
     <cs.replace.properties>build/replace.properties</cs.replace.properties>
     <cs.libvirt-java.version>0.4.9</cs.libvirt-java.version>
     <cs.target.dir>target</cs.target.dir>
+    <cs.daemon.version>1.0.10</cs.daemon.version>
   </properties>
 
   <distributionManagement>
diff --git a/usage/pom.xml b/usage/pom.xml
@@ -40,7 +40,7 @@
     <dependency>
     	<groupId>commons-daemon</groupId>
     	<artifactId>commons-daemon</artifactId>
-    	<version>1.0.10</version>
+    	<version>${cs.daemon.version}</version>
     </dependency>
   </dependencies>
   <build>
