]> git.llucax.com Git - software/sercom-old.git/blobdiff - src/sc_test
Se limpia el intento luego de las pruebas, se baja la probabilidad de RC entre que...
[software/sercom-old.git] / src / sc_test
index 34402ff441c583f618a24a4c73ecb78317a2ff93..d30a0c07be3698d987c11c7522fecef3d1a67c81 100755 (executable)
@@ -7,6 +7,7 @@ import os
 import pwd
 import time
 import signal
+import glob
 import shutil
 import datetime
 import subprocess
@@ -36,12 +37,15 @@ class secure_process:
         resource.setrlimit(resource.RLIMIT_MEMLOCK, x2(100))
         resource.setrlimit(resource.RLIMIT_NOFILE, x2(100))
         resource.setrlimit(resource.RLIMIT_NPROC, x2(0))
+        # Tratamos de forzar un sync para que entre al sleep del padre
+        time.sleep(0)
 
 def sigchld(signum, frame):
     """Signal handler para SIGCHILD."""
-    #global hijo_muerto
-    #hijo_muerto = True
-    pass
+    global hijo_muerto
+    global log
+    log.debug('Murió el hijo')
+    hijo_muerto = True
 
 def sigterm(signum, frame):
     """Signal handler para SIGTERM y SIGINT."""
@@ -49,12 +53,11 @@ def sigterm(signum, frame):
     continuar = False
     log.debug('Señal %d recibida', signum)
 
-def compilar(intento, intento_dir, entrega_dir, log):
+def compilar(intento, data_dir, ejercicio_dir, intento_dir, log):
     # Busco makefile
-    makefile = os.path.join(entrega_dir, 'Makefile')
+    makefile = os.path.join(ejercicio_dir, 'Makefile')
     if not os.path.exists(makefile):
         makefile = os.path.join(data_dir, 'Makefile')
-    shutil.copy(makefile, intento_dir)
     # Compilo
     log.debug('Ejecutando: make -f %s', makefile)
     intento.inicioCompila = datetime.datetime.now()
@@ -82,7 +85,9 @@ def preparar(intento_dir, chroot_dir):
     os.mkdir(chroot_dir)
     shutil.move(os.path.join(intento_dir, 'tp'), ejecutable)
 
-def probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log):
+def probar(intento, caso_de_prueba, uid, gid, intento_dir, ejercicio_dir, chroot_dir, conf, conn, log):
+    global hijo_muerto # Viene del SIGCHLD
+    hijo_muerto = False # Reseteo variable de SIGCHLD
     # Obtengo datos útiles del caso de prueba
     tiempo_cpu = caso_de_prueba.tiempoCpu
     if tiempo_cpu is None:
@@ -94,16 +99,18 @@ def probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log):
     log.debug('Prueba: %s', prueba)
     # Abro archivos para fds básicos
     #XXX sacar nombres de archivos de prueba????
-    #stdin = file(os.path.join(intento_dir, 'stdin'), 'r')
+    stdin = file(os.path.join(ejercicio_dir, 'casos_de_prueba', caso_de_prueba.nombre, 'stdin'), 'r')
     stdout = file(os.path.join(intento_dir, 'stdout'), 'w')
     stderr = file(os.path.join(intento_dir, 'stderr'), 'w')
     # Ejecuto programa
-    log.debug('Ejecutando /tp %s', caso_de_prueba.parametros)
+    params = ['/tp']
+    if caso_de_prueba.parametros:
+        params += params2seq(caso_de_prueba.parametros)
+    log.debug('Ejecutando %s', ' '.join(params))
     os.seteuid(0) # Dios! (para chroot)
-    # FIXME caso_de_prueba.parametros
     try:
-        proc = subprocess.Popen('/tp', stdout=stdout, stderr=stderr, #stdin=stdin,
-            preexec_fn=secure_process(chroot_dir, uid, gid, tiempo_cpu))
+        proc = subprocess.Popen(params, stdout=stdout, stderr=stderr, stdin=stdin,
+            cwd=chroot_dir, close_fds=True, preexec_fn=secure_process(chroot_dir, uid, gid, tiempo_cpu))
     except Exception, e: # FIXME poner en el manejo de exceptiones estandar
         try:
             print e.child_traceback
@@ -111,9 +118,10 @@ def probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log):
         except:
             raise
     os.seteuid(uid) # Mortal de nuevo
-    time.sleep(tiempo_cpu)
+    if not hijo_muerto: # Recibido por el sigchld, para saber si murió
+        time.sleep(tiempo_cpu)
     # Si el proceso sigue andando lo tenemos que matar
