bring over GetBootMedium
[plcapi.git] / PLC / Methods / GetBootMedium.py
index 00d6f2d..c2e60a5 100644 (file)
@@ -1,5 +1,3 @@
-# $Id$
-# $URL$
 import random
 import base64
 import os
@@ -11,24 +9,33 @@ from PLC.Method import Method
 from PLC.Parameter import Parameter, Mixed
 from PLC.Auth import Auth
 
+from PLC.IpAddresses import IpAddress, IpAddresses
+from PLC.Routes import Route, Routes
 from PLC.Nodes import Node, Nodes
 from PLC.Interfaces import Interface, Interfaces
 from PLC.InterfaceTags import InterfaceTag, InterfaceTags
 from PLC.NodeTags import NodeTag, NodeTags
 
-from PLC.Accessors.Accessors_standard import *                 # import node accessors
+from PLC.Accessors.Accessors_standard import *                  # import node accessors
 
 # could not define this in the class..
 # create a dict with the allowed actions for each type of node
+# reservable nodes being more recent, we do not support the floppy stuff anymore
 allowed_actions = {
-                'regular' : [ 'node-preview',
-                              'node-floppy',
-                              'node-iso',
-                              'node-usb',
-                              'generic-iso',
-                              'generic-usb',
-                               ],
-                }
+    'regular' :
+    [ 'node-preview',
+      'node-floppy',
+      'node-iso',
+      'node-usb',
+      'generic-iso',
+      'generic-usb',
+      ],
+    'reservable':
+    [ 'node-preview',
+      'node-iso',
+      'node-usb',
+      ],
+    }
 
 # compute a new key
 def compute_key():
