Closed sayan-biswas closed 6 months 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/api/main.go | 11.9% | 11.4% | -0.5 |
pkg/api/server/config/config.go | Do not exist | 0.0% |
@gabemontero Here's the PR for enabling basic pprof.
Although we can port-forward from pod also, I think having it in the service is the right way. I'll make the changes, also fix the lint issue.
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 | 11.9% | 11.4% | -0.5 |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% | |
pkg/api/server/config/config.go | Do not exist | 0.0% |
@gabemontero: changing LGTM is restricted to collaborators
@enarha Let's merge this.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: avinal, gabemontero, khrm
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Changes
This feature add golang's native profiling server to expose debug profiles.
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