Se limpia el intento luego de las pruebas, se baja la probabilidad de RC entre que...

diff --git a/src/sc_test b/src/sc_test
index e532a7fb4f8ac12b89352915cfe51116c54345e0..d30a0c07be3698d987c11c7522fecef3d1a67c81 100755 (executable)
--- a/src/sc_test
+++ b/src/sc_test
@@ -7,6 +7,7 @@ import os
 import pwd
 import time
 import signal
 import pwd
 import time
 import signal

+import glob

 import shutil
 import datetime
 import subprocess
 import shutil
 import datetime
 import subprocess


@@ -36,11 +37,14 @@ class secure_process:
         resource.setrlimit(resource.RLIMIT_MEMLOCK, x2(100))
         resource.setrlimit(resource.RLIMIT_NOFILE, x2(100))
         resource.setrlimit(resource.RLIMIT_NPROC, x2(0))
         resource.setrlimit(resource.RLIMIT_MEMLOCK, x2(100))
         resource.setrlimit(resource.RLIMIT_NOFILE, x2(100))
         resource.setrlimit(resource.RLIMIT_NPROC, x2(0))

-        sleep(0) # para tratar de forzar un sync para que entre al sleep del padre
+        # 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
 
 def sigchld(signum, frame):
     """Signal handler para SIGCHILD."""
     global hijo_muerto

+    global log
+    log.debug('Murió el hijo')

     hijo_muerto = True
 
 def sigterm(signum, frame):
     hijo_muerto = True
 
 def sigterm(signum, frame):


@@ -49,9 +53,9 @@ def sigterm(signum, frame):
     continuar = False
     log.debug('Señal %d recibida', signum)
 
     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
     # 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')
     # Compilo
     if not os.path.exists(makefile):
         makefile = os.path.join(data_dir, 'Makefile')
     # Compilo


@@ -81,8 +85,9 @@ def preparar(intento_dir, chroot_dir):
     os.mkdir(chroot_dir)
     shutil.move(os.path.join(intento_dir, 'tp'), ejecutable)
 
     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
     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:
     # Obtengo datos útiles del caso de prueba
     tiempo_cpu = caso_de_prueba.tiempoCpu
     if tiempo_cpu is None:


@@ -94,7 +99,7 @@ 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????
     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
     stdout = file(os.path.join(intento_dir, 'stdout'), 'w')
     stderr = file(os.path.join(intento_dir, 'stderr'), 'w')
     # Ejecuto programa


@@ -104,8 +109,8 @@ def probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log):
     log.debug('Ejecutando %s', ' '.join(params))
     os.seteuid(0) # Dios! (para chroot)
     try:
     log.debug('Ejecutando %s', ' '.join(params))
     os.seteuid(0) # Dios! (para chroot)
     try:

-        proc = subprocess.Popen(params, 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
     except Exception, e: # FIXME poner en el manejo de exceptiones estandar
         try:
             print e.child_traceback


@@ -116,7 +121,7 @@ def probar(intento, caso_de_prueba, uid, gid, chroot_dir, conf, conn, log):
     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 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)
         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)


@@ -261,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'))
     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
     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)
         continue
     # Prepara archivos
     preparar(intento_dir, chroot_dir)


@@ -272,9 +278,13 @@ while continuar:
     intento.inicioPruebas = datetime.datetime.now()
     pruebas = []
     for caso_de_prueba in intento.entrega.ejercicio.casosDePrueba:
     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()
     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
     #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