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

Add Vagrant+chef support #69

Closed
wants to merge 1 commit into from
Closed

Conversation

telemaco
Copy link

No description provided.

@kikito kikito self-assigned this Jul 31, 2015
@kikito
Copy link
Contributor

kikito commented Jul 31, 2015

Hola!

Un par de comentarios:

  • El comando vagrant up parece dar un error al instalar chef_gem[pg] (Traza completa: https://gist.github.com/kikito/67880f71d5e62d0fa08e). Al final la aplicación funciona ¿Es normal este comportamiento? Si es así, habría que indicarlo en el README.
  • No sé si es mucho pedir: uno de los usos habituales de un vagrant es poder correr los tests sin tener que instalarse todas las dependencias en local. ¿Crees que podrías añadir a tu PR una base de datos de tests e instrucciones para ejecutar bundle exec bin/rspec dentro del vagrant?

@oinak
Copy link

oinak commented Jul 31, 2015

tiiiiiiios, vagrant es el pasado (y lento): Docker a tope!

@telemaco
Copy link
Author

@kikito el error ese es una cosa del instalador de omnibus, el instalador de chef. Que falla, pero luego se recupera y lo hace bien. Pero la salida es un poco fea.
Sobre lo de test. Aquí es donde se hace el despliegue de la app. Que te gustaría tener? Que se pueda configurar distintos environment de rails? Dime como lo quieres y te lo dejo preparado.

@oinak Tengo intención de hacer eso después, aunque para ello molaria que el Ayuntamiento se hiciera cuenta en dockerhub :D

@oinak
Copy link

oinak commented Jul 31, 2015

@telemaco: habla conmigo cuando te pongas con docker, tengo cosas recien probadas en varios proyectos que me han ido de lujo y no necesité cuenta en dockerhub

@oinak
Copy link

oinak commented Jul 31, 2015

de hecho, te ahorraré un paso, te lo he puesto por escrito: http://www.oinak.com/2015/07/the-happy-developer-environment-strikes.html

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

kikito commented Aug 3, 2015

@telemaco, aunque te agradecemos tu contribución, creo que @oinak tiene razón en este caso. Ambos resuelven (más o menos) el mismo problema, pero Vagrant se usa cada vez menos mientras que Docker se usa cada vez más. Así que he decidido cerrar esta PR y he abierto una nueva para la dockerización (#78).

Si cualquiera de vosotros quiere mandarnos otra PR con docker, será bienvenida. Os sugiero que utilicéis la issue para coordinaros (p.ej. comentando si alguno empieza a trabajar en ello) y así evitamos que se repita trabajo.

Gracias!

@kikito kikito closed this Aug 3, 2015
bertocq pushed a commit that referenced this pull request Feb 7, 2018
thiswd added a commit to participemais/consul that referenced this pull request Sep 28, 2020
…ory-filter-and-img-map

Remove category filter and img map
iljandi added a commit to oecd-opsi/consul that referenced this pull request Sep 30, 2021
iljandi added a commit to oecd-opsi/consul that referenced this pull request Oct 6, 2021
microweb10 referenced this pull request in stemvanonshogeland/consul Mar 23, 2022
asterix123 added a commit to SimonMD84/consul that referenced this pull request Dec 6, 2022
* minor view improvement

* bug
markusgeert pushed a commit to markusgeert/consul that referenced this pull request Jun 14, 2023
Fix styles for headers with background images
javierm pushed a commit that referenced this pull request May 16, 2024
Add installing on a digital ocean vps documentation
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