@@ -43,18 +50,18 @@ def compute_key():
 
 class GetBootMedium(Method):
     """
-    This method is a redesign based on former, supposedly dedicated, 
+    This method is a redesign based on former, supposedly dedicated,
     AdmGenerateNodeConfFile
 
-    As compared with its ancestor, this method provides a much more 
+    As compared with its ancestor, this method provides a much more
     detailed interface, that allows to
-    (*) either just preview the node config file -- in which case 
+    (*) either just preview the node config file -- in which case
         the node key is NOT recomputed, and NOT provided in the output
-    (*) or regenerate the node config file for storage on a floppy 
-        that is, exactly what the ancestor method used todo, 
+    (*) or regenerate the node config file for storage on a floppy
+        that is, exactly what the ancestor method used todo,
         including renewing the node's key
     (*) or regenerate the config file and bundle it inside an ISO or USB image
-    (*) or just provide the generic ISO or USB boot images 
+    (*) or just provide the generic ISO or USB boot images
         in which case of course the node_id_or_hostname parameter is not used
 
     action is expected among the following string constants according the
@@ -70,32 +77,34 @@ class GetBootMedium(Method):
 
     Apart for the preview mode, this method generates a new node key for the
     specified node, effectively invalidating any old boot medium.
+    Note that 'reservable' nodes do not support 'node-floppy',
+    'generic-iso' nor 'generic-usb'.
 
     In addition, two return mechanisms are supported.
-    (*) The default behaviour is that the file's content is returned as a 
+    (*) The default behaviour is that the file's content is returned as a
         base64-encoded string. This is how the ancestor method used to work.
         To use this method, pass an empty string as the file parameter.
 
-    (*) Or, for efficiency -- this makes sense only when the API is used 
-        by the web pages that run on the same host -- the caller may provide 
-        a filename, in which case the resulting file is stored in that location instead. 
-        The filename argument can use the following markers, that are expanded 
+    (*) Or, for efficiency -- this makes sense only when the API is used
+        by the web pages that run on the same host -- the caller may provide
+        a filename, in which case the resulting file is stored in that location instead.
+        The filename argument can use the following markers, that are expanded
         within the method
         - %d : default root dir (some builtin dedicated area under /var/tmp/)
                Using this is recommended, and enforced for non-admin users
-        - %n : the node's name when this makes sense, or a mktemp-like name when 
+        - %n : the node's name when this makes sense, or a mktemp-like name when
                generic media is requested
         - %s : a file suffix appropriate in the context (.txt, .iso or the like)
         - %v : the bootcd version string (e.g. 4.0)
         - %p : the PLC name
         - %f : the nodefamily
         - %a : arch
-        With the file-based return mechanism, the method returns the full pathname 
-        of the result file; 
+        With the file-based return mechanism, the method returns the full pathname
+        of the result file;
         ** WARNING **
         It is the caller's responsability to remove this file after use.
 
-    Options: an optional array of keywords. 
+    Options: an optional array of keywords.
         options are not supported for generic images
       Currently supported are
         - 'partition' - for USB actions only
@@ -106,19 +115,19 @@ class GetBootMedium(Method):
         tty - baudrate - parity - bits
         e.g. ttyS0:115200:n:8
         - 'variant:<variantname>'
-        passed to build.sh as -V <variant> 
+        passed to build.sh as -V <variant>
         variants are used to run a different kernel on the bootCD
         see kvariant.sh for how to create a variant
         - 'no-hangcheck' - disable hangcheck
 
     Tags: the following tags are taken into account when attached to the node:
         'serial', 'cramfs', 'kvariant', 'kargs', 'no-hangcheck'
-        
+
     Security:
         - Non-admins can only generate files for nodes at their sites.
         - Non-admins, when they provide a filename, *must* specify it in the %d area
 
-   Housekeeping: 
+   Housekeeping:
         Whenever needed, the method stores intermediate files in a
         private area, typically not located under the web server's
         accessible area, and are cleaned up by the method.
@@ -156,7 +165,7 @@ class GetBootMedium(Method):
         if len(parts) < 2:
             raise PLCInvalidArgument, "Node hostname %s is invalid"%node['hostname']
         return parts
-        
+
     # Generate the node (plnode.txt) configuration content.
     #
     # This function will create the configuration file a node
@@ -190,6 +199,7 @@ class GetBootMedium(Method):
         # renew the key and save it on the database
         if renew_key:
             node['key'] = compute_key()
+            node.update_last_download(commit=False)
             node.sync()
 
         # Generate node configuration file suitable for BootCD
@@ -207,13 +217,36 @@ class GetBootMedium(Method):
         file += 'IP_METHOD="%s"\n' % primary['method']
 
         if primary['method'] == 'static':
-            file += 'IP_ADDRESS="%s"\n' % primary['ip']
-            file += 'IP_GATEWAY="%s"\n' % primary['gateway']
-            file += 'IP_NETMASK="%s"\n' % primary['netmask']
-            file += 'IP_NETADDR="%s"\n' % primary['network']
-            file += 'IP_BROADCASTADDR="%s"\n' % primary['broadcast']
-            file += 'IP_DNS1="%s"\n' % primary['dns1']
-            file += 'IP_DNS2="%s"\n' % (primary['dns2'] or "")
+            # FIXME: We currently get the first ip address
+            # only. plnode.txt depends on interface having a single ip
+            # address. We assume that the first ip address in the
+            # primary interface will be the primary ip address. This
+            # assumumption is probably not the right way to go... - baris
+            routes = Routes(self.api, {'node_id': primary['node_id']})
+            default_route = [r for r in routes if r['subnet'] == u'0.0.0.0/0']
+            gateway = ""
+            if default_route:
+                gateway = default_route[0]['next_hop']
+            
+            node = Nodes(self.api, primary['node_id'])[0]
+            dns = node['dns'].split(',')
+            dns1 = None
+            dns2 = None
+            if dns:
+                dns1 = dns[0]
+            if len(dns) > 1:
+                dns2 = dns[1]
+
+            ip_addresses = IpAddresses(self.api, primary['ip_address_ids'])
+            if ip_addresses:
+                primary_ip_address = ip_addresses[0]
+                file += 'IP_ADDRESS="%s"\n' % primary_ip_address['ip_addr']
+                file += 'IP_GATEWAY="%s"\n' % gateway
+                file += 'IP_NETMASK="%s"\n' % primary_ip_address['netmask']
+                file += 'IP_NETADDR=""\n'
+                file += 'IP_BROADCASTADDR=""\n'
+                file += 'IP_DNS1="%s"\n' % (dns1 or "")
+                file += 'IP_DNS2="%s"\n' % (dns2 or "")
 
         file += 'HOST_NAME="%s"\n' % host
         file += 'DOMAIN_NAME="%s"\n' % domain
@@ -225,7 +258,7 @@ class GetBootMedium(Method):
         for setting in settings:
             if setting['category'] is not None:
                 categories.add(setting['category'])
-        
+
         for category in categories:
             category_settings = InterfaceTags(self.api,{'interface_id':interface['interface_id'],
                                                               'category':category})
@@ -250,19 +283,19 @@ class GetBootMedium(Method):
         arch = self.api.config.PLC_FLAVOUR_NODE_ARCH
         if not node:
             return (pldistro,fcdistro,arch)
-        
+
         node_id=node['node_id']
-        
+
         # no support for deployment-based BootCD's, use kvariants instead
-        node_pldistro = GetNodePldistro (self.api).call(auth, node_id)
+        node_pldistro = GetNodePldistro (self.api,self.caller).call(auth, node_id)
         if node_pldistro: pldistro = node_pldistro
 
-        node_fcdistro = GetNodeFcdistro (self.api).call(auth, node_id)
+        node_fcdistro = GetNodeFcdistro (self.api,self.caller).call(auth, node_id)
         if node_fcdistro: fcdistro = node_fcdistro
 
-        node_arch = GetNodeArch (self.api).call(auth,node_id)
+        node_arch = GetNodeArch (self.api,self.caller).call(auth,node_id)
         if node_arch: arch = node_arch
-        
+
         return (pldistro,fcdistro,arch)
 
     def bootcd_version (self):
@@ -270,7 +303,7 @@ class GetBootMedium(Method):
             return file(self.BOOTCDDIR + "/build/version.txt").readline().strip()
         except:
             raise Exception,"Unknown boot cd version - probably wrong bootcd dir : %s"%self.BOOTCDDIR
-    
+
     def cleantrash (self):
         for file in self.trash:
             if self.DEBUG:
@@ -279,7 +312,7 @@ class GetBootMedium(Method):
                 os.unlink(file)
 
     ### handle filename
-    # build the filename string 
+    # build the filename string
     # check for permissions and concurrency
     # returns the filename
     def handle_filename (self, filename, nodename, suffix, arch):
@@ -304,10 +337,18 @@ class GetBootMedium(Method):
                     raise PLCInvalidArgument, "File %s not under %s"%(filename,self.WORKDIR)
 
             ### output should not exist (concurrent runs ..)
+            # numerous reports of issues with this policy
+            # looks like people sometime suspend/cancel their download
+            # and this leads to the old file sitting in there forever
+            # so, if the file is older than 5 minutes, we just trash
+            grace=5
+            if os.path.exists(filename) and (time.time()-os.path.getmtime(filename)) >= (grace*60):
+                os.unlink(filename)
             if os.path.exists(filename):
-                raise PLCInvalidArgument, "Resulting file %s already exists"%filename
+                raise PLCInvalidArgument, "Resulting file %s already exists - please try again in %d minutes"%\
+                    (filename,grace)
 
-            ### we can now safely create the file, 
+            ### we can now safely create the file,
             ### either we are admin or under a controlled location
             filedir=os.path.dirname(filename)
             # dirname does not return "." for a local filename like its shell counterpart
@@ -328,17 +369,17 @@ class GetBootMedium(Method):
 
         # regular node, make build's arguments
         # and build the full command line to be called
-        if node_type == 'regular':
+        if node_type in [ 'regular', 'reservable' ]:
 
             build_sh_options=""
-            if "cramfs" in build_sh_spec: 
+            if "cramfs" in build_sh_spec:
                 type += "_cramfs"
-            if "serial" in build_sh_spec: 
+            if "serial" in build_sh_spec:
                 build_sh_options += " -s %s"%build_sh_spec['serial']
             if "variant" in build_sh_spec:
                 build_sh_options += " -V %s"%build_sh_spec['variant']
-            
-            for karg in build_sh_spec['kargs'][0]:
+
+            for karg in build_sh_spec['kargs']:
                 build_sh_options += ' -k "%s"'%karg
 
             log_file="%s.log"%node_image
@@ -353,7 +394,7 @@ class GetBootMedium(Method):
         if self.DEBUG:
             print "The build command line is %s" % command
 
-        return command 
+        return command
 
     def call(self, auth, node_id_or_hostname, action, filename, options = []):
 
@@ -390,12 +431,12 @@ class GetBootMedium(Method):
             if options:
                 raise PLCInvalidArgument, "Options are not supported for node configs"
         else:
-            # create a dict for build.sh 
+            # create a dict for build.sh
             build_sh_spec={'kargs':[]}
             # use node tags as defaults
             # check for node tag equivalents
-            tags = NodeTags(self.api, 
-                            {'node_id': node['node_id'], 
+            tags = NodeTags(self.api,
+                            {'node_id': node['node_id'],
                              'tagname': ['serial', 'cramfs', 'kvariant', 'kargs', 'no-hangcheck']},
                             ['tagname', 'value'])
             if tags:
@@ -407,7 +448,7 @@ class GetBootMedium(Method):
                     if tag['tagname'] == 'kvariant':
                         build_sh_spec['variant'] = tag['value']
                     if tag['tagname'] == 'kargs':
-                        build_sh_spec['kargs'].append(tag['value'].split())
+                        build_sh_spec['kargs'] += tag['value'].split()
                     if tag['tagname'] == 'no-hangcheck':
                         build_sh_spec['kargs'].append('hcheck_reboot0')
             # then options can override tags
@@ -447,9 +488,9 @@ class GetBootMedium(Method):
         # apply on globals
         for attr in [ "BOOTCDDIR", "BOOTCDBUILD", "GENERICDIR" ]:
             setattr(self,attr,getattr(self,attr).replace("@NODEFAMILY@",self.nodefamily))
-            
+
         filename = self.handle_filename(filename, nodename, suffix, arch)
-        
+
         # log call
         if node:
             self.message='GetBootMedium on node %s - action=%s'%(nodename,action)
@@ -502,7 +543,7 @@ class GetBootMedium(Method):
 
             ### check we've got required material
             version = self.bootcd_version()
-            
+
             if not os.path.isfile(self.BOOTCDBUILD):
                 raise PLCAPIError, "Cannot locate bootcd/build.sh script %s"%self.BOOTCDBUILD
 
@@ -513,7 +554,7 @@ class GetBootMedium(Method):
                     os.chmod(self.WORKDIR,0777)
                 except:
                     raise PLCPermissionDenied, "Could not create dir %s"%self.WORKDIR
-            
+
             try:
                 # generate floppy config
                 floppy_text = self.floppy_contents(node,True)
@@ -543,7 +584,7 @@ class GetBootMedium(Method):
 
                 if not os.path.isfile (node_image):
                     raise PLCAPIError,"Unexpected location of build.sh output - %s"%node_image
-            
+
                 # handle result
                 if filename:
                     ret=os.system('mv "%s" "%s"'%(node_image,filename))
@@ -561,7 +602,6 @@ class GetBootMedium(Method):
             except:
                 self.cleantrash()
                 raise
-                
+
         # we're done here, or we missed something
         raise PLCAPIError,'Unhandled action %s'%action
-