20602: Report queue time for pos/neg/zero-priority reqs.

[arvados.git] / lib / service / cmd.go

diff --git a/lib/service/cmd.go b/lib/service/cmd.go
index c66c279432dc2f0ae6d5777f6776a1c2a3d063fb..f4d4ec8efe4b8878d97fafdfc70ba6e364f92689 100644 (file)
--- a/lib/service/cmd.go
+++ b/lib/service/cmd.go
@@ -264,13 +264,17 @@ func (c *command) requestPriority(req *http.Request, queued time.Time) int64 {
                return httpserver.IneligibleForQueuePriority
        case req.Method == http.MethodPost && strings.HasPrefix(req.URL.Path, "/arvados/v1/logs"):
                // "Create log entry" is the most harmless kind of
-               // request to drop.
-               return 0
+               // request to drop. Negative priority is called "low"
+               // in aggregate metrics.
+               return -1
        case req.Header.Get("Origin") != "":
-               // Handle interactive requests first.
-               return 2
-       default:
+               // Handle interactive requests first. Positive
+               // priority is called "high" in aggregate metrics.
                return 1
+       default:
+               // Zero priority is called "normal" in aggregate
+               // metrics.
+               return 0
        }
 }