8000 Include docker configuration, load config from env, and add healthchecks by fgallese · Pull Request #56 · PabloCastellano/libreborme · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Apr 15, 2025. It is now read-only.

Include docker configuration, load config from env, and add healthchecks #56

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

Include docker configuration, load config from env, and add healthchecks #56

wants to merge 2 commits into from

Conversation

fgallese
Copy link
@fgallese fgallese commented Apr 4, 2020

Buenas !

Estoy deployando tu proyecto en Kubernetes, por lo que necesite hacer algunas modificaciones:

  • Los settings se cargan desde variables de ambiente, con defaults de forma que si no se establecen todo defaultea a los valores actuales presentes en el proyecto.
  • Los settings pueden ser sobreescritos desde un archivo .env (util para ambiente local).
  • Incluyo un dockerfile para poder correr el proyecto en docker.
  • Agrego healthchecks, util para kubernetes.
  • Edite el docker-compose.yaml para que funcione desde 0 sin tener que configurar nada.

Creo este PR ya que asumo que puede ser util para otras personas que tambien esten pensando en correr este proyecto con docker.

@fgallese
Copy link
Author
fgallese commented Apr 4, 2020

He tenido tambien que actualizar la dependencia bormeparser ya que la version que instalaba tenia un bug que veo has corregido en los ultimos releases.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@tresmonitos tresmonitos tresmonitos approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fgallese @tresmonitos
0