Some updates proposed to your repository #6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
En Dockerfile - cambiamos la imagen base a Centos 8 Stream - instalamos openjdk - bajamos Tomcat y lo instalamos - ejecutamos tomcat En README.md - Cambiamos link de instalación de docker a podman - Cambiamos referencias a docker por podman - Agregamos ejecución de otra instancia de contenedor para balanceo - Agregamos configuración de balanceador con mod_proxy_balancer en apache - Agregamos configuración de SELinux para balanceo.
Update tomcat URL.
Se corrigió un error en el Dockerfile con la versión de tomcat Se corrigió en el README el formato de la configuración del reverse proxy.
Added simple reverse proxy configuration for a single container.
Corrección de enlace
Se cambia el Dockerfile por Containerfile, y se actualiza para usar Centos Stream 9, OpenJDK 11 y la última versión de Tomcat 8. Se mejora el README.md para que sea más legible y consistente y se aclaran algunos puntos.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello, I've been using your proyect for some time in my teaching of containers, and over time I've been doing some changes to keep it current, so, if you like them, can merge them to your proyect.
I've changed from Dockerfile to Container file, and expanded the instructions in the README to use podman instead of docker. Also, updated the Containerfile to use Centos Steam 9, OpenJDK 11 and the latest release of Tomcat 8.
Also included instructions to create a second container and add Load Balancing and reverse proxy to Apache.
Best regards,
Enrique Verdes