[ARVADOS] updated: 2.1.0-112-gf99d4e313

Git user git at public.arvados.org
Fri Dec 4 22:41:23 UTC 2020


Summary of changes:
 lib/controller/federation/conn.go  | 2 +-
 lib/controller/integration_test.go | 4 ++--
 lib/controller/rpc/conn.go         | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

       via  f99d4e313c18bef3eeb2e84abc809452c2af2e69 (commit)
      from  14d8b783fb5940db594f377920c4ecd20ac262a9 (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 f99d4e313c18bef3eeb2e84abc809452c2af2e69
Author: Nico Cesar <nico at nicocesar.com>
Date:   Fri Dec 4 17:40:27 2020 -0500

    user context is needed to pass tests ..
    
    Arvados-DCO-1.1-Signed-off-by: Nico Cesar <nico at curii.com>

diff --git a/lib/controller/federation/conn.go b/lib/controller/federation/conn.go
index 30c15b57c..3ec17c17e 100644
--- a/lib/controller/federation/conn.go
+++ b/lib/controller/federation/conn.go
@@ -343,7 +343,7 @@ func (conn *Conn) ContainerRequestList(ctx context.Context, options arvados.List
 
 func (conn *Conn) ContainerRequestCreate(ctx context.Context, options arvados.CreateOptions) (arvados.ContainerRequest, error) {
 	be := conn.chooseBackend(options.ClusterID)
-	log.Printf("THIS IS THE federation.Conn.ContainerRequestCreate() for %s we are %s", options.ClusterID, conn.cluster.ClusterID)
+	log.Printf("THIS IS federation.Conn.ContainerRequestCreate() for %s we are %s", options.ClusterID, conn.cluster.ClusterID)
 	if be == conn.local {
 		return be.ContainerRequestCreate(ctx, options)
 	}
diff --git a/lib/controller/integration_test.go b/lib/controller/integration_test.go
index e75e996f4..03b760eaa 100644
--- a/lib/controller/integration_test.go
+++ b/lib/controller/integration_test.go
@@ -538,7 +538,7 @@ func (s *IntegrationSuite) TestRuntimeTokenInCR(c *check.C) {
 func (s *IntegrationSuite) TestIntermediateCluster(c *check.C) {
 	conn1 := s.conn("z1111")
 	rootctx1, _, _ := s.rootClients("z1111")
-	_, ac1, _, _ := s.userClients(rootctx1, c, conn1, "z1111", true)
+	uctx1, ac1, _, _ := s.userClients(rootctx1, c, conn1, "z1111", true)
 	//conn2 := s.conn("z2222")
 	//rootctx2, _, _ := s.rootClients("z2222")
 	//_, ac2, _, _ := s.userClients(rootctx2, c, conn2, "z2222", true)
@@ -559,7 +559,7 @@ func (s *IntegrationSuite) TestIntermediateCluster(c *check.C) {
 			"output_path":     "/",
 			"runtime_token":   tt.token,
 		}
-		cr, err := conn1.ContainerRequestCreate(rootctx1, arvados.CreateOptions{ClusterID: "z2222", Attrs: rq})
+		cr, err := conn1.ContainerRequestCreate(uctx1, arvados.CreateOptions{ClusterID: "z2222", Attrs: rq})
 
 		c.Check(err, check.IsNil)
 		c.Check(cr, check.NotNil)
diff --git a/lib/controller/rpc/conn.go b/lib/controller/rpc/conn.go
index b3fd541bb..254f0adfc 100644
--- a/lib/controller/rpc/conn.go
+++ b/lib/controller/rpc/conn.go
@@ -288,7 +288,7 @@ func (conn *Conn) ContainerUnlock(ctx context.Context, options arvados.GetOption
 }
 
 func (conn *Conn) ContainerRequestCreate(ctx context.Context, options arvados.CreateOptions) (arvados.ContainerRequest, error) {
-	log.Printf("THIS IS THE rcp.Conn.ContainerRequestCreate(): %s", options.ClusterID)
+	log.Printf("THIS IS THE rcp.Conn.ContainerRequestCreate() for %s we are %s", options.ClusterID, conn.clusterID)
 	ep := arvados.EndpointContainerRequestCreate
 	var resp arvados.ContainerRequest
 	err := conn.requestAndDecode(ctx, &resp, ep, nil, options)

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list