From ac9c3f7c79c3c1f6ad34562fb92d810a6cbec0f5 Mon Sep 17 00:00:00 2001 From: gabemontero Date: Fri, 12 Jul 2024 13:11:18 -0400 Subject: [PATCH 1/2] turn on pprof in core controller to allow for goroutine dumps per the golang recommendations that pprof enablement is OK in production, we are enabling to allow for gathering goroutine dumps as part of diagnosing deadlocks. rh-pre-commit.version: 2.3.0 rh-pre-commit.check-secrets: ENABLED --- .../openshift-pipelines/tekton-config.yaml | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/operator/gitops/argocd/pipeline-service/openshift-pipelines/tekton-config.yaml b/operator/gitops/argocd/pipeline-service/openshift-pipelines/tekton-config.yaml index 3c1915ab9..229fb91ec 100644 --- a/operator/gitops/argocd/pipeline-service/openshift-pipelines/tekton-config.yaml +++ b/operator/gitops/argocd/pipeline-service/openshift-pipelines/tekton-config.yaml @@ -87,6 +87,14 @@ spec: options: disabled: false configMaps: + config-observability: + data: + metrics.count.enable-reason: 'false' + metrics.pipelinerun.duration-type: histogram + metrics.pipelinerun.level: pipeline + metrics.taskrun.duration-type: histogram + metrics.taskrun.level: task + profiling.enable: "true" config-logging: data: loglevel.controller: info From a9bf913042e8e91bcb16e0cb77c51977b2cd2f99 Mon Sep 17 00:00:00 2001 From: gabemontero Date: Mon, 15 Jul 2024 09:53:17 -0400 Subject: [PATCH 2/2] enable watcher pprof rh-pre-commit.version: 2.3.0 rh-pre-commit.check-secrets: ENABLED --- .../argocd/pipeline-service/tekton-results/kustomization.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/operator/gitops/argocd/pipeline-service/tekton-results/kustomization.yaml b/operator/gitops/argocd/pipeline-service/tekton-results/kustomization.yaml index 885e0d97d..9042efec4 100644 --- a/operator/gitops/argocd/pipeline-service/tekton-results/kustomization.yaml +++ b/operator/gitops/argocd/pipeline-service/tekton-results/kustomization.yaml @@ -26,6 +26,10 @@ configMapGenerator: name: api-config options: disableNameSuffixHash: true + - behavior: merge + name: config-observability + literals: + - profiling.enable="true" patches: - path: api-db-config.yaml