From 644d17cbaa34469afab6411435208d8c5e56dc2b Mon Sep 17 00:00:00 2001 From: Leandro Lucarella Date: Wed, 23 Feb 2005 21:44:07 +0000 Subject: [PATCH 1/1] Se limpia el intento luego de las pruebas, se baja la probabilidad de RC entre que muere el hijo de la prueba y entra al sleep el padre y bugfixes. --- src/sc_test | 34 ++++++++++++++++++++++------------ 1 file changed, 22 insertions(+), 12 deletions(-) diff --git a/src/sc_test b/src/sc_test index e532a7f..d30a0c0 100755 --- a/src/sc_test +++ b/src/sc_test @@ -7,6 +7,7 @@ import os import pwd import time import signal +import glob 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)) - 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 + global log + log.debug('Murió el hijo') hijo_muerto = True def sigterm(signum, frame): @@ -49,9 +53,9 @@ 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') # Compilo @@ -81,8 +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,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???? - #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 @@ -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: - 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 @@ -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 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) @@ -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')) - 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) @@ -272,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 -- 2.43.0