Fixed relative paths in Linux Application
[nepi.git] / src / neco / resources / linux / application.py
index bdd271b..4f2b989 100644 (file)
@@ -7,7 +7,7 @@ from neco.util import sshfuncs
 import logging
 import os
 
-DELAY ="1s"
+reschedule_delay = "0.5s"
 
 # TODO: Resolve wildcards in commands!! 
 
@@ -62,12 +62,6 @@ class LinuxApplication(ResourceManager):
         stdin = Attribute("stdin", "Standard input", flags = Flags.ExecReadOnly)
         stdout = Attribute("stdout", "Standard output", flags = Flags.ExecReadOnly)
         stderr = Attribute("stderr", "Standard error", flags = Flags.ExecReadOnly)
-        update_home = Attribute("updateHome", "If application hash has changed remove old directory and"
-                "re-upload before starting experiment. If not keep the same directory", 
-                default = True,
-                type = Types.Bool, 
-                flags = Flags.ExecReadOnly)
-
         tear_down = Attribute("tearDown", "Bash script to be executed before "
                 "releasing the resource", 
                 flags = Flags.ReadOnly)
@@ -84,7 +78,6 @@ class LinuxApplication(ResourceManager):
         cls._register_attribute(stdin)
         cls._register_attribute(stdout)
         cls._register_attribute(stderr)
-        cls._register_attribute(update_home)
         cls._register_attribute(tear_down)
 
     @classmethod
@@ -116,16 +109,16 @@ class LinuxApplication(ResourceManager):
         return None
 
     @property
-    def home(self):
-        return os.path.join(self.node.exp_dir, self._home)
+    def app_home(self):
+        return os.path.join(self.node.exp_home, self._home)
 
     @property
     def src_dir(self):
-        return os.path.join(self.home, 'src')
+        return os.path.join(self.app_home, 'src')
 
     @property
     def build_dir(self):
-        return os.path.join(self.home, 'build')
+        return os.path.join(self.app_home, 'build')
 
     @property
     def pid(self):
@@ -136,10 +129,12 @@ class LinuxApplication(ResourceManager):
         return self._ppid
 
     def trace(self, name, attr = TraceAttr.ALL, block = 512, offset = 0):
-        path = os.path.join(self.home, name)
+        self.info("Retrieving '%s' trace %s " % (name, attr))
+
+        path = os.path.join(self.app_home, name)
         
-        cmd = "(test -f %s && echo 'success') || echo 'error'" % path
-        (out, err), proc = self.node.execute(cmd)
+        command = "(test -f %s && echo 'success') || echo 'error'" % path
+        (out, err), proc = self.node.execute(command)
 
         if (err and proc.poll()) or out.find("error") != -1:
             msg = " Couldn't find trace %s " % name
@@ -150,7 +145,7 @@ class LinuxApplication(ResourceManager):
             return path
 
         if attr == TraceAttr.ALL:
-            (out, err), proc = self.node.check_output(self.home, name)
+            (out, err), proc = self.node.check_output(self.app_home, name)
             
             if err and proc.poll():
                 msg = " Couldn't read trace %s " % name
@@ -177,10 +172,8 @@ class LinuxApplication(ResourceManager):
         return out
             
     def provision(self, filters = None):
-        # TODO: verify home hash or clean home
-
         # create home dir for application
-        self.node.mkdir(self.home)
+        self.node.mkdir(self.app_home)
 
         # upload sources
         self.upload_sources()
@@ -197,10 +190,23 @@ class LinuxApplication(ResourceManager):
         # Install
         self.install()
 
+        command = self.get("command")
+        x11 = self.get("forwardX11")
+        if not x11 and command:
+            self.info("Uploading command '%s'" % command)
+        
+            # TODO: missing set PATH and PYTHONPATH!!
+
+            # If the command runs asynchronous, pre upload the command 
+            # to the app.sh file in the remote host
+            dst = os.path.join(self.app_home, "app.sh")
+            command = self.replace_paths(command)
+            self.node.upload(command, dst, text = True)
+
         super(LinuxApplication, self).provision()
 
     def upload_sources(self):
