6 Commits (e19c5863eaaeb96d4de20a5a2e93061ec7e74c6f)

Author	SHA1	Message	Date
Owen Diehl	8c348614e3	Querier worker/inflight metrics (#5124) ... * refactors querier worker metrics to use metrics struct * metrics and logs for querier worker execution * loki_querier_worker_concurrency metric * alter log line * wires up prometheus.registerer args to prevent duplicate registration errors on tests	3 years ago
Christian Haudum	7f54641166	Set correct `Content-Type` header in query response (#4828) ... * Set correct `Content-Type` header in query response Because the internal query request handlers did not set the response content type explicitly to `application/json`, and the external request handler takes the response content type from the internal response to set it for the resulting HTTP response, that response was also incorrectly returning `Content-Type: text/plain`. Setting the content type in the internal response correctly results in the expected `Content-Type` header `application/json; charset=UTF-8`. Fixes #4791 Signed-off-by: Christian Haudum <christian.haudum@gmail.com> * fixup! Set correct `Content-Type` header in query response Signed-off-by: Christian Haudum <christian.haudum@gmail.com> * fixup! fixup! Set correct `Content-Type` header in query response Signed-off-by: Christian Haudum <christian.haudum@gmail.com>	4 years ago
Ed Welch	88ef940ede	Loki: Set querier worker max concurrent regardless of run configuration. (#4761) ... * always set the querier worker MaxConcurrentRequests * removing a test which doesn't apply as this setting isn't being set in worker_service.go anymore	4 years ago
Ed Welch	89ee022c21	Loki: fix handling of tail requests when using target `all` or `read` (#4642) ... * better handling of tail requests when the frontend and querier are running in the same process * adding function to determine if a module is enable based on the target supplied to Loki * fix tests and incorrect registering of tail routes * renamed function and made check for active recursive	4 years ago
Ed Welch	2b5f3000b6	Loki: Add a ring to the query scheduler to allow discovery via the ring as an alternative to DNS (#4424) ... * Fork the frontend and scheduler so we can add support for discovery via the ring. * Add a ring to the query scheduler which is then used by the frontend worker and querier workers to find the scheduler address as an alternative to using DNS * update some of the forked code to use dskit grpcclient and grpcutil since #4312 was merged after the original fork was created. * remove query scheduler dependency from querier Signed-off-by: Trevor Whitney <trevorjwhitney@gmail.com> * migrate logging package and make linter happy Signed-off-by: Trevor Whitney <trevorjwhitney@gmail.com> * add SafeReadRing for instances when scheduler is not enabled on same instance * Doc changes from code review A few doc fixes Co-authored-by: Karen Miller <84039272+KMiller-Grafana@users.noreply.github.com> * go mod tidy on go-kit dep * update changelog * remove ReadRing in favor of SafeReadRing Co-authored-by: Trevor Whitney <trevorjwhitney@gmail.com> Co-authored-by: Karen Miller <84039272+KMiller-Grafana@users.noreply.github.com>	4 years ago
Trevor Whitney	7208caf2b9	Feature: allow querier and query frontend targets to run on same process (#4301) ... * allow querier and query frontend to both the registered in same process Signed-off-by: Trevor Whitney <trevorjwhitney@gmail.com> * rename QuerierWorkerService to WorkerService Signed-off-by: Trevor Whitney <trevorjwhitney@gmail.com> * add debug log for standalone evaluation Signed-off-by: Trevor Whitney <trevorjwhitney@gmail.com> * inline generation of querier.WorkerServiceConfig Co-authored-by: Ed Welch <edward.welch@grafana.com> * Clarify auth middleware test, inline config gen * Fix typo Signed-off-by: Trevor Whitney <trevorjwhitney@gmail.com> * reconcile go mod with HEAD of main Signed-off-by: Trevor Whitney <trevorjwhitney@gmail.com> Co-authored-by: Ed Welch <edward.welch@grafana.com>	4 years ago