[ARVADOS] updated: 1.3.0-1235-g380a54a7d

Git user git at public.curoverse.com
Mon Jul 1 14:57:27 UTC 2019


Summary of changes:
 doc/_config.yml                                    |  1 +
 doc/admin/cloudtest.html.textile.liquid            | 71 ++++++++++++++++++++++
 .../install-dispatch-cloud.html.textile.liquid     | 29 ++++++++-
 lib/cloud/cloudtest/tester.go                      | 37 +++++++----
 4 files changed, 126 insertions(+), 12 deletions(-)
 create mode 100644 doc/admin/cloudtest.html.textile.liquid

       via  380a54a7d97b34119cbaa3bee05d6b6cd241eee5 (commit)
       via  ade5e27ae2bbf45347097b9eeae2f5bb11c2ae2e (commit)
       via  b13f1f4222fc0a211d9f03d4078353ef11900907 (commit)
       via  4aa3a732cac495fb384a521b2046f6b7f09dd13d (commit)
       via  222e7c73cded78e392a4007aded4d01113149b85 (commit)
      from  506d054ca6417627f8e0d913621ff8774daba4aa (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 380a54a7d97b34119cbaa3bee05d6b6cd241eee5
Merge: 506d054ca ade5e27ae
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Mon Jul 1 10:56:41 2019 -0400

    Merge branch '15026-cloudtest'
    
    refs #15026
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>


commit ade5e27ae2bbf45347097b9eeae2f5bb11c2ae2e
Author: Tom Clegg <tclegg at veritasgenetics.com>
Date:   Mon Jul 1 10:55:26 2019 -0400

    15026: Clean up confusing variable name.
    
    Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tclegg at veritasgenetics.com>

diff --git a/lib/cloud/cloudtest/tester.go b/lib/cloud/cloudtest/tester.go
index e0a6a3597..adc57803f 100644
--- a/lib/cloud/cloudtest/tester.go
+++ b/lib/cloud/cloudtest/tester.go
@@ -342,15 +342,18 @@ func (t *tester) destroyTestInstance() bool {
 		return true
 	}
 	for {
-		t.Logger.WithField("Instance", t.testInstance.ID()).Info("destroying instance")
+		lgr := t.Logger.WithField("Instance", t.testInstance.ID())
+		lgr.Info("destroying instance")
 		t0 := time.Now()
+
 		err := t.testInstance.Destroy()
-		lgr := t.Logger.WithField("Instance", t.testInstance.ID())
-		if lgr := lgr.WithField("Duration", time.Since(t0)); err != nil {
-			lgr.WithError(err).Error("error destroying instance")
+		lgrDur := lgr.WithField("Duration", time.Since(t0))
+		if err != nil {
+			lgrDur.WithError(err).Error("error destroying instance")
 		} else {
-			lgr.Info("destroyed instance")
+			lgrDur.Info("destroyed instance")
 		}
+
 		err = t.refreshTestInstance()
 		if err == errTestInstanceNotFound {
 			lgr.Info("instance no longer appears in list")

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list