search

openshift-pipelines / pipeline-service

SaaS for Tekton Pipelines
Apache License 2.0
23 stars 44 forks source link

PLNSRVCE-1526: expose core tekton webhook replicas setting in operator TektonConfig #931

Closed gabemontero closed 6 months ago

gabemontero commented 6 months ago

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

From local testing:

tekton-operator-proxy-webhook-7f6d7f7d74-bf6xs       1/1     Running     0          2m31s
tekton-operator-proxy-webhook-7f6d7f7d74-mtcdr       1/1     Running     0          2m31s
tekton-pipelines-webhook-84c6fc988d-2kqzd            1/1     Running     0          2m32s
tekton-pipelines-webhook-84c6fc988d-srwdz            1/1     Running     0          2m32s

After seeing tests pass here with 2 replicas, I'll move down back to one replica, and bump the replicas via a gitops patch in infra-deployments

@openshift-pipelines/pipelines-service FYI

ramessesii2 commented 6 months ago

/retest test-pipeline-service-upgrade-ocp-414

gabemontero commented 6 months ago

all green ... going to reduce the replicas to one, but leave the setting explicit, and will try to override via a gitops / kustomize patch in infra-deployments

if that proves unexpectedly difficult, I'll come back here and force it to two

gabemontero commented 6 months ago

the test-docker-builds taskrun was cancelled for some reason .... it passed previously, not going to worry about retest

gabemontero commented 6 months ago

And the upgrade test has been stuck in setup-ci-runner-container for about 30 minutes:

step-copy-plnsvc-code
Copy source code of pipeline service to the ci-runner container
+ echo 'Copy source code of pipeline service to the ci-runner container'
+ for _ in {1..10}
+ kubectl cp ./ default/ci-runner:/workspace/source

@xinredhat @Roming22 a known thing ^^ ??

I'll wait a bit longer, but will probably hit the merge button soon