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

Fix Docker install #130

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

Conversation

ryuran
Copy link
Contributor

@ryuran ryuran commented Jun 23, 2017

Fix #121

@ryuran ryuran force-pushed the fix/docker-install branch from fc47bda to a0d7fa8 Compare June 23, 2017 08:58
@@ -10,8 +10,7 @@ Merci de suivre les instructions sur [le site officiel](https://www.docker.com/p
Création de l’image en locale
-------------------------------------------------------------------------------

> **NOTE :** _À terme,
il est prévu que l’image soit disponible sur le Docker hub._
> **NOTE :** _l’image étant disponible sur le Docker hub, cette étape n’est pas necéssaire à part si vous voulez votre propre image lié à votre projet._

Choose a reason for hiding this comment

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

votre propre image liée à votre projet

Dockerfile Outdated
@@ -15,13 +15,16 @@ RUN mkdir -p /usr/src/app
WORKDIR /usr/src/app
COPY . /usr/src/app

Choose a reason for hiding this comment

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

N'a pas de sens étant déjà monté via volume dans ta commande.

Copy link

@jycamier jycamier left a comment

Choose a reason for hiding this comment

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

Rajouter une docker-compose pour simplifier la vie des Garden

version: '2'
services:
    gulp:
        image: cleverage/garden-starter-kit
        volumes:
            - "./:/usr/src/app"
        ports:
            - "8000:8000"
            - "3001:3001"
        command: ['gulp', 'live']
    npm-install:
        image: cleverage/garden-starter-kit
        volumes:
            - "./:/usr/src/app"
        ports:
            - "8000:8000"
            - "3001:3001"
        command: ['npm', 'install']
    clever-garden-kit:
        image: cleverage/garden-starter-kit
        volumes:
            - "./:/usr/src/app"
        ports:
            - "8000:8000"
            - "3001:3001"

du coup, ça simplifie les commandes

$ docker-compose up npm-install
$ docker-compose up gulp
$ docker-compose up clever-garden-kit '<ma commande npm>'

@shavounet
Copy link

Attention aux ports cela dit. Si tu fais un docker-compose up tout seul ça ne marchera pas car les mappings sont identiques sur chaque container... (et tous ne sont pas nécessaire selon la commande ?)

@jycamier
Copy link

Oui c'est certain. Après concrètement, je crois que seul le gulp est intéressant en fait en niveau mapping de port. Le reste, on peut le supprimer

@ryuran
Copy link
Contributor Author

ryuran commented Jun 23, 2017

C'est pas si simple.
Toutes les actions qui lancent gulp connect on besoin du mapping de port

Cependant à l'usage on lance une container qui fait le gulp connect dans un terminal et dans un autre on lance les autre commande gulp

J’ai mon idée. je vous soumet les modif dés que j'ai avancé

@ryuran ryuran force-pushed the fix/docker-install branch 2 times, most recently from 51d739e to 44aafec Compare June 23, 2017 15:11
@ryuran ryuran force-pushed the fix/docker-install branch from 44aafec to d486cb1 Compare August 28, 2017 08:49
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.

4 participants