drone / drone-wall

Dashboard for the Drone CI server
260 stars 44 forks source link

Drone wall crash, undefined: 0 #25

Closed epipho closed 8 years ago

epipho commented 9 years ago

At 12:01:01 the following error shows up in the log and drone-wall exits

Nov 18 12:01:01 core-01 sh[867]: undefined:0
Nov 18 12:01:01 core-01 sh[867]: ^
Nov 18 12:01:01 core-01 sh[867]: SyntaxError: Unexpected end of input
Nov 18 12:01:01 core-01 sh[867]: at Object.parse (native)
Nov 18 12:01:01 core-01 sh[867]: at IncomingMessage.<anonymous> (/webapp_root/server.js:45:29)
Nov 18 12:01:01 core-01 sh[867]: at IncomingMessage.emit (events.js:117:20)
Nov 18 12:01:01 core-01 sh[867]: at _stream_readable.js:943:16
Nov 18 12:01:01 core-01 sh[867]: at process._tickCallback (node.js:419:13)

No other log entries surrounding the crash and restarting the process made everything happy again.

oren commented 9 years ago

I had the same issue. it might be something with docker. In the meantime try running it without docker. It's working for me:

  npm install
  npm install bower grunt-cli --save-dev                                                                                                                                                                            
  ./node_modules/.bin/bower --allow-root install
  ./node_modules/.bin/grunt deploy
  API_SCHEME=http API_DOMAIN=localhost API_TOKEN=<token> API_PORT=3000 PORT=4000 node server.js

Notice the following:

Let me know if it helps.

scottferg commented 9 years ago

I think this might be a case of the the official node container having some old bugs that have since been fixed. Testing it now to verify. I did verify that @oren 's solution works well if you want to run it outside of a container.

Tathanen commented 8 years ago

This should be "fixed" insomuch as not much of this old config is even present in the current version. There's an issue (https://github.com/drone/drone-wall/issues/32) for a new dockerfile, let us know if that config, when implemented, exhibits any similar issue. Closing for now.