search

openshift-pipelines / pipeline-service

SaaS for Tekton Pipelines
Apache License 2.0
20 stars 43 forks source link

pin watcher to mem leak version, revert latest fix attempt #1010

Closed gabemontero closed 3 weeks ago

gabemontero commented 3 weeks ago

undo only half of https://github.com/openshift-pipelines/pipeline-service/pull/1007 as the older watcher and newest api server work OK together

let's keep the api server tuning options

@enarha PTAL

rh-pre-commit.version: 2.3.0 rh-pre-commit.check-secrets: ENABLED

gabemontero commented 3 weeks ago

PR updated with your yq suggestion @enarha PTAL thanks

enarha commented 3 weeks ago

/lgtm

gabemontero commented 3 weeks ago

unfortunately yq is not found @enarha ... from the open-infra-deployment-pr-n6vks:

step-git-clone-infra-deployments
2024/06/24 16:56:43 warning: unsuccessful cred copy: ".docker" from "/tekton/creds" to "/": unable to create destination directory: mkdir /.docker: permission denied
Cloning into 'cloned'...

step-run-update-script
sh: line 3: yq: command not found
sh: line 6: yq: command not found

step-get-diff-files
2024/06/24 16:56:43 warning: unsuccessful cred copy: ".docker" from "/tekton/creds" to "/": unable to create destination directory: mkdir /.docker: permission denied
Script has modified files:
components/monitoring/grafana/base/dashboards/pipeline-service/kustomization.yaml
components/pipeline-service/development/kustomization.yaml
components/pipeline-service/staging/base/kustomization.yaml

step-create-mr

I'll refresh on how this repo installs yq for its images and see if that is translatable.