andrewchilds / overcast

Orchestrate VMs from your terminal
MIT License
478 stars 36 forks source link

EventEmitter memory leak detected. 11 listeners added on v0.4.5 #14

Closed thariman closed 10 years ago

thariman commented 10 years ago

Running v0.4.5 Work fine for overcast run workers "egrep 'MMC' *.log | tail -n1 | cut -c29-" --continueOnError | egrep "HPM" > HPM.out on 37 node

But got the same error if using -p

(node) warning: possible EventEmitter memory leak detected. 11 listeners added. Use emitter.setMaxListeners() to increase limit. Trace at ReadStream.EventEmitter.addListener (events.js:160:15) at sshExec (/usr/lib/node_modules/overcast/modules/ssh.js:121:17) at runOnInstance (/usr/lib/node_modules/overcast/modules/ssh.js:38:3) at /usr/lib/node_modules/overcast/modules/ssh.js:13:7 at forOwn (/usr/lib/node_modules/overcast/node_modules/lodash/dist/lodash.js:2105:15) at Function.forEach (/usr/lib/node_modules/overcast/node_modules/lodash/dist/lodash.js:3302:9) at Object.exports.run (/usr/lib/node_modules/overcast/modules/ssh.js:12:7) at Object.exports.run (/usr/lib/node_modules/overcast/modules/commands/run.js:15:7) at execute (/usr/lib/node_modules/overcast/overcast.js:30:13) at /usr/lib/node_modules/overcast/overcast.js:9:7

andrewchilds commented 10 years ago

This should be fixed in v0.4.6. Thanks!

thariman commented 10 years ago

Tried it. Perfect. Thank you