Closed khrm closed 4 weeks ago
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/converter/main.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
I have removed the cronjob and added a flag to the API server to enable converter. https://github.com/tektoncd/results/compare/e70451083446c26c8eb0a8e98b1dda5310b126ae..2f931e13096492c7b4755c22875a6400b0239492
/test pull-tekton-results-build-tests
Strange. There's something weird going on. Somehow it's showing changes from main branch also.
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/api/main.go | 9.3% | 9.0% | -0.3 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
/test pull-tekton-results-build-tests
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/api/main.go | 9.3% | 9.0% | -0.3 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
/test pull-tekton-results-build-tests
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/api/main.go | 9.3% | 9.0% | -0.3 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
@avinal Please look into this.
/cc @enarha @vdemeester
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/api/main.go | 9.3% | 9.0% | -0.3 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
/test pull-tekton-results-build-tests
/test pull-tekton-results-build-tests
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/api/main.go | 9.3% | 9.0% | -0.3 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
/test pull-tekton-results-build-tests
@vdemeester @enarha Can you please look into this PR?
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/api/main.go | 9.3% | 9.0% | -0.3 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
@enarha LGTM got removed due to rebase and error fix. Please review again.
The following is the coverage report on the affected files.
Say /test pull-tekton-results-go-coverage
to re-run this coverage report
File | Old Coverage | New Coverage | Delta |
---|---|---|---|
cmd/api/main.go | 9.3% | 9.0% | -0.3 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% | |
pkg/converter/convert.go | Do not exist | 0.0% |
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: enarha
The full list of commands accepted by this bot can be found here.
The pull request process is described here
This ensures that only v1 records are existing in the db instead of a mix of v1beta1 and v1
Changes
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you review them:
/kind <type>
. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tepRelease Notes