From 2b77da799189ff9bb13a86db683c2b8c2db435af Mon Sep 17 00:00:00 2001 From: Apeksha Bhosale <7846888+ApekshaBhosale@users.noreply.github.com> Date: Thu, 30 Jan 2025 17:40:34 +0530 Subject: [PATCH] chore: added buckets for 2s, 5s, 10s and 20s to be measured on grafana (#38910) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## Description > [!TIP] > _Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team)._ > > _Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR._ Fixes #`Issue Number` _or_ Fixes `Issue URL` > [!WARNING] > _If no issue exists, please create an issue first, and check with the maintainers if the issue is valid._ ## Automation /ok-to-test tags="@tag.All" ### :mag: Cypress test results > [!TIP] > 🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉 > Workflow run: > Commit: d03b480857d925f7e5b0c0375e58cee6c215b796 > Cypress dashboard. > Tags: `@tag.All` > Spec: >
Thu, 30 Jan 2025 10:24:53 UTC ## Communication Should the DevRel and Marketing teams inform users about this change? - [ ] Yes - [ ] No ## Summary by CodeRabbit - **Chores** - Enhanced HTTP server request metrics configuration with more granular size thresholds for improved performance monitoring. --- .../appsmith-server/src/main/resources/application.properties | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/server/appsmith-server/src/main/resources/application.properties b/app/server/appsmith-server/src/main/resources/application.properties index 37a09c3ed519..9a5be44783a5 100644 --- a/app/server/appsmith-server/src/main/resources/application.properties +++ b/app/server/appsmith-server/src/main/resources/application.properties @@ -95,7 +95,7 @@ management.opentelemetry.resource-attributes.service.instance.id=${HOSTNAME:apps management.opentelemetry.resource-attributes.deployment.name=${APPSMITH_DEPLOYMENT_NAME:self-hosted} management.tracing.sampling.probability=${APPSMITH_SAMPLING_PROBABILITY:0.1} management.prometheus.metrics.export.descriptions=true -management.metrics.distribution.slo.http.server.requests=100,200,500,1000,30000 +management.metrics.distribution.slo.http.server.requests=100,200,500,1000,2000,5000,10000,20000,30000 # Observability and Micrometer related configs # Keeping this license key around, until later