Closed jkhelil closed 4 days ago
The following is the coverage report on the affected files.
Say /test pull-tekton-operator-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/reconciler/openshift/tektonconfig/common.go | 54.8% | 0.0% | -54.8 |
pkg/reconciler/openshift/tektonconfig/rbac.go | 45.6% | 0.0% | -45.6 |
The following is the coverage report on the affected files.
Say /test pull-tekton-operator-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
pkg/reconciler/openshift/tektonconfig/common.go | 54.8% | 0.0% | -54.8 |
pkg/reconciler/openshift/tektonconfig/rbac.go | 45.6% | 0.0% | -45.6 |
Tested locally by upgrading from release-v0.71.x to this PR changes Steps:
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: savitaashture
The full list of commands accepted by this bot can be found here.
The pull request process is described here
cc @PuneetPunamiya
/lgtm
/cherry-pick release-v0.73.x
@jkhelil: new pull request created: #2349
Changes
CreateResource function is ensuring rbac resource for opensift namespaces. When implemented through concurrency, we observed unstability due the fact that concurrent workers tries to update a shared object clusterinterceptor cluster role binding. concurrency seems not bring improvement in performance, and bring unstability The fix reverts concurrency implementation and fixes the creates resource function to not block the hole reconciliation when an issue is observed for one namespace
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:
make test lint
before submitting a PRSee the contribution guide for more details.
Release Notes