Skip to content

Agregados los ficheros para testear otras versiones. #2

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

abdedarghal111
Copy link

Temas relevantes:

  • Han sido agregados y se despliegan correctamente los nuevos contenedores.
  • No he realizado testing a las versiones agregadas, solo creado correctamente los contenedores.
  • Se ha actualizado el parámetro de mysql porque el anterior estaba deprecado.

@@ -22,3 +22,30 @@ services:
restart: always
ports:
- 8080:8080

# facturascripts74:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Te has dejado un montón de código comentado. Mejor quítalo.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El código comentado se refiere a las demás versiones.


COPY facturascripts.sh /usr/local/bin/facturascripts
RUN chmod +x /usr/local/bin/facturascripts
CMD ["../../facturascripts"]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Por qué ../../ ???

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

porque referencia al script original, así se evita copiar y pegar el archivo 3 veces.

@abdedarghal111
Copy link
Author

Ya he removido el código que me has pedido, si hay que realizar algo más avisame.

@elguitarraverde
Copy link
Contributor

elguitarraverde commented Mar 24, 2025

@abdedarghal111 revisa que si solo lo que cambia es la versión de php, puedes reutilizar el Dockerfile pasandole como argumento la versión de php

ARG PHP_VERSION=7.4
FROM php:${PHP_VERSION}-apache

y cuando construyas el contenedor le pasas la versión. de esta forma no vas a tener que tocar el achivo aunque vaya por la vesión 9.2

igual pasa con la vesión de FacturaScripts. si la configuras como argumento se la vas pasando desde fuera y ya no tienes que tocar los archivos en cada versión

@abdedarghal111
Copy link
Author

abdedarghal111 commented Apr 1, 2025

@abdedarghal111 revisa que si solo lo que cambia es la versión de php, puedes reutilizar el Dockerfile pasandole como argumento la versión de php

ARG PHP_VERSION=7.4 FROM php:${PHP_VERSION}-apache

y cuando construyas el contenedor le pasas la versión. de esta forma no vas a tener que tocar el achivo aunque vaya por la vesión 9.2

igual pasa con la vesión de FacturaScripts. si la configuras como argumento se la vas pasando desde fuera y ya no tienes que tocar los archivos en cada versión

Acabo de investigarlo en un momento (búsquedas rápidas y chatgpt) y al parecer no es posible. No se puede agregar variables de entorno para buscar imágenes de docker, pero lo que si se podría hacer es construir la imagen desde cero e instalar esa versión ya que se consideran instrucciones de terminal y si admiten variables de entorno, aunque es más tedioso.

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.

3 participants