Rename proxy package (re-run of #4550) (#4638)

* Rename agent/proxy package to reflect that it is limited to managed proxy processes

Rationale: we have several other components of the agent that relate to Connect proxies for example the ProxyConfigManager component needed for Envoy work. Those things are pretty separate from the focus of this package so far which is only concerned with managing external proxy processes so it's nota good fit to put code for that in here, yet there is a naming clash if we have other packages related to proxy functionality that are not in the `agent/proxy` package.

Happy to bikeshed the name. I started by calling it `managedproxy` but `managedproxy.Manager` is especially unpleasant. `proxyprocess` seems good in that it's more specific about purpose but less clearly connected with the concept of "managed proxies". The names in use are cleaner though e.g. `proxyprocess.Manager`.

This rename was completed automatically using golang.org/x/tools/cmd/gomvpkg.

Depends on #4541

* Fix missed windows tagged files

agent/agent.go

@@ -27,7 +27,7 @@ import (
 	"github.com/hashicorp/consul/agent/config"
 	"github.com/hashicorp/consul/agent/consul"
 	"github.com/hashicorp/consul/agent/local"
-	"github.com/hashicorp/consul/agent/proxy"
+	"github.com/hashicorp/consul/agent/proxyprocess"
 	"github.com/hashicorp/consul/agent/structs"
 	"github.com/hashicorp/consul/agent/systemd"
 	"github.com/hashicorp/consul/agent/token"
@@ -211,7 +211,7 @@ type Agent struct {
 	tokens *token.Store
 
 	// proxyManager is the proxy process manager for managed Connect proxies.
-	proxyManager *proxy.Manager
+	proxyManager *proxyprocess.Manager
 
 	// proxyLock protects proxy information in the local state from concurrent modification
 	proxyLock sync.Mutex
@@ -282,7 +282,7 @@ func (a *Agent) setupProxyManager() error {
 	if err != nil {
 		return fmt.Errorf("[INFO] agent: Connect managed proxies are disabled due to providing an invalid HTTP configuration")
 	}
-	a.proxyManager = proxy.NewManager()
+	a.proxyManager = proxyprocess.NewManager()
 	a.proxyManager.AllowRoot = a.config.ConnectProxyAllowManagedRoot
 	a.proxyManager.State = a.State
 	a.proxyManager.Logger = a.logger

agent/proxyprocess/daemon.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"fmt"

agent/proxyprocess/daemon_test.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"io/ioutil"

agent/proxyprocess/exitstatus_other.go

@@ -1,6 +1,6 @@
 // +build !darwin,!linux,!windows
 
-package proxy
+package proxyprocess
 
 import "os"
 

agent/proxyprocess/exitstatus_syscall.go

@@ -1,6 +1,6 @@
 // +build darwin linux windows
 
-package proxy
+package proxyprocess
 
 import (
 	"os"

agent/proxyprocess/manager.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"fmt"

agent/proxyprocess/manager_test.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"io/ioutil"

agent/proxyprocess/noop.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 // Noop implements Proxy and does nothing.
 type Noop struct{}

agent/proxyprocess/noop_test.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"testing"

agent/proxyprocess/process.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"strings"

agent/proxyprocess/process_unix.go

@@ -1,6 +1,6 @@
 // +build !windows
 
-package proxy
+package proxyprocess
 
 import (
 	"fmt"

agent/proxyprocess/process_windows.go

@@ -1,6 +1,6 @@
 // +build windows
 
-package proxy
+package proxyprocess
 
 import (
 	"os"

agent/proxyprocess/proxy.go

@@ -5,7 +5,7 @@
 //
 // This package does not contain the built-in proxy for Connect. The source
 // for that is available in the "connect/proxy" package.
-package proxy
+package proxyprocess
 
 import (
 	"github.com/hashicorp/consul/agent/structs"

agent/proxyprocess/proxy_test.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"fmt"

agent/proxyprocess/root.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"os"

agent/proxyprocess/snapshot.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 import (
 	"encoding/json"

agent/proxyprocess/test.go

@@ -1,4 +1,4 @@
-package proxy
+package proxyprocess
 
 // defaultTestProxy is the test proxy that is instantiated for proxies with
 // an execution mode of ProxyExecModeTest.

command/connect/proxy/proxy.go

@@ -12,7 +12,7 @@ import (
 	"sort"
 	"strconv"
 
-	proxyAgent "github.com/hashicorp/consul/agent/proxy"
+	proxyAgent "github.com/hashicorp/consul/agent/proxyprocess"
 	"github.com/hashicorp/consul/api"
 	"github.com/hashicorp/consul/command/flags"
 	proxyImpl "github.com/hashicorp/consul/connect/proxy"