-        # check if sources need to be uploaded and upload them
+        # TODO: check if sources need to be uploaded and upload them
         sources = self.get("sources")
         if sources:
             self.info(" Uploading sources ")
@@ -208,7 +214,7 @@ class LinuxApplication(ResourceManager):
             # create dir for sources
             self.node.mkdir(self.src_dir)
 
-            sources = self.sources.split(' ')
+            sources = sources.split(' ')
 
             http_sources = list()
             for source in list(sources):
@@ -217,12 +223,26 @@ class LinuxApplication(ResourceManager):
                     sources.remove(source)
 
             # Download http sources
-            for source in http_sources:
-                dst = os.path.join(self.src_dir, source.split("/")[-1])
-                command = "wget -o %s %s" % (dst, source)
-                self.node.execute(command)
-
-            self.node.upload(sources, self.src_dir)
+            if http_sources:
+                cmd = " wget -c --directory-prefix=${SOURCES} "
+                verif = ""
+
+                for source in http_sources:
+                    cmd += " %s " % (source)
+                    verif += " ls ${SOURCES}/%s ;" % os.path.basename(source)
+                
+                # Wget output goes to stderr :S
+                cmd += " 2> /dev/null ; "
+
+                # Add verification
+                cmd += " %s " % verif
+
+                # Upload the command to a file, and execute asynchronously
+                self.upload_and_run(cmd, 
+                        "http_sources.sh", "http_sources_pid", 
+                        "http_sources_out", "http_sources_err")
+            if sources:
+                self.node.upload(sources, self.src_dir)
 
     def upload_code(self):
         code = self.get("code")
@@ -239,7 +259,7 @@ class LinuxApplication(ResourceManager):
         depends = self.get("depends")
         if depends:
             self.info(" Installing dependencies %s" % depends)
-            self.node.install_packages(depends, home = self.home)
+            self.node.install_packages(depends, home = self.app_home)
 
     def build(self):
         build = self.get("build")
@@ -249,35 +269,31 @@ class LinuxApplication(ResourceManager):
             # create dir for build
             self.node.mkdir(self.build_dir)
 
-            cmd = self.replace_paths(build)
-
-            (out, err), proc = self.run_and_wait(cmd, self.home,
-                pidfile = "build_pid",
-                stdout = "build_log", 
-                stderr = "build_err", 
-                raise_on_error = True)
+            # Upload the command to a file, and execute asynchronously
+            self.upload_and_run(build, 
+                    "build.sh", "build_pid", 
+                    "build_out", "build_err")
  
     def install(self):
         install = self.get("install")
         if install:
             self.info(" Installing sources ")
 
-            cmd = self.replace_paths(install)
-
-            (out, err), proc = self.run_and_wait(cmd, self.home, 
-                pidfile = "install_pid",
-                stdout = "install_log", 
-                stderr = "install_err", 
-                raise_on_error = True)
+            # Upload the command to a file, and execute asynchronously
+            self.upload_and_run(install, 
+                    "install.sh", "install_pid", 
+                    "install_out", "install_err")
 
     def deploy(self):
         # Wait until node is associated and deployed
         node = self.node
         if not node or node.state < ResourceState.READY:
             self.debug("---- RESCHEDULING DEPLOY ---- node state %s " % self.node.state )
-            self.ec.schedule(DELAY, self.deploy)
+            self.ec.schedule(reschedule_delay, self.deploy)
         else:
             try:
+                command = self.get("command") or ""
+                self.info(" Deploying command '%s' " % command)
                 self.discover()
                 self.provision()
             except:
@@ -287,45 +303,62 @@ class LinuxApplication(ResourceManager):
             super(LinuxApplication, self).deploy()
 
     def start(self):
-        command = self.replace_paths(self.get("command"))
+        command = self.get("command")
         env = self.get("env")
         stdin = 'stdin' if self.get("stdin") else None
