From 4b4dfacda5b008cf7b34892b2f295f8e7fb2de8f Mon Sep 17 00:00:00 2001 From: Seth Hoenig Date: Tue, 7 Jan 2020 08:48:11 -0600 Subject: [PATCH] client: remove unused indirection for referencing consul executable Was thinking about using the testing pattern where you create executable shell scripts as test resources which "mock" the process a bit of code is meant to fork+exec. Turns out that wasn't really necessary in this case. --- client/allocrunner/taskrunner/envoybootstrap_hook.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/client/allocrunner/taskrunner/envoybootstrap_hook.go b/client/allocrunner/taskrunner/envoybootstrap_hook.go index 5de2551a4..29e53fb49 100644 --- a/client/allocrunner/taskrunner/envoybootstrap_hook.go +++ b/client/allocrunner/taskrunner/envoybootstrap_hook.go @@ -43,9 +43,6 @@ type envoyBootstrapHook struct { // Consul's gRPC endpoint. consulHTTPAddr string - // executable is executable file that is consul - executable string - // logger is used to log things logger hclog.Logger }