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

Docker image with pm2 balancer #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Docker image with pm2 balancer #5

wants to merge 1 commit into from

Conversation

Gingernaut
Copy link

Added a Dockerfile to build and run this project.

@crisbal
Copy link
Owner

crisbal commented Oct 2, 2017

Thanks for the PR. Never heard of pm2 before, looks cool.

We were discussing if and how to dockerize this project in #4

As I said earlier I don't know if it is worth providing images and/or complex dockerfiles for the project, because it is (in my opinion) out of scope. As of right now I will leave this PR open, it is definitely useful for someone but it might not be useful for the target audience of this repository

I will link this PR to the docs so people looking for dockerizing the project will find this

Thanks again

@crisbal
Copy link
Owner

crisbal commented Oct 2, 2017

Added a link to this PR with 583c506

@vsorrokin
Copy link

vsorrokin commented Dec 13, 2017

docker build -t projectname
is incorrect - try to run this.

Should be
docker build -t projectname ./

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.

3 participants