[ARVADOS] updated: 1.2.0-46-g5b3935a02

Git user git at public.curoverse.com
Fri Aug 31 16:16:18 EDT 2018


Summary of changes:
 lib/dispatchcloud/azure.go | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

       via  5b3935a024c9b7339fb53d731e5654fe7419ed92 (commit)
      from  d9ac7243e81ee55504bbcb96bb4edebd1dc10056 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 5b3935a024c9b7339fb53d731e5654fe7419ed92
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Fri Aug 31 16:15:55 2018 -0400

    13964: Fix string pointer capture
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/lib/dispatchcloud/azure.go b/lib/dispatchcloud/azure.go
index 28efe4f5c..6cc4e9908 100644
--- a/lib/dispatchcloud/azure.go
+++ b/lib/dispatchcloud/azure.go
@@ -264,7 +264,8 @@ func (az *AzureProvider) Create(ctx context.Context,
 	tags := make(map[string]*string)
 	tags["created-at"] = &timestamp
 	for k, v := range newTags {
-		tags["dispatch-"+k] = &v
+		newstr := v
+		tags["dispatch-"+k] = &newstr
 	}
 
 	tags["dispatch-instance-type"] = &instanceType.Name
@@ -552,7 +553,8 @@ func (ai *AzureInstance) SetTags(ctx context.Context, newTags InstanceTags) erro
 		}
 	}
 	for k, v := range newTags {
-		tags["dispatch-"+k] = &v
+		newstr := v
+		tags["dispatch-"+k] = &newstr
 	}
 
 	vmParameters := compute.VirtualMachine{
@@ -599,7 +601,7 @@ func (ai *AzureInstance) VerifyPublicKey(ctx context.Context, receivedKey ssh.Pu
 		if remoteFingerprint == tg {
 			return nil
 		} else {
-			return fmt.Errorf("Key fingerprint did not match")
+			return fmt.Errorf("Key fingerprint did not match, expected %q got %q", tg, remoteFingerprint)
 		}
 	}
 
@@ -619,7 +621,7 @@ func (ai *AzureInstance) VerifyPublicKey(ctx context.Context, receivedKey ssh.Pu
 	log.Printf("%q %q", nodetoken, expectedToken)
 
 	if strings.TrimSpace(nodetoken) != expectedToken {
-		return fmt.Errorf("Node token did not match")
+		return fmt.Errorf("Node token did not match, expected %q got %q", expectedToken, nodetoken)
 	}
 
 	sess, err = client.NewSession()
@@ -637,7 +639,7 @@ func (ai *AzureInstance) VerifyPublicKey(ctx context.Context, receivedKey ssh.Pu
 	log.Printf("%q %q", remoteFingerprint, sp[1])
 
 	if remoteFingerprint != sp[1] {
-		return fmt.Errorf("Key fingerprint did not match")
+		return fmt.Errorf("Key fingerprint did not match, expected %q got %q", sp[1], remoteFingerprint)
 	}
 
 	tags["ssh-pubkey-fingerprint"] = sp[1]

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list