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

Issue 78 dockerizacion #122

Closed
wants to merge 6 commits into from

Conversation

oinak
Copy link

@oinak oinak commented Aug 8, 2015

Configuración para desarrollar con docker

  • El README está actualizado con las instrucciones en ambos idiomas
  • Todos los tests pasan
  • He probado a registrarme y crear un debate con éxito.

@oinak oinak mentioned this pull request Aug 8, 2015
@kikito
Copy link
Contributor

kikito commented Aug 12, 2015

Esto no se nos ha olvidado, lo que ocurre es que hay otras cosas que ahora mismo son prioritarias.

Un par de reflexiones de momento:

El hecho de tener dos .yml.examples distintos, y el hecho de tener un "if" en development.rb, son un tanto "feas". Las dos ocurren porque en una máquina de desarrollo "normal", la base de datos y letter_opener están el localhost, mientras que en docker (con esta solución) están en dockers distintos.

Una posible solución podría ser prescindir de los diferentes dockers y meterlos todos en el mismo container (rails, postgres y letter_opener). Con eso también nos quitaríamos la dependencia de docker-compose: lo podríamos hacer todo con un Dockerfile. La desventaja obvia es que esto no es "the docker way", donde cada servicio está en un componente distinto. El tradeoff aquí es elegir entre eso (the docker way) y tener más complejidad en los ficheros de configuración.

En estos momentos tiendo más a no hacer las cosas "the Docker way": lo que pretende la issue es facilitar el desarrollo, no de implementar containers de la forma más correcta. No obstante le quiero dar una vuelta más.

@oinak
Copy link
Author

oinak commented Aug 12, 2015

Yo lo haría "docker way", se puede tener mailcatcher en local (sin docker) con un simple "gem install" pero no se puede tener letter_opener en containers, ni servidores de staging, para mi mailcatcher es superior por versatilidad y abandonaría (abandoné en mis proyectos) letter_opener.

@kikito
Copy link
Contributor

kikito commented Sep 11, 2015

Hola, de momento vamos a cerrar esta pr, cuando tenga un poco de tiempo lo haremos con una solución un poco más "de andar por casa".

De todas formas te agradecemos muchísimo tu pull request. Seguro que habrá algo en el futuro que nos puedas mandar y que te aceptemos.

@kikito kikito closed this Sep 11, 2015
@oinak
Copy link
Author

oinak commented Sep 13, 2015

siento que no os haya encajado, para mi desarrollar dockerizado ha sido un punto sin retorno, no me imagino volviendo a "the old way", a ver si puedo convertir esto en una charla de madrid-rb o algo

@dalareo
Copy link

dalareo commented Mar 22, 2016

Hola! Estamos estudiando el despliegue en Coruña y somos partidarios de hacerlo "a la docker". Una pena que este pull request no esté integrado en el master. Hay alguna versión dockerizable que sea estable como para llevar a producción?

@kikito
Copy link
Contributor

kikito commented Mar 23, 2016

@dalareo No hay ninguna versión dockerizable y estable.

palomahnhp pushed a commit to palomahnhp/consul that referenced this pull request May 13, 2016
bertocq pushed a commit that referenced this pull request Feb 7, 2018
…-54-revert-46-revert-42-revert-37-DEV

Revert "Revert "Revert "Revert "Revert "Update 04/10/2017"""""
markusgeert pushed a commit to markusgeert/consul that referenced this pull request Jun 14, 2023
…button

Change poll button if results are enabled
javierm pushed a commit to rockandror/consuldemocracy that referenced this pull request May 10, 2024
javierm pushed a commit to rockandror/consuldemocracy that referenced this pull request May 10, 2024
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.

None yet

3 participants