Skip to content

Commit

Permalink
Merge pull request #219 from gemini-testing/sp.removeWorkerRetries
Browse files Browse the repository at this point in the history
fix: do not retry worker init on fail
  • Loading branch information
sipayRT authored Feb 7, 2018
2 parents e68d52e + 28352cd commit 65c8da6
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion lib/runner/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,8 @@ module.exports = class MainRunner extends AsyncEmitter {
const params = {
maxConcurrentWorkers: this._config.system.workers,
maxConcurrentCallsPerWorker: Infinity,
autoStart: true
autoStart: true,
maxRetries: 0
};

return workerFarm(params, workerFilepath, [
Expand Down
3 changes: 2 additions & 1 deletion test/lib/runner/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,8 @@ describe('Runner', () => {
assert.calledOnceWith(workerFarm, {
maxConcurrentWorkers: 100500,
maxConcurrentCallsPerWorker: Infinity,
autoStart: true
autoStart: true,
maxRetries: 0
}, path.join(process.cwd(), 'lib/worker/index.js'), [
{name: 'init', broadcast: true},
{name: 'syncConfig', broadcast: true},
Expand Down

0 comments on commit 65c8da6

Please sign in to comment.