Bug #14475

[Controller] POST .../collections/$uuid seems to have broken Workbench integration tests

Added by Tom Clegg about 1 year ago. Updated about 1 year ago.

Status:
Resolved
Priority:
Normal
Assigned To:
-
Category:
-
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
Story points:
-
Release:
Release relationship:
Auto

Description

Success: Failure:

Diff: https://dev.arvados.org/projects/arvados/repository/revisions/7c2aa951ed9e7d0010fd58d59dc1e98c9d5e2800/diff


Related issues

Related to Arvados - Feature #14198: [CWL] run steps on remote clustersResolved11/01/2018

Associated revisions

Revision f731a77a
Added by Tom Clegg about 1 year ago

Merge branch '14475-update-collection-error'

fixes #14475

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <>

History

#1 Updated by Tom Clegg about 1 year ago

#2 Updated by Tom Clegg about 1 year ago

  • Description updated (diff)

#3 Updated by Tom Clegg about 1 year ago

{"RequestID":"req-1rup84b3vrwu1uvt85xl","level":"info","msg":"request","remoteAddr":"127.0.0.1:59966","reqBytes":11,"reqForwardedFor":"127.0.0.1","reqHost":"0.0.0.0:60187","reqMethod":"POST","reqPath":"arvados/v1/collections/zzzzz-4zz18-js48y3ykkfdfjd3","reqQuery":"","time":"2018-11-13T15:21:24.464064896-05:00"}

...

&http.Request{Method:"POST", URL:(*url.URL)(0xc42013b400), Proto:"HTTP/1.0", ProtoMajor:1, ProtoMinor:0, Header:http.Header{"User-Agent":[]string{"Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Firefox/60.0"}, "Accept-Language":[]string{"en-US,en;q=0.5"}, "Origin":[]string{"http://127.0.0.1:39121"}, "X-Request-Id":[]string{"req-1rup84b3vrwu1uvt85xl"}, "X-Forwarded-For":[]string{"127.0.0.1"}, "X-Forwarded-Proto":[]string{"https"}, "Connection":[]string{"close"}, "Content-Type":[]string{"application/x-www-form-urlencoded; charset=UTF-8"}, "Authorization":[]string{"OAuth2 3kg6k6lzmp9kj5cpkcoxie963cmvjahbt2fod9zru30k1jqdmi"}, "Content-Length":[]string{"11"}, "Accept":[]string{"application/json, text/javascript, */*; q=0.01"}, "Accept-Encoding":[]string{"gzip, deflate, br"}, "Referer":[]string{"http://127.0.0.1:39121/collections/zzzzz-4zz18-js48y3ykkfdfjd3?"}}, Body:(*http.body)(0xc420174b80), GetBody:(func() (io.ReadCloser, error))(nil), ContentLength:11, TransferEncoding:[]string(nil), Close:true, Host:"0.0.0.0:60187", Form:url.Values{"_method":[]string{"GET"}}, PostForm:url.Values{"_method":[]string{"GET"}}, MultipartForm:(*multipart.Form)(nil), Trailer:http.Header(nil), RemoteAddr:"127.0.0.1:59966", RequestURI:"/arvados/v1/collections/zzzzz-4zz18-js48y3ykkfdfjd3", TLS:(*tls.ConnectionState)(nil), Cancel:(<-chan struct {})(nil), Response:(*http.Response)(nil), ctx:(*context.timerCtx)(0xc4201798c0)}

...

[req-1rup84b3vrwu1uvt85xl] #<ActionController::RoutingError: Path not found>
[req-1rup84b3vrwu1uvt85xl] Error 1542140484+9f504c2e: 404
{"method":"POST","path":"/arvados/v1/collections/zzzzz-4zz18-js48y3ykkfdfjd3","format":"html","controller":"StaticController","action":"render_not_found","status":404,"duration":2.45,"view":0.71,"db":0.0,"request_id":"req-1rup84b3vrwu1uvt85xl","client_ipaddr":"127.0.0.1","client_auth":"zzzzz-gj3su-077z32aux8dg2s1","params":{"a":"arvados/v1/collections/zzzzz-4zz18-js48y3ykkfdfjd3"},"@timestamp":"2018-11-13T20:21:24.491520776Z","@version":"1","message":"[404] POST /arvados/v1/collections/zzzzz-4zz18-js48y3ykkfdfjd3 (StaticController#render_not_found)"}

...

{"RequestID":"req-1rup84b3vrwu1uvt85xl","level":"info","msg":"response","remoteAddr":"127.0.0.1:59966","reqBytes":11,"reqForwardedFor":"127.0.0.1","reqHost":"0.0.0.0:60187","reqMethod":"POST","reqPath":"arvados/v1/collections/zzzzz-4zz18-js48y3ykkfdfjd3","reqQuery":"","respBytes":65,"respStatus":"Not Found","respStatusCode":404,"time":"2018-11-13T15:21:24.495950254-05:00","timeToStatus":0.031859,"timeTotal":0.031935,"timeWriteBody":0.000076}

#4 Updated by Tom Clegg about 1 year ago

14475-update-collection-error @ 2463e56150d0d10997f6b11ab0828c4b23d575df

See commit message.

#5 Updated by Tom Clegg about 1 year ago

  • Status changed from New to In Progress

#6 Updated by Lucas Di Pentima about 1 year ago

2463e56150d0d10997f6b11ab0828c4b23d575df LGTM, but I saw an error message related to the run-tests.sh update on 66d07bb91f91b0cd4c92c7ffa913f7181a3d4942: is git head a real command or a typo? version 2.14.1 doesn't recognize it.

#7 Updated by Tom Clegg about 1 year ago

Oops, "git head" is my .gitconfig alias for "log -n1 --format=%H". Fixed.

#8 Updated by Tom Clegg about 1 year ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

#9 Updated by Tom Morris about 1 year ago

  • Release set to 14

Also available in: Atom PDF