[ARVADOS] updated: 1.3.0-3102-g49a89ce98
Git user
git at public.arvados.org
Fri Sep 4 15:25:57 UTC 2020
Summary of changes:
lib/controller/federation/conn.go | 36 ++++++++++++++++--
lib/controller/integration_test.go | 77 ++++++++++++++++++++++++++++++++++----
2 files changed, 103 insertions(+), 10 deletions(-)
via 49a89ce984eb69ef4316882e91dec652dc353e39 (commit)
via 3ee2186cb06b822f113696ccb24a78b79269d318 (commit)
via 47aa52f1b343c93e09908b69d40bf8b389e8b15c (commit)
via 51d7a5b2a23074a130aa6dd74cbaf5f335920769 (commit)
from 472fff42d6105a4457deeb1579e9d14caffc82dc (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 49a89ce984eb69ef4316882e91dec652dc353e39
Merge: 472fff42d 3ee2186cb
Author: Peter Amstutz <peter.amstutz at curii.com>
Date: Fri Sep 4 11:25:42 2020 -0400
Merge branch '16778-setup-fed-user' refs #16778
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>
commit 3ee2186cb06b822f113696ccb24a78b79269d318
Author: Peter Amstutz <peter.amstutz at curii.com>
Date: Fri Sep 4 11:25:15 2020 -0400
16778: Make a copy of options
Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>
diff --git a/lib/controller/federation/conn.go b/lib/controller/federation/conn.go
index a84f1b017..d715734c6 100644
--- a/lib/controller/federation/conn.go
+++ b/lib/controller/federation/conn.go
@@ -486,20 +486,14 @@ func (conn *Conn) UserSetup(ctx context.Context, options arvados.UserSetupOption
// it should be created on the local cluster,
// not the remote one.
- setupVM := options.VMUUID
- setupRepo := options.RepoName
- options.VMUUID = ""
- options.RepoName = ""
+ upstreamOptions := options
+ upstreamOptions.VMUUID = ""
+ upstreamOptions.RepoName = ""
- ret, err := upstream.UserSetup(ctx, options)
+ ret, err := upstream.UserSetup(ctx, upstreamOptions)
if err != nil {
return ret, err
}
-
- // Restore VMUUID and RepoName for the call to local
- // UserSetup below.
- options.VMUUID = setupVM
- options.RepoName = setupRepo
}
return conn.local.UserSetup(ctx, options)
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list