X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/57074c15587686e066d7e44adec7b6c7154d6ac4..342ec439e7965d889a7aca306475424d3a3ff986:/lib/controller/federation_test.go diff --git a/lib/controller/federation_test.go b/lib/controller/federation_test.go index 8dd8e806de..b94efa6aec 100644 --- a/lib/controller/federation_test.go +++ b/lib/controller/federation_test.go @@ -6,15 +6,20 @@ package controller import ( "encoding/json" + "fmt" + "io/ioutil" + "log" "net/http" "net/http/httptest" "net/url" + "os" "strings" "time" "git.curoverse.com/arvados.git/sdk/go/arvados" "git.curoverse.com/arvados.git/sdk/go/arvadostest" "git.curoverse.com/arvados.git/sdk/go/httpserver" + "git.curoverse.com/arvados.git/sdk/go/keepclient" "github.com/Sirupsen/logrus" check "gopkg.in/check.v1" ) @@ -23,12 +28,18 @@ import ( var _ = check.Suite(&FederationSuite{}) type FederationSuite struct { - log *logrus.Logger - localServer *httpserver.Server - remoteServer *httpserver.Server + log *logrus.Logger + // testServer and testHandler are the controller being tested, + // "zhome". + testServer *httpserver.Server + testHandler *Handler + // remoteServer ("zzzzz") forwards requests to the Rails API + // provided by the integration test environment. + remoteServer *httpserver.Server + // remoteMock ("zmock") appends each incoming request to + // remoteMockRequests, and returns an empty 200 response. remoteMock *httpserver.Server remoteMockRequests []http.Request - handler *Handler } func (s *FederationSuite) SetUpTest(c *check.C) { @@ -47,16 +58,17 @@ func (s *FederationSuite) SetUpTest(c *check.C) { Controller: arvados.SystemServiceInstance{Listen: ":"}, RailsAPI: arvados.SystemServiceInstance{Listen: ":1"}, // local reqs will error "connection refused" } - s.handler = &Handler{Cluster: &arvados.Cluster{ - ClusterID: "zhome", + s.testHandler = &Handler{Cluster: &arvados.Cluster{ + ClusterID: "zhome", + PostgreSQL: integrationTestCluster().PostgreSQL, NodeProfiles: map[string]arvados.NodeProfile{ "*": nodeProfile, }, }, NodeProfile: &nodeProfile} - s.localServer = newServerFromIntegrationTestEnv(c) - s.localServer.Server.Handler = httpserver.AddRequestIDs(httpserver.LogRequests(s.log, s.handler)) + s.testServer = newServerFromIntegrationTestEnv(c) + s.testServer.Server.Handler = httpserver.AddRequestIDs(httpserver.LogRequests(s.log, s.testHandler)) - s.handler.Cluster.RemoteClusters = map[string]arvados.RemoteCluster{ + s.testHandler.Cluster.RemoteClusters = map[string]arvados.RemoteCluster{ "zzzzz": { Host: s.remoteServer.Addr, Proxy: true, @@ -69,7 +81,9 @@ func (s *FederationSuite) SetUpTest(c *check.C) { }, } - c.Assert(s.localServer.Start(), check.IsNil) + c.Assert(s.testServer.Start(), check.IsNil) + + s.remoteMockRequests = nil } func (s *FederationSuite) remoteMockHandler(w http.ResponseWriter, req *http.Request) { @@ -80,131 +94,549 @@ func (s *FederationSuite) TearDownTest(c *check.C) { if s.remoteServer != nil { s.remoteServer.Close() } - if s.localServer != nil { - s.localServer.Close() + if s.testServer != nil { + s.testServer.Close() } } +func (s *FederationSuite) testRequest(req *http.Request) *http.Response { + resp := httptest.NewRecorder() + s.testServer.Server.Handler.ServeHTTP(resp, req) + return resp.Result() +} + func (s *FederationSuite) TestLocalRequest(c *check.C) { req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+strings.Replace(arvadostest.WorkflowWithDefinitionYAMLUUID, "zzzzz-", "zhome-", 1), nil) - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) + resp := s.testRequest(req) s.checkHandledLocally(c, resp) } -func (s *FederationSuite) checkHandledLocally(c *check.C, resp *httptest.ResponseRecorder) { +func (s *FederationSuite) checkHandledLocally(c *check.C, resp *http.Response) { // Our "home" controller can't handle local requests because // it doesn't have its own stub/test Rails API, so we rely on // "connection refused" to indicate the controller tried to // proxy the request to its local Rails API. - c.Check(resp.Code, check.Equals, http.StatusInternalServerError) + c.Check(resp.StatusCode, check.Equals, http.StatusBadGateway) s.checkJSONErrorMatches(c, resp, `.*connection refused`) } func (s *FederationSuite) TestNoAuth(c *check.C) { req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+arvadostest.WorkflowWithDefinitionYAMLUUID, nil) - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) - c.Check(resp.Code, check.Equals, http.StatusUnauthorized) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusUnauthorized) s.checkJSONErrorMatches(c, resp, `Not logged in`) } func (s *FederationSuite) TestBadAuth(c *check.C) { req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+arvadostest.WorkflowWithDefinitionYAMLUUID, nil) req.Header.Set("Authorization", "Bearer aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa") - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) - c.Check(resp.Code, check.Equals, http.StatusUnauthorized) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusUnauthorized) s.checkJSONErrorMatches(c, resp, `Not logged in`) } func (s *FederationSuite) TestNoAccess(c *check.C) { req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+arvadostest.WorkflowWithDefinitionYAMLUUID, nil) req.Header.Set("Authorization", "Bearer "+arvadostest.SpectatorToken) - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) - c.Check(resp.Code, check.Equals, http.StatusNotFound) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) s.checkJSONErrorMatches(c, resp, `.*not found`) } func (s *FederationSuite) TestGetUnknownRemote(c *check.C) { req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+strings.Replace(arvadostest.WorkflowWithDefinitionYAMLUUID, "zzzzz-", "zz404-", 1), nil) req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) - c.Check(resp.Code, check.Equals, http.StatusNotFound) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) s.checkJSONErrorMatches(c, resp, `.*no proxy available for cluster zz404`) } func (s *FederationSuite) TestRemoteError(c *check.C) { - rc := s.handler.Cluster.RemoteClusters["zzzzz"] + rc := s.testHandler.Cluster.RemoteClusters["zzzzz"] rc.Scheme = "https" - s.handler.Cluster.RemoteClusters["zzzzz"] = rc + s.testHandler.Cluster.RemoteClusters["zzzzz"] = rc req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+arvadostest.WorkflowWithDefinitionYAMLUUID, nil) req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) - c.Check(resp.Code, check.Equals, http.StatusInternalServerError) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusBadGateway) s.checkJSONErrorMatches(c, resp, `.*HTTP response to HTTPS client`) } func (s *FederationSuite) TestGetRemoteWorkflow(c *check.C) { req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+arvadostest.WorkflowWithDefinitionYAMLUUID, nil) req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) - c.Check(resp.Code, check.Equals, http.StatusOK) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) var wf arvados.Workflow - c.Check(json.Unmarshal(resp.Body.Bytes(), &wf), check.IsNil) + c.Check(json.NewDecoder(resp.Body).Decode(&wf), check.IsNil) c.Check(wf.UUID, check.Equals, arvadostest.WorkflowWithDefinitionYAMLUUID) c.Check(wf.OwnerUUID, check.Equals, arvadostest.ActiveUserUUID) } +func (s *FederationSuite) TestOptionsMethod(c *check.C) { + req := httptest.NewRequest("OPTIONS", "/arvados/v1/workflows/"+arvadostest.WorkflowWithDefinitionYAMLUUID, nil) + req.Header.Set("Origin", "https://example.com") + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + body, err := ioutil.ReadAll(resp.Body) + c.Check(err, check.IsNil) + c.Check(string(body), check.Equals, "") + c.Check(resp.Header.Get("Access-Control-Allow-Origin"), check.Equals, "*") + for _, hdr := range []string{"Authorization", "Content-Type"} { + c.Check(resp.Header.Get("Access-Control-Allow-Headers"), check.Matches, ".*"+hdr+".*") + } + for _, method := range []string{"GET", "HEAD", "PUT", "POST", "DELETE"} { + c.Check(resp.Header.Get("Access-Control-Allow-Methods"), check.Matches, ".*"+method+".*") + } +} + func (s *FederationSuite) TestRemoteWithTokenInQuery(c *check.C) { req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+strings.Replace(arvadostest.WorkflowWithDefinitionYAMLUUID, "zzzzz-", "zmock-", 1)+"?api_token="+arvadostest.ActiveToken, nil) - s.handler.ServeHTTP(httptest.NewRecorder(), req) + s.testRequest(req) c.Assert(len(s.remoteMockRequests), check.Equals, 1) - c.Check(s.remoteMockRequests[0].URL.String(), check.Not(check.Matches), `.*api_token=.*`) + pr := s.remoteMockRequests[0] + // Token is salted and moved from query to Authorization header. + c.Check(pr.URL.String(), check.Not(check.Matches), `.*api_token=.*`) + c.Check(pr.Header.Get("Authorization"), check.Equals, "Bearer v2/zzzzz-gj3su-077z32aux8dg2s1/7fd31b61f39c0e82a4155592163218272cedacdc") } -func (s *FederationSuite) TestUpdateRemoteWorkflow(c *check.C) { - updateDescription := func(descr string) *httptest.ResponseRecorder { - req := httptest.NewRequest("PATCH", "/arvados/v1/workflows/"+arvadostest.WorkflowWithDefinitionYAMLUUID, strings.NewReader(url.Values{ - "workflow": {`{"description":"` + descr + `"}`}, +func (s *FederationSuite) TestLocalTokenSalted(c *check.C) { + req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+strings.Replace(arvadostest.WorkflowWithDefinitionYAMLUUID, "zzzzz-", "zmock-", 1), nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + s.testRequest(req) + c.Assert(len(s.remoteMockRequests), check.Equals, 1) + pr := s.remoteMockRequests[0] + // The salted token here has a "zzzzz-" UUID instead of a + // "ztest-" UUID because ztest's local database has the + // "zzzzz-" test fixtures. The "secret" part is HMAC(sha1, + // arvadostest.ActiveToken, "zmock") = "7fd3...". + c.Check(pr.Header.Get("Authorization"), check.Equals, "Bearer v2/zzzzz-gj3su-077z32aux8dg2s1/7fd31b61f39c0e82a4155592163218272cedacdc") +} + +func (s *FederationSuite) TestRemoteTokenNotSalted(c *check.C) { + // remoteToken can be any v1 token that doesn't appear in + // ztest's local db. + remoteToken := "abcdef00000000000000000000000000000000000000000000" + req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+strings.Replace(arvadostest.WorkflowWithDefinitionYAMLUUID, "zzzzz-", "zmock-", 1), nil) + req.Header.Set("Authorization", "Bearer "+remoteToken) + s.testRequest(req) + c.Assert(len(s.remoteMockRequests), check.Equals, 1) + pr := s.remoteMockRequests[0] + c.Check(pr.Header.Get("Authorization"), check.Equals, "Bearer "+remoteToken) +} + +func (s *FederationSuite) TestWorkflowCRUD(c *check.C) { + wf := arvados.Workflow{ + Description: "TestCRUD", + } + { + body := &strings.Builder{} + json.NewEncoder(body).Encode(&wf) + req := httptest.NewRequest("POST", "/arvados/v1/workflows", strings.NewReader(url.Values{ + "workflow": {body.String()}, }.Encode())) req.Header.Set("Content-type", "application/x-www-form-urlencoded") req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) - resp := httptest.NewRecorder() - s.handler.ServeHTTP(resp, req) + rec := httptest.NewRecorder() + s.remoteServer.Server.Handler.ServeHTTP(rec, req) // direct to remote -- can't proxy a create req because no uuid + resp := rec.Result() s.checkResponseOK(c, resp) - return resp - } + json.NewDecoder(resp.Body).Decode(&wf) - // Update description twice so running this test twice in a - // row still causes ModifiedAt to change - updateDescription("updated once by TestUpdateRemoteWorkflow") - resp := updateDescription("updated twice by TestUpdateRemoteWorkflow") + defer func() { + req := httptest.NewRequest("DELETE", "/arvados/v1/workflows/"+wf.UUID, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + s.remoteServer.Server.Handler.ServeHTTP(httptest.NewRecorder(), req) + }() + c.Check(wf.UUID, check.Not(check.Equals), "") - var wf arvados.Workflow - c.Check(json.Unmarshal(resp.Body.Bytes(), &wf), check.IsNil) - c.Check(wf.UUID, check.Equals, arvadostest.WorkflowWithDefinitionYAMLUUID) - c.Assert(wf.ModifiedAt, check.NotNil) - c.Logf("%s", *wf.ModifiedAt) - c.Check(time.Since(*wf.ModifiedAt) < time.Minute, check.Equals, true) + c.Assert(wf.ModifiedAt, check.NotNil) + c.Logf("wf.ModifiedAt: %v", wf.ModifiedAt) + c.Check(time.Since(*wf.ModifiedAt) < time.Minute, check.Equals, true) + } + for _, method := range []string{"PATCH", "PUT", "POST"} { + form := url.Values{ + "workflow": {`{"description": "Updated with ` + method + `"}`}, + } + if method == "POST" { + form["_method"] = []string{"PATCH"} + } + req := httptest.NewRequest(method, "/arvados/v1/workflows/"+wf.UUID, strings.NewReader(form.Encode())) + req.Header.Set("Content-type", "application/x-www-form-urlencoded") + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + s.checkResponseOK(c, resp) + err := json.NewDecoder(resp.Body).Decode(&wf) + c.Check(err, check.IsNil) + + c.Check(wf.Description, check.Equals, "Updated with "+method) + } + { + req := httptest.NewRequest("DELETE", "/arvados/v1/workflows/"+wf.UUID, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + s.checkResponseOK(c, resp) + err := json.NewDecoder(resp.Body).Decode(&wf) + c.Check(err, check.IsNil) + } + { + req := httptest.NewRequest("GET", "/arvados/v1/workflows/"+wf.UUID, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) + } } -func (s *FederationSuite) checkResponseOK(c *check.C, resp *httptest.ResponseRecorder) { - c.Check(resp.Code, check.Equals, http.StatusOK) - if resp.Code != http.StatusOK { - c.Logf("... response body = %s\n", resp.Body.String()) +func (s *FederationSuite) checkResponseOK(c *check.C, resp *http.Response) { + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + if resp.StatusCode != http.StatusOK { + body, err := ioutil.ReadAll(resp.Body) + c.Logf("... response body = %q, %v\n", body, err) } } -func (s *FederationSuite) checkJSONErrorMatches(c *check.C, resp *httptest.ResponseRecorder, re string) { +func (s *FederationSuite) checkJSONErrorMatches(c *check.C, resp *http.Response, re string) { var jresp httpserver.ErrorResponse - err := json.Unmarshal(resp.Body.Bytes(), &jresp) + err := json.NewDecoder(resp.Body).Decode(&jresp) c.Check(err, check.IsNil) c.Assert(len(jresp.Errors), check.Equals, 1) c.Check(jresp.Errors[0], check.Matches, re) } + +func (s *FederationSuite) localServiceReturns404(c *check.C) *httpserver.Server { + srv := &httpserver.Server{ + Server: http.Server{ + Handler: http.HandlerFunc(func(w http.ResponseWriter, req *http.Request) { + w.WriteHeader(404) + }), + }, + } + + c.Assert(srv.Start(), check.IsNil) + + np := arvados.NodeProfile{ + Controller: arvados.SystemServiceInstance{Listen: ":"}, + RailsAPI: arvados.SystemServiceInstance{Listen: srv.Addr, + TLS: false, Insecure: true}} + s.testHandler.Cluster.NodeProfiles["*"] = np + s.testHandler.NodeProfile = &np + + return srv +} + +func (s *FederationSuite) TestGetLocalCollection(c *check.C) { + np := arvados.NodeProfile{ + Controller: arvados.SystemServiceInstance{Listen: ":"}, + RailsAPI: arvados.SystemServiceInstance{Listen: os.Getenv("ARVADOS_TEST_API_HOST"), + TLS: true, Insecure: true}} + s.testHandler.Cluster.ClusterID = "zzzzz" + s.testHandler.Cluster.NodeProfiles["*"] = np + s.testHandler.NodeProfile = &np + + req := httptest.NewRequest("GET", "/arvados/v1/collections/"+arvadostest.UserAgreementCollection, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var col arvados.Collection + c.Check(json.NewDecoder(resp.Body).Decode(&col), check.IsNil) + c.Check(col.UUID, check.Equals, arvadostest.UserAgreementCollection) + c.Check(col.ManifestText, check.Matches, + `\. 6a4ff0499484c6c79c95cd8c566bd25f\+249025\+A[0-9a-f]{40}@[0-9a-f]{8} 0:249025:GNU_General_Public_License,_version_3.pdf +`) +} + +func (s *FederationSuite) TestGetRemoteCollection(c *check.C) { + defer s.localServiceReturns404(c).Close() + + req := httptest.NewRequest("GET", "/arvados/v1/collections/"+arvadostest.UserAgreementCollection, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var col arvados.Collection + c.Check(json.NewDecoder(resp.Body).Decode(&col), check.IsNil) + c.Check(col.UUID, check.Equals, arvadostest.UserAgreementCollection) + c.Check(col.ManifestText, check.Matches, + `\. 6a4ff0499484c6c79c95cd8c566bd25f\+249025\+Rzzzzz-[0-9a-f]{40}@[0-9a-f]{8} 0:249025:GNU_General_Public_License,_version_3.pdf +`) +} + +func (s *FederationSuite) TestGetRemoteCollectionError(c *check.C) { + defer s.localServiceReturns404(c).Close() + + req := httptest.NewRequest("GET", "/arvados/v1/collections/zzzzz-4zz18-fakefakefakefak", nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) +} + +func (s *FederationSuite) TestSignedLocatorPattern(c *check.C) { + // Confirm the regular expression identifies other groups of hints correctly + c.Check(keepclient.SignedLocatorRe.FindStringSubmatch(`6a4ff0499484c6c79c95cd8c566bd25f+249025+B1+C2+A05227438989d04712ea9ca1c91b556cef01d5cc7@5ba5405b+D3+E4`), + check.DeepEquals, + []string{"6a4ff0499484c6c79c95cd8c566bd25f+249025+B1+C2+A05227438989d04712ea9ca1c91b556cef01d5cc7@5ba5405b+D3+E4", + "6a4ff0499484c6c79c95cd8c566bd25f", + "+249025", + "+B1+C2", "+C2", + "+A05227438989d04712ea9ca1c91b556cef01d5cc7@5ba5405b", + "05227438989d04712ea9ca1c91b556cef01d5cc7", "5ba5405b", + "+D3+E4", "+E4"}) +} + +func (s *FederationSuite) TestGetLocalCollectionByPDH(c *check.C) { + np := arvados.NodeProfile{ + Controller: arvados.SystemServiceInstance{Listen: ":"}, + RailsAPI: arvados.SystemServiceInstance{Listen: os.Getenv("ARVADOS_TEST_API_HOST"), + TLS: true, Insecure: true}} + s.testHandler.Cluster.NodeProfiles["*"] = np + s.testHandler.NodeProfile = &np + + req := httptest.NewRequest("GET", "/arvados/v1/collections/"+arvadostest.UserAgreementPDH, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var col arvados.Collection + c.Check(json.NewDecoder(resp.Body).Decode(&col), check.IsNil) + c.Check(col.PortableDataHash, check.Equals, arvadostest.UserAgreementPDH) + c.Check(col.ManifestText, check.Matches, + `\. 6a4ff0499484c6c79c95cd8c566bd25f\+249025\+A[0-9a-f]{40}@[0-9a-f]{8} 0:249025:GNU_General_Public_License,_version_3.pdf +`) +} + +func (s *FederationSuite) TestGetRemoteCollectionByPDH(c *check.C) { + defer s.localServiceReturns404(c).Close() + + req := httptest.NewRequest("GET", "/arvados/v1/collections/"+arvadostest.UserAgreementPDH, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + + var col arvados.Collection + c.Check(json.NewDecoder(resp.Body).Decode(&col), check.IsNil) + c.Check(col.PortableDataHash, check.Equals, arvadostest.UserAgreementPDH) + c.Check(col.ManifestText, check.Matches, + `\. 6a4ff0499484c6c79c95cd8c566bd25f\+249025\+Rzzzzz-[0-9a-f]{40}@[0-9a-f]{8} 0:249025:GNU_General_Public_License,_version_3.pdf +`) +} + +func (s *FederationSuite) TestGetCollectionByPDHError(c *check.C) { + defer s.localServiceReturns404(c).Close() + + req := httptest.NewRequest("GET", "/arvados/v1/collections/99999999999999999999999999999999+99", nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + + resp := s.testRequest(req) + defer resp.Body.Close() + + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) +} + +func (s *FederationSuite) TestGetCollectionByPDHErrorBadHash(c *check.C) { + defer s.localServiceReturns404(c).Close() + + srv2 := &httpserver.Server{ + Server: http.Server{ + Handler: http.HandlerFunc(func(w http.ResponseWriter, req *http.Request) { + w.WriteHeader(200) + // Return a collection where the hash + // of the manifest text doesn't match + // PDH that was requested. + var col arvados.Collection + col.PortableDataHash = "99999999999999999999999999999999+99" + col.ManifestText = `. 6a4ff0499484c6c79c95cd8c566bd25f\+249025 0:249025:GNU_General_Public_License,_version_3.pdf +` + enc := json.NewEncoder(w) + enc.Encode(col) + }), + }, + } + + c.Assert(srv2.Start(), check.IsNil) + defer srv2.Close() + + // Direct zzzzz to service that returns a 200 result with a bogus manifest_text + s.testHandler.Cluster.RemoteClusters["zzzzz"] = arvados.RemoteCluster{ + Host: srv2.Addr, + Proxy: true, + Scheme: "http", + } + + req := httptest.NewRequest("GET", "/arvados/v1/collections/99999999999999999999999999999999+99", nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + + resp := s.testRequest(req) + defer resp.Body.Close() + + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) +} + +func (s *FederationSuite) TestSaltedTokenGetCollectionByPDH(c *check.C) { + np := arvados.NodeProfile{ + Controller: arvados.SystemServiceInstance{Listen: ":"}, + RailsAPI: arvados.SystemServiceInstance{Listen: os.Getenv("ARVADOS_TEST_API_HOST"), + TLS: true, Insecure: true}} + s.testHandler.Cluster.NodeProfiles["*"] = np + s.testHandler.NodeProfile = &np + + req := httptest.NewRequest("GET", "/arvados/v1/collections/"+arvadostest.UserAgreementPDH, nil) + req.Header.Set("Authorization", "Bearer v2/zzzzz-gj3su-077z32aux8dg2s1/282d7d172b6cfdce364c5ed12ddf7417b2d00065") + resp := s.testRequest(req) + + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var col arvados.Collection + c.Check(json.NewDecoder(resp.Body).Decode(&col), check.IsNil) + c.Check(col.PortableDataHash, check.Equals, arvadostest.UserAgreementPDH) + c.Check(col.ManifestText, check.Matches, + `\. 6a4ff0499484c6c79c95cd8c566bd25f\+249025\+A[0-9a-f]{40}@[0-9a-f]{8} 0:249025:GNU_General_Public_License,_version_3.pdf +`) +} + +func (s *FederationSuite) TestSaltedTokenGetCollectionByPDHError(c *check.C) { + np := arvados.NodeProfile{ + Controller: arvados.SystemServiceInstance{Listen: ":"}, + RailsAPI: arvados.SystemServiceInstance{Listen: os.Getenv("ARVADOS_TEST_API_HOST"), + TLS: true, Insecure: true}} + s.testHandler.Cluster.NodeProfiles["*"] = np + s.testHandler.NodeProfile = &np + + req := httptest.NewRequest("GET", "/arvados/v1/collections/99999999999999999999999999999999+99", nil) + req.Header.Set("Authorization", "Bearer v2/zzzzz-gj3su-077z32aux8dg2s1/282d7d172b6cfdce364c5ed12ddf7417b2d00065") + resp := s.testRequest(req) + + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) +} + +func (s *FederationSuite) TestGetRemoteContainerRequest(c *check.C) { + defer s.localServiceReturns404(c).Close() + req := httptest.NewRequest("GET", "/arvados/v1/container_requests/"+arvadostest.QueuedContainerRequestUUID, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var cr arvados.ContainerRequest + c.Check(json.NewDecoder(resp.Body).Decode(&cr), check.IsNil) + c.Check(cr.UUID, check.Equals, arvadostest.QueuedContainerRequestUUID) + c.Check(cr.Priority, check.Equals, 1) +} + +func (s *FederationSuite) TestUpdateRemoteContainerRequest(c *check.C) { + defer s.localServiceReturns404(c).Close() + req := httptest.NewRequest("PATCH", "/arvados/v1/container_requests/"+arvadostest.QueuedContainerRequestUUID, + strings.NewReader(`{"container_request": {"priority": 696}}`)) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + req.Header.Set("Content-type", "application/json") + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var cr arvados.ContainerRequest + c.Check(json.NewDecoder(resp.Body).Decode(&cr), check.IsNil) + c.Check(cr.UUID, check.Equals, arvadostest.QueuedContainerRequestUUID) + c.Check(cr.Priority, check.Equals, 696) +} + +func (s *FederationSuite) TestCreateRemoteContainerRequest1(c *check.C) { + defer s.localServiceReturns404(c).Close() + req := httptest.NewRequest("POST", "/arvados/v1/container_requests", + strings.NewReader(`{ + "cluster_id": "zzzzz", + "container_request": { + "name": "hello world", + "state": "Uncommitted", + "output_path": "/", + "container_image": "123", + "command": ["abc"] + } +} +`)) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + req.Header.Set("Content-type", "application/json") + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var cr arvados.ContainerRequest + c.Check(json.NewDecoder(resp.Body).Decode(&cr), check.IsNil) + c.Check(cr.Name, check.Equals, "hello world") + c.Check(strings.HasPrefix(cr.UUID, "zzzzz-"), check.Equals, true) +} + +func (s *FederationSuite) TestCreateRemoteContainerRequest2(c *check.C) { + defer s.localServiceReturns404(c).Close() + // pass cluster_id via query parameter, this allows arvados-controller + // to avoid parsing the body + req := httptest.NewRequest("POST", "/arvados/v1/container_requests?cluster_id=zzzzz", + strings.NewReader(`{ + "container_request": { + "name": "hello world", + "state": "Uncommitted", + "output_path": "/", + "container_image": "123", + "command": ["abc"] + } +} +`)) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + req.Header.Set("Content-type", "application/json") + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var cr arvados.ContainerRequest + c.Check(json.NewDecoder(resp.Body).Decode(&cr), check.IsNil) + c.Check(cr.Name, check.Equals, "hello world") + c.Check(strings.HasPrefix(cr.UUID, "zzzzz-"), check.Equals, true) +} + +func (s *FederationSuite) TestCreateRemoteContainerRequestError(c *check.C) { + defer s.localServiceReturns404(c).Close() + // pass cluster_id via query parameter, this allows arvados-controller + // to avoid parsing the body + req := httptest.NewRequest("POST", "/arvados/v1/container_requests?cluster_id=zz404", + strings.NewReader(`{ + "container_request": { + "name": "hello world", + "state": "Uncommitted", + "output_path": "/", + "container_image": "123", + "command": ["abc"] + } +} +`)) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + req.Header.Set("Content-type", "application/json") + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusNotFound) +} + +func (s *FederationSuite) TestGetRemoteContainer(c *check.C) { + defer s.localServiceReturns404(c).Close() + req := httptest.NewRequest("GET", "/arvados/v1/containers/"+arvadostest.QueuedContainerUUID, nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var cn arvados.Container + c.Check(json.NewDecoder(resp.Body).Decode(&cn), check.IsNil) + c.Check(cn.UUID, check.Equals, arvadostest.QueuedContainerUUID) +} + +func (s *FederationSuite) TestListRemoteContainer(c *check.C) { + defer s.localServiceReturns404(c).Close() + req := httptest.NewRequest("GET", "/arvados/v1/containers?filters="+ + url.QueryEscape(fmt.Sprintf(`[["uuid", "in", ["%v"]]]`, arvadostest.QueuedContainerUUID)), nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + c.Check(resp.StatusCode, check.Equals, http.StatusOK) + var cn arvados.ContainerList + c.Check(json.NewDecoder(resp.Body).Decode(&cn), check.IsNil) + c.Check(cn.Items[0].UUID, check.Equals, arvadostest.QueuedContainerUUID) +} + +func (s *FederationSuite) TestListMultiRemoteContainers(c *check.C) { + defer s.localServiceReturns404(c).Close() + req := httptest.NewRequest("GET", "/arvados/v1/containers?filters="+ + url.QueryEscape(fmt.Sprintf(`[["uuid", "in", ["%v", "zhome-xvhdp-cr5queuedcontnr"]]]`, arvadostest.QueuedContainerUUID)), nil) + req.Header.Set("Authorization", "Bearer "+arvadostest.ActiveToken) + resp := s.testRequest(req) + log.Printf("got %+v", resp) + c.Assert(resp.StatusCode, check.Equals, http.StatusOK) + var cn arvados.ContainerList + c.Check(json.NewDecoder(resp.Body).Decode(&cn), check.IsNil) + c.Check(cn.Items[0].UUID, check.Equals, arvadostest.QueuedContainerUUID) +}