New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dockerfile for dockerize application #195
Conversation
Ep, aixo no funcionarà perque el mailtoticket encara no està passat a python 3 (#187) . Potser primer s'hauria de provar amb python 2.7 i llavors fer els canvis. |
Ya lohe pasado a version 2.7, pero no se como probarlo.... se te ocurre como probarlo? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
He fet alguns comentaris amb propostes de canvi però en general m'agrada l'aproximació senzilla d'aquest contenidor.
Evita que copiem a la imatge docker el directori recomanat al README per als virtualenv.
No cal modificar el PATH perquè no hi ha cap virtualenv.
Amb el suport per LDAP cal tenir instal·lades algunes llibreries addicionals.
Enlloc de guardar dins la imatge la configuració de la instància és preferible muntar-la com un volum.
Altrament la instal·lació de les dependències no es pot desar a la cache de les imatges perquè la còpia invalida tot el que vingui després.
Firts version