import shutil
import datetime
import subprocess
+import email.MIMEMultipart
# Módulos locales
import sercom
-import sercom.sqlobject
-from sercom.sqlobject import *
+import sercom.sqlo
+from sercom.sqlo import *
class secure_process:
def __init__(self, chroot, uid, gid, cpu):
continuar = False
log.debug('Señal %d recibida', signum)
-def chroot_dir(intento):
- return os.path.join(intento.path, 'chroot')
-
-def compilar(intento):
+def compilar(intento, mail):
global log
# Busco makefile
makefile = os.path.join(intento.entrega.ejercicio.path, 'Makefile')
# Compilo
log.debug('Ejecutando: make -f %s', makefile)
intento.inicioCompila = datetime.datetime.now()
- make = subprocess.Popen(('make', '-f', makefile), stdout=subprocess.PIPE,
- stderr=subprocess.PIPE, cwd=intento.path)
+ make = subprocess.Popen(('make', '-f', makefile), stderr=subprocess.PIPE,
+ cwd=intento.path)
make.wait()
intento.finCompila = datetime.datetime.now()
log.debug('Fin del comando: make -f %s', makefile)
# Verifico compilación
- if make.returncode:
- log.debug('Error al compilar, código de retorno: %d, salida estándar: '
- '%s, salida de error: %s)', make.returncode, make.stdout.read(),
- make.stderr.read())
- intento.compila = False
- #TODO enviar_respuesta(R_ERR, $mail, "ERROR AL COMPILAR!\n\n$err\n\nCódigo de retorno: $ret\n", $intento);
- return False
- log.debug('Compilado OK')
- intento.compila = True
- #TODO mail acumulativo
- return True
+ stderr = make.stderr.read()
+ intento.compila = not make.returncode
+ msg = 'Compilación: '
+ if intento.compila and not stderr:
+ msg += 'BIEN'
+ elif intento.compila:
+ msg += 'CON ADVERTENCIAS'
+ else:
+ msg += 'ERROR (código de retorno: %d)' % make.returncode
+ mail.body += msg
+ if stderr:
+ msg += '''
+Salida:
+------------------------------------------------------------------------
+%s
+------------------------------------------------------------------------
+
+''' % stderr
+ mail.body += "\n(se adjunta salida, archivo 'make.stderr')\n"
+ mail.attachText(stderr, 'make.stderr')
+ mail.body += '\n\n'
+ intento.observaciones = msg + '\n\n'
+ log.debug(msg)
def preparar(intento):
# Creo chroot - TODO copiarlo de algún lado donde ande el valgrind?
- os.mkdir(chroot_dir(intento))
+ os.mkdir(intento.chrootPath)
shutil.copy(os.path.join(intento.path, 'tp'),
- os.path.join(chroot_dir(intento), 'tp'))
+ os.path.join(intento.chrootPath, 'tp'))
def limpiar(intento):
# Borro chroot entero
- shutil.rmtree(chroot_dir(intento))
+ shutil.rmtree(intento.chrootPath)
-def probar(intento, caso_de_prueba):
+def probar(intento, caso_de_prueba, mail):
+ def diff(prueba, mail, orig, new, name, origname='alumno', newname='catedra', longname=None):
+ from difflib import unified_diff, HtmlDiff
+ #TODO diferenciar pruebas públicas de privadas
+ if longname is None: longname = name
+ diff = ''.join(list(unified_diff(orig, new, fromfile=name+'.'+origname, tofile=name+'.'+newname)))
+ nota = ''
+ if diff:
+ prueba.pasada = False
+ nota = "- %s no coincide con lo esperado (archivo '%s.diff').\n" % (longname, name)
+ if not prueba.casoDePrueba.privado:
+ mail.attachText(diff, name + '.diff')
+ diff = HtmlDiff().make_file(orig, new, fromdesc=name+'.'+origname, todesc=name+'.'+newname, context=True, numlines=3)
+ mail.attachText(diff, name + '.diff.html', 'html')
+ return nota
# Cosas útiles
global log, conn, conf, uid, gid
# Para manejo de SIGCHLD
prueba = Prueba(intento=intento, casoDePrueba=caso_de_prueba,
inicio=datetime.datetime.now(), connection=conn)
log.debug('Iniciando prueba: %s', prueba)
+ log.debug('Caso de prueba: %s', prueba.casoDePrueba)
# Abro archivos para fds básicos
- #XXX sacar nombres de archivos de prueba????
options = {
- 'cwd': chroot_dir(intento),
+ 'cwd': intento.chrootPath,
'close_fds': True,
- 'preexec_fn': secure_process(chroot_dir(intento), uid, gid, tiempo_cpu),
+ 'preexec_fn': secure_process(intento.chrootPath, uid, gid, tiempo_cpu),
}
if os.path.exists(os.path.join(caso_de_prueba.path, 'stdin')):
options['stdin'] = file(os.path.join(caso_de_prueba.path, 'stdin'), 'r')
raise
os.seteuid(uid) # Mortal de nuevo
if not hijo_muerto: # Recibido por el sigchld, para saber si murió
- time.sleep(tiempo_cpu)
+ time.sleep(tiempo_cpu) # Controlo que no tarde mucho
# Si el proceso sigue andando lo tenemos que matar
if not hijo_muerto:
log.debug('La prueba tardó más del tiempo permitido (%d segundos)',
'(%d seg)' % tiempo_cpu
prueba.fin = datetime.datetime.now()
prueba.pasada = False
+ mail.agregarResultado(prueba)
return prueba
proc.wait() # Para que no queden zombies
- prueba.fin = str(datetime.datetime.now())
+ prueba.fin = datetime.datetime.now()
# Salió con una señal?
if proc.returncode < 0:
sig = -proc.returncode
'(%d seg)' % tiempo_cpu
else:
prueba.observaciones = 'Salió con la señal %d' % sig
+ mail.agregarResultado(prueba)
return prueba
# Si tenemos que verificar el código de retorno
if caso_de_prueba.codigoRetorno is not None:
prueba.observaciones = 'Código de retorno incorrecto (debía ' \
'ser %d y se obtuvo %d)' % (caso_de_prueba.codigoRetorno,
proc.returncode)
- #TODO verificar salidas, hacer diff
+ mail.agregarResultado(prueba)
+ return prueba
+ # Verifico salidas estándar/de error
+ prueba.pasada = True # Asumo que está bien, ya habrá tiempo para cambiarlo
+ obs = diff(prueba, mail, proc.stdout.readlines(),
+ file(os.path.join(caso_de_prueba.path, 'stdout')).readlines(),
+ caso_de_prueba.nombre + '.stdout', longname='La salida estándar')
+ obs += diff(prueba, mail, proc.stderr.readlines(),
+ file(os.path.join(caso_de_prueba.path, 'stderr')).readlines(),
+ caso_de_prueba.nombre + '.stderr', longname='La salida de error')
+ for f in caso_de_prueba.archivosSalida:
+ if f not in prueba.archivosSalida:
+ #TODO agregar error FALTA ARCHIVO
+ log.debug('Falta el archivo %s y no lo hizo.', f)
+ prueba.pasada = False
+ obs += "- Falta el archivo de salida '%s'.\n" % f
+ continue
+ obs += diff(prueba, mail,
+ file(os.path.join(intento.chrootPath, f)).readlines(),
+ file(os.path.join(caso_de_prueba.pathSalidas, f)).readlines(),
+ caso_de_prueba.nombre + '.' + f)
+ for f in prueba.archivosSalida - (caso_de_prueba.archivosSalida | caso_de_prueba.archivosEntrada):
+ #TODO agregar error SOBRA ARCHIVO
+ log.debug('El programa debía generar el archivo %s y no lo hizo.', f)
+ prueba.pasada = False
+ obs += "- Se generó un archivo ('%s') que no se esperaba.\n" % f
log.debug('Fin de ejecución de caso de prueba (hijo: %d, ret: %d)',
proc.pid, proc.returncode)
- prueba.pasada = True
- log.debug('Prueba OK: %s', prueba)
+ if obs:
+ prueba.observaciones = obs
+ mail.agregarResultado(prueba)
+ log.debug('Resultado de la prueba: %s', prueba)
return prueba
def params2seq(params):
seq.append(buff)
return seq
+class MailIntento(email.MIMEMultipart.MIMEMultipart, object):
+ def __init__(self, intento):
+ global conf
+ from email.MIMEMultipart import MIMEMultipart
+ from email.MIMEMessage import MIMEMessage
+ from email.MIMEText import MIMEText
+ MIMEMultipart.__init__(self)
+ self.subject = '[%s] Resultado del intento %d (ejercicio %d.%d)' % \
+ (conf.get('mail', 'prefijo'), intento.numero,
+ intento.entrega.nroEjercicio, intento.entrega.entrega)
+ self['From'] = conf.get('mail', 'from')
+ self['To'] = intento.mailRespuesta
+ self['Reply-To'] = conf.get('mail', 'admin')
+ self['Return-Path'] = conf.get('mail', 'admin')
+ self['X-Mailer'] = 'sercom ' + sercom.VERSION
+ self['X-Priority'] = '5'
+ self.epilogue = 'Para ver correctamente este e-mail su cliente debe ' \
+ 'soportar MIME.\n\n'
+ self.prologue = '' # Garantiza que termine en \n el mensaje
+ self.attach(MIMEMessage(MIMEText('', 'plain', 'iso-8859-1')))
+ self.resultado = None
+ def __set_body(self, body):
+ self.get_payload(0).get_payload(0).set_payload(body)
+ def __get_body(self):
+ return self.get_payload(0).get_payload(0).get_payload()
+ body = property(__get_body, __set_body, doc='Cuerpo del mensaje.')
+ def attachText(self, text, nombre=None, subtype='plain'):
+ from email.MIMEText import MIMEText
+ attach = MIMEText(text, subtype, 'iso-8859-1')
+ if nombre:
+ attach.add_header('Content-Disposition', 'attachment', filename=nombre)
+ self.attach(attach)
+ def send(self, resultado=None):
+ import smtplib
+ global conf
+ smtp = smtplib.SMTP(conf.get('mail', 'smtp'))
+ if resultado:
+ self.subject += ': ' + resultado
+ self['Subject'] = self.subject
+ smtp.sendmail(self['From'], self['To'], self.as_string())
+ smtp.close()
+ def agregarResultado(self, prueba):
+ if not prueba.casoDePrueba.privado:
+ if prueba.pasada:
+ result = 'BIEN'
+ else:
+ result = 'ERROR'
+ self.body += '''
+Prueba '%s': %s
+%s
+''' % (prueba.casoDePrueba.nombre, result, prueba.observaciones or '')
+ pass
+
+
# Manejadores de señales
signal.signal(signal.SIGTERM, sigterm)
signal.signal(signal.SIGINT, sigterm)
intervalo = float(conf.get('general', 'intervalo'))
# Utilizo el directorio de datos como base para todos los SQLObjects
-sercom.sqlobject.dir_base = conf.get('general', 'data_dir')
+sercom.sqlo.dir_base = conf.get('general', 'data_dir')
# Hasta que nos maten
while continuar:
time.sleep(intervalo)
continue
log.info('Nuevo intento a probar (%s)', intento)
+ mail = MailIntento(intento)
# Compila
- if not compilar(intento):
- #TODO mandar mail
+ compilar(intento, mail)
+ if not intento.compila:
+ mail.send('NO COMPILA')
continue
# Ejecución de casos de prueba
intento.inicioPruebas = datetime.datetime.now()
- pruebas = []
+ resultado = True
for caso_de_prueba in intento.entrega.ejercicio.casosDePrueba:
# Preparo chroot
preparar(intento)
# Pruebo y agrego prueba a la lista
- pruebas.append(probar(intento, caso_de_prueba))
+ prueba = probar(intento, caso_de_prueba, mail)
+ if not prueba.casoDePrueba.privado and not prueba.pasada:
+ resultado = False
# Limpio chroot
limpiar(intento)
intento.finPruebas = datetime.datetime.now()
+ # Envío mail con resultado al alumno
+ if resultado:
+ mail.send('ACEPTADO')
+ else:
+ mail.send('FALLA PRUEBAS')
# Limpio directorio
log.debug('Borrando ejecutable y código objeto (*.o)')
os.remove(os.path.join(intento.path, 'tp'))
[os.remove(obj) for obj in glob.glob(os.path.join(intento.path, '*.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
- pass
- time.sleep(intervalo)
+ # time.sleep(intervalo) #XXX Puede servir para enlentecer el server