prevent plugins in Windows from creating terminals (#2701)

Signed-off-by: Wayne Witzel III <wayne@riotousliving.com>

changelogs/unreleased/2143-wwitzel3

@@ -0,0 +1 @@
+fixed Go plugins spawning terminals under Windows

pkg/plugin/manager.go

@@ -12,7 +12,6 @@ package plugin
 import (
 	"context"
 	"fmt"
-	"os/exec"
 	"path/filepath"
 	"sort"
 	"sync"
@@ -60,10 +59,12 @@ func (f *DefaultClientFactory) Init(ctx context.Context, cmd string) Client {
 		dashLogger: log.From(ctx),
 	}
 
+	c := pluginCmd(cmd)
+
 	return plugin.NewClient(&plugin.ClientConfig{
 		HandshakeConfig: Handshake,
 		Plugins:         pluginMap,
-		Cmd:             exec.Command(cmd),
+		Cmd:             c,
 		AllowedProtocols: []plugin.Protocol{
 			plugin.ProtocolGRPC,
 		},

pkg/plugin/plugincmd_unix.go

@@ -0,0 +1,17 @@
+// +build !windows
+
+/*
+Copyright (c) 2019 the Octant contributors. All Rights Reserved.
+SPDX-License-Identifier: Apache-2.0
+*/
+
+package plugin
+
+import (
+	"os/exec"
+)
+
+func pluginCmd(cmd string) *exec.Cmd {
+	pluginCmd := exec.Command(cmd)
+	return pluginCmd
+}

pkg/plugin/plugincmd_windows.go

@@ -0,0 +1,19 @@
+// +build windows
+
+/*
+Copyright (c) 2019 the Octant contributors. All Rights Reserved.
+SPDX-License-Identifier: Apache-2.0
+*/
+
+package plugin
+
+import (
+	"os/exec"
+	"syscall"
+)
+
+func pluginCmd(cmd string) *exec.Cmd {
+	pluginCmd := exec.Command(cmd)
+	pluginCmd.SysProcAttr = &syscall.SysProcAttr{HideWindow: true}
+	return pluginCmd
+}