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

[fix] Recrear contenedores con docker-compose #15

Merged
merged 2 commits into from
Sep 25, 2020
Merged

[fix] Recrear contenedores con docker-compose #15

merged 2 commits into from
Sep 25, 2020

Conversation

MinusFour
Copy link
Contributor

@MinusFour MinusFour commented Sep 25, 2020

Continuación de #14, #13, #12, #11...

El archivo de configuración de docker tiene un nombre para el contenedor de webpack. Al intentar levantar un contenedor con la nueva imagen, hay una colisión entre el viejo contenedor y el nuevo contenedor porque usan el mismo nombre. Con la opción --force-recreate docker-compose no debería tener problemas.

También he agregado una pequeña optimización a la build inicial. Lo que tarda mucho en la build es descargar la imagen y extraerla. Será mejor una vez que tenga un nuevo rPI.

Docker compose has a problem with running containers names. If container
names are the same they need to be recreated or they'll clash. This also
removes trailing whitespace.
This will make the github jobs run faster by caching the image
layers. Current layers shouldn't be that big so this is a small
optimization. It's unknown how much free disk space it's using to cache
the layers.
@MinusFour MinusFour merged commit b1291a2 into elhacker-net:dev Sep 25, 2020
@MinusFour MinusFour deleted the fix-workflow-rpi-3 branch September 25, 2020 17:02
@MinusFour MinusFour added fix Fixes something workflow labels Sep 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes something workflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant