From 4d99c19428485e27beaa0ec3f9f8abed31833bad Mon Sep 17 00:00:00 2001 From: Alex Dadgar Date: Sat, 22 Oct 2016 18:20:50 -0700 Subject: [PATCH] Small fixes --- client/client.go | 4 ++-- client/task_runner.go | 3 +-- nomad/vault.go | 2 ++ vendor/vendor.json | 8 ++++---- 4 files changed, 9 insertions(+), 8 deletions(-) diff --git a/client/client.go b/client/client.go index 8028b75c8..76193dac9 100644 --- a/client/client.go +++ b/client/client.go @@ -1711,8 +1711,8 @@ func (c *Client) deriveToken(alloc *structs.Allocation, taskNames []string, vcli // Derive the tokens var resp structs.DeriveVaultTokenResponse if err := c.RPC("Node.DeriveVaultToken", &req, &resp); err != nil { - c.logger.Printf("[ERR] client.vault: failed to derive vault tokens: %v", err) - return nil, fmt.Errorf("failed to derive vault tokens: %v", err) + c.logger.Printf("[ERR] client.vault: DeriveVaultToken RPC failed: %v", err) + return nil, fmt.Errorf("DeriveVaultToken RPC failed: %v", err) } if resp.Error != nil { c.logger.Printf("[ERR] client.vault: failed to derive vault tokens: %v", resp.Error) diff --git a/client/task_runner.go b/client/task_runner.go index dfb84b94d..bbee77b97 100644 --- a/client/task_runner.go +++ b/client/task_runner.go @@ -587,8 +587,7 @@ OUTER: } // deriveVaultToken derives the Vault token using exponential backoffs. It -// returns the Vault token and whether the token is valid. If it is not valid we -// are shutting down +// returns the Vault token and whether the manager should exit. func (r *TaskRunner) deriveVaultToken() (token string, exit bool) { attempts := 0 for { diff --git a/nomad/vault.go b/nomad/vault.go index 0a1327b9d..5f11e8514 100644 --- a/nomad/vault.go +++ b/nomad/vault.go @@ -215,6 +215,7 @@ func (v *vaultClient) flush() { v.client = nil v.auth = nil v.connEstablished = false + v.connEstablishedErr = nil v.token = "" v.tokenData = nil v.revoking = make(map[*structs.VaultAccessor]time.Time) @@ -357,6 +358,7 @@ OUTER: v.l.Lock() v.connEstablished = true + v.connEstablishedErr = nil v.l.Unlock() } diff --git a/vendor/vendor.json b/vendor/vendor.json index c8afb90b9..c0624a8ce 100644 --- a/vendor/vendor.json +++ b/vendor/vendor.json @@ -270,14 +270,14 @@ { "checksumSHA1": "tdhmIGUaoOMEDymMC23qTS7bt0g=", "path": "github.com/docker/docker/pkg/ioutils", - "revision": "da39e9a4f920a15683dd0f23923c302d4db6eed5", - "revisionTime": "2016-05-28T08:11:04Z" + "revision": "52debcd58ac91bf68503ce60561536911b74ff05", + "revisionTime": "2016-05-20T15:17:10Z" }, { "checksumSHA1": "tdhmIGUaoOMEDymMC23qTS7bt0g=", "path": "github.com/docker/docker/pkg/ioutils", - "revision": "52debcd58ac91bf68503ce60561536911b74ff05", - "revisionTime": "2016-05-20T15:17:10Z" + "revision": "da39e9a4f920a15683dd0f23923c302d4db6eed5", + "revisionTime": "2016-05-28T08:11:04Z" }, { "checksumSHA1": "ndnAFCfsGC3upNQ6jAEwzxcurww=",