+        stdout = 'stdout' if self.get("stdout") else 'stdout'
+        stderr = 'stderr' if self.get("stderr") else 'stderr'
         sudo = self.get('sudo') or False
         x11 = self.get("forwardX11") or False
         failed = False
 
         super(LinuxApplication, self).start()
 
-        self.info("Starting command %s" % command)
+        if not command:
+            self.info("No command to start ")
+            self._state = ResourceState.FINISHED
+            return 
+    
+        self.info("Starting command '%s'" % command)
 
         if x11:
+            # If the command requires X11 forwarding, we
+            # can't run it asynchronously
             (out, err), proc = self.node.execute(command,
                     sudo = sudo,
                     stdin = stdin,
-                    stdout = 'stdout',
-                    stderr = 'stderr',
+                    stdout = stdout,
+                    stderr = stderr,
                     env = env,
                     forward_x11 = x11)
 
             if proc.poll() and err:
                 failed = True
         else:
-            (out, err), proc = self.node.run(command, self.home, 
+            # Command was  previously uploaded, now run the remote
+            # bash file asynchronously
+            if env:
+                env = self.replace_paths(env)
+
+            cmd = "bash ./app.sh"
+            (out, err), proc = self.node.run(cmd, self.app_home, 
                 stdin = stdin, 
+                stdout = stdout,
+                stderr = stderr,
                 sudo = sudo)
 
             if proc.poll() and err:
                 failed = True
         
             if not failed:
-                pid, ppid = self.node.wait_pid(home = self.home)
+                pid, ppid = self.node.wait_pid(home = self.app_home)
                 if pid: self._pid = int(pid)
                 if ppid: self._ppid = int(ppid)
 
             if not self.pid or not self.ppid:
                 failed = True
  
-        (out, chkerr), proc = self.node.check_output(self.home, 'stderr')
+        (out, chkerr), proc = self.node.check_output(self.app_home, 'stderr')
 
         if failed or out or chkerr:
             # check if execution errors occurred
@@ -374,7 +407,7 @@ class LinuxApplication(ResourceManager):
     @property
     def state(self):
         if self._state == ResourceState.STARTED:
-            (out, err), proc = self.node.check_output(self.home, 'stderr')
+            (out, err), proc = self.node.check_output(self.app_home, 'stderr')
 
             if out or err:
                 if err.find("No such file or directory") >= 0 :
@@ -395,6 +428,32 @@ class LinuxApplication(ResourceManager):
 
         return self._state
 
+    def upload_and_run(self, cmd, fname, pidfile, outfile, errfile):
+        dst = os.path.join(self.app_home, fname)
+        cmd = self.replace_paths(cmd)
+        self.node.upload(cmd, dst, text = True)
+
+        cmd = "bash ./%s" % fname
+        (out, err), proc = self.node.run_and_wait(cmd, self.app_home,
+            pidfile = pidfile,
+            stdout = outfile, 
+            stderr = errfile, 
+            raise_on_error = True)
+
+    def replace_paths(self, command):
+        """
+        Replace all special path tags with shell-escaped actual paths.
+        """
+        def absolute_dir(d):
+            return d if d.startswith("/") else os.path.join("${HOME}", d)
+
+        return ( command
+            .replace("${SOURCES}", absolute_dir(self.src_dir))
+            .replace("${BUILD}", absolute_dir(self.build_dir))
+            .replace("${APP_HOME}", absolute_dir(self.app_home))
+            .replace("${NODE_HOME}", absolute_dir(self.node.node_home))
+            .replace("${EXP_HOME}", self.node.exp_home) )
+        
     def valid_connection(self, guid):
         # TODO: Validate!
         return True
@@ -425,14 +484,3 @@ class LinuxApplication(ResourceManager):
         skey = "".join(map(str, args))
         return hashlib.md5(skey).hexdigest()
 
-    def replace_paths(self, command):
-        """
-        Replace all special path tags with shell-escaped actual paths.
-        """
-        return ( command
-            .replace("${SOURCES}", self.src_dir)
-            .replace("${BUILD}", self.build_dir) 
-            .replace("${APPHOME}", self.home) 
-            .replace("${NODEHOME}", self.node.home) )
-
-