-    if proc.poll() is None:
+    if not hijo_muerto:
         log.debug('La prueba tardó más del tiempo permitido (%d segundos)',
             tiempo_cpu)
         os.seteuid(0) # Dios! (corre como Dios, hay que matarlo como Dios)
@@ -161,17 +169,83 @@ def probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log):
     log.debug('Prueba OK: %s', prueba)
     return prueba
 
+def params2seq(params):
+    r"""Parsea un string de forma similar al bash, separando por espacios y
+    teniendo en cuenta comillas simples y dobles para agrupar. Para poner
+    comillas se puede usar el \ como caracter de escape (\' y \") y también
+    interpreta \n y \t. Devuelve una lista con los parámetros encontrados."""
+    # Constantes
+    SEP, TOKEN, DQUOTE, SQUOTE = ' ', None, '"', "'"
+    seq = []
+    buff = ''
+    escape = False
+    state = SEP
+    for c in params:
+        # Es un caracter escapado
+        if escape:
+            if c == 'n':
+                buff += '\n'
+            elif c == 't':
+                buff += '\t'
+            else:
+                buff += c
+            escape = False
+            continue
+        # Es una secuencia de escape
+        if c == '\\':
+            escape = True
+            continue
+        # Si está buscando espacios
+        if state == SEP:
+            if c == SEP:
+                continue
+            else:
+                state = TOKEN # Encontró
+        if state == TOKEN:
+            if c == DQUOTE:
+                state = DQUOTE
+                continue
+            if c == SQUOTE:
+                state = SQUOTE
+                continue
+            if c == SEP:
+                state = SEP
+                seq.append(buff)
+                buff = ''
+                continue
+            buff += c
+            continue
+        if state == DQUOTE:
+            if c == DQUOTE:
+                state = TOKEN
+                continue
+            buff += c
+            continue
+        if state == SQUOTE:
+            if c == SQUOTE:
+                state = TOKEN
+                continue
+            buff += c
+            continue
+        raise Exception, 'No tiene sentido'
+    if state == DQUOTE or state == SQUOTE:
+        raise Exception, 'Parse error, falta cerrar comilla (%s)' % state
+    if buff:
+        seq.append(buff)
+    return seq
+
 # Conecto señales
-signal.signal(signal.SIGCHLD, sigchld)
 signal.signal(signal.SIGTERM, sigterm)
 signal.signal(signal.SIGINT, sigterm)
+signal.signal(signal.SIGCHLD, sigchld)
+hijo_muerto = False
 
 # Inicializo
 conf, conn, log =  sercom.init('test')
 log.info('Iniciado')
 
 # Obtengo id de usuario con el cual ejecutar las pruebas
-(uid, gid) = pwd.getpwnam(conf.get('general', 'user'))[2:4]
+uid, gid = pwd.getpwnam(conf.get('general', 'user'))[2:4]
 
 # Cambio UID efectivo
 os.seteuid(uid)
@@ -192,10 +266,11 @@ while continuar:
     log.info('Nuevo intento a probar (%s)', intento)
     # Obtengo paths
     intento_dir = os.path.join(data_dir, intento.path('intentos'))
-    entrega_dir = os.path.join(data_dir, 'ejercicios', str(intento.entrega.ejercicioID))
+    ejercicio_dir = os.path.join(data_dir, 'ejercicios', str(intento.entrega.ejercicioID))
     chroot_dir = os.path.join(intento_dir, 'chroot')
     # Compila
-    if not compilar(intento, intento_dir, entrega_dir, log):
+    if not compilar(intento, data_dir, ejercicio_dir, intento_dir, log):
+        #TODO mandar mail
         continue
     # Prepara archivos
     preparar(intento_dir, chroot_dir)
@@ -203,9 +278,13 @@ while continuar:
     intento.inicioPruebas = datetime.datetime.now()
     pruebas = []
     for caso_de_prueba in intento.entrega.ejercicio.casosDePrueba:
-        pruebas.append(probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log))
+        pruebas.append(probar(intento, caso_de_prueba, uid, gid, intento_dir, ejercicio_dir, chroot_dir, conf, conn, log))
     intento.finPruebas = datetime.datetime.now()
-    #TODO make clean
+    # Limpio directorio
+    log.debug('Borrando chroot')
+    shutil.rmtree(chroot_dir)
+    log.debug('Borrando código objeto (*.o)')
+    [os.remove(obj) for obj in glob.glob(os.path.join(intento_dir, '*.o'))]
     #TODO Armar mail de respuesta al alumno
     for prueba in pruebas:
         #TODO Si es publica, veo si se hizo ok o no y voy creando mail