From d479d410c6c9854d01af3fa1ba798aa28dd76823 Mon Sep 17 00:00:00 2001
From: Spencer <spalger@users.noreply.github.com>
Date: Tue, 26 Sep 2017 13:37:17 -0700
Subject: [PATCH] [cli/dev] rely on commander for deduping argv (#14181)

In eb317804b559aaf41e94870f9f1f19030a3f16fa I started combining custom argv with the process's argv, and naively used union to combine the lists. This breaks cli arguments that are supposed to be repeated and isn't necessary since commander handles parsing the argv and deduping/merging based on config.
---
 src/cli/cluster/worker.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/cli/cluster/worker.js b/src/cli/cluster/worker.js
index 06528f276531327..25f45ca3c1da0f1 100644
--- a/src/cli/cluster/worker.js
+++ b/src/cli/cluster/worker.js
@@ -39,10 +39,9 @@ export default class Worker extends EventEmitter {
     this.clusterBinder = new BinderFor(cluster);
     this.processBinder = new BinderFor(process);
 
-    const argv = _.union(baseArgv, opts.argv || []);
     this.env = {
       kbnWorkerType: this.type,
-      kbnWorkerArgv: JSON.stringify(argv)
+      kbnWorkerArgv: JSON.stringify(baseArgv.concat(opts.argv || []))
     };
   }