Arvados

This LGTM, thanks. Just a style nitpick, the "if ok"s should probably be "if !ok { return early }" instead of a pyramid. Could do this:

m, _ := ctx.Value(&mutexContextKey).(sync.Mutex)
c, _ := ctx.Value(&responseLogFieldsContextKey).(logrus.Fields)
if m == nil || c == nil {
  return
}
// normal case

Tom Clegg wrote:

This LGTM, thanks. Just a style nitpick, the "if ok"s should probably be "if !ok { return early }" instead of a pyramid. Could do this:

[...]

Thanks, done, and thanks for the test!

developer-run-tests: #2998