From 014dee07fa6fa5971271524d203258e05accbb29 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 06528f2765313..25f45ca3c1da0 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 || []))
     };
   }