Skip to content
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

2.12.x series #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

2.12.x series #1

wants to merge 3 commits into from

Conversation

ftassi
Copy link
Member

@ftassi ftassi commented May 20, 2022

@edelprino ho modificato un po' il dockerfile.

Ci sono due modifiche, ma ho fatto tutto un po' a martellate e non mi sono preso la briga di dividere le cose.

La modifica numero 1 è che questa versione installa le release esatte che ci ha segnalato matteo per il corso. Non so se è importante, ma ho preferito avere un ambiente con la roba giusta, non si sa mai.
La modifica numero 2 riguarda la gestione degli utenti: ho creato un utente applicativo me e un file di entrypoint che, a runtime gli cambia l'id con quello dell'host.

Dato che la sua home è vuota questo dovrebbe essere super veloce, ma in compenso ci permette di eseguire il container con quell'utente e avere il giusto mapping per non avere menate di permessi tra host e container.

Questa seconda roba da sola si potrebbe applicare al tuo docker file se volessimo usare le ultime release invece di quelle segnalate.

Non ho ancora provato a fare esercizi base per assicurarmi che non ci siano altri problemi di permess, nel weekend faccio dei test, ma credo dovrebbe andare tutto bene

ftassi added 2 commits May 20, 2022 08:53
This also tweaks permission management a little bit so we don't have any
issues sharing project volumes with the container
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant