Skip to content
This repository was archived by the owner on Dec 1, 2025. It is now read-only.

Added ipv6 information so the docker won't fail to run#561

Open
themotu wants to merge 2 commits intosdelements:masterfrom
themotu:master
Open

Added ipv6 information so the docker won't fail to run#561
themotu wants to merge 2 commits intosdelements:masterfrom
themotu:master

Conversation

@themotu
Copy link
Copy Markdown

@themotu themotu commented Jul 16, 2015

No description provided.

@corn13read
Copy link
Copy Markdown

Also fixes the following error

events.js:85
      throw er; // Unhandled 'error' event
            ^
Error: listen EACCES
    at exports._errnoException (util.js:746:11)
    at Server._listen2 (net.js:1139:19)
    at listen (net.js:1182:10)
    at net.js:1280:9
    at dns.js:85:18
    at process._tickCallback (node.js:355:11)

npm ERR! Linux 2.6.32-504.16.2.el6.x86_64
npm ERR! argv "node" "/usr/local/bin/npm" "start"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! code ELIFECYCLE
npm ERR! lets-chat@0.4.2 start: `node app.js`
npm ERR! Exit status 1
npm ERR! 
npm ERR! Failed at the lets-chat@0.4.2 start script 'node app.js'.
npm ERR! This is most likely a problem with the lets-chat package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node app.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls lets-chat
npm ERR! There is likely additional logging output above.
npm ERR! Linux 2.6.32-504.16.2.el6.x86_64
npm ERR! argv "node" "/usr/local/bin/npm" "start"
npm ERR! node v0.12.7
npm ERR! npm  v2.11.3
npm ERR! path npm-debug.log.7da2e9feb6af80ea1c45d89105a542fd
npm ERR! code EACCES
npm ERR! errno -13

npm ERR! Error: EACCES, open 'npm-debug.log.7da2e9feb6af80ea1c45d89105a542fd'
npm ERR!     at Error (native)
npm ERR!  { [Error: EACCES, open 'npm-debug.log.7da2e9feb6af80ea1c45d89105a542fd']
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   path: 'npm-debug.log.7da2e9feb6af80ea1c45d89105a542fd' }
npm ERR! 
npm ERR! Please try running this command again as root/Administrator.

npm ERR! Please include the following file with any support request:
npm ERR!     /usr/src/app/npm-debug.log

Comment thread docker/Dockerfile
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think we want to keep this?

@themotu
Copy link
Copy Markdown
Author

themotu commented Jul 24, 2015

@hhaidar If we run it as node user then it fails, I tested it. But adding node as a group is fine I just don't see the purpose.

@TimWolla
Copy link
Copy Markdown
Contributor

This cannot work anyway. Let's Chat does not listen on IPv6 addresses: https://github.com/sdelements/lets-chat/blob/master/app.js#L202

I am not sure what you are trying to achieve?

@themotu
Copy link
Copy Markdown
Author

themotu commented Jul 29, 2015

Look. Try to build it as is, when you see it fail then try it my way. Don't pull it I don't care. It failed without adding ipv6 to the container for me, so I fixed it and made a pull request.

@TimWolla
Copy link
Copy Markdown
Contributor

Don't pull it I don't care

I am no employee of SD Elements, therefore it is not me who decides. But I care about the Docker image, as I run it in production.

It works just fine, as is. With the restricted user. I do run the Docker daemon with IPv6 support, though.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants