Skip to content

Improve logging for webhooks from GitHub #8144

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Conversation

jldec
Copy link
Contributor

@jldec jldec commented Feb 10, 2022

Description

Improve our GitHub webhook handler to log errors instead of only tracing.

Related Issue(s)

Fixes #7938

Release Notes

NONE
  • /werft with-clean-slate-deployment

@codecov
Copy link

codecov bot commented Feb 10, 2022

Codecov Report

Merging #8144 (ce198b0) into main (a48e177) will decrease coverage by 16.99%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #8144       +/-   ##
===========================================
- Coverage   28.17%   11.17%   -17.00%     
===========================================
  Files          41       18       -23     
  Lines        3478      993     -2485     
===========================================
- Hits          980      111      -869     
+ Misses       2443      880     -1563     
+ Partials       55        2       -53     
Flag Coverage Δ
components-gitpod-cli-app 11.17% <ø> (ø)
components-local-app-app-darwin-amd64 ?
components-local-app-app-darwin-arm64 ?
components-local-app-app-linux-amd64 ?
components-local-app-app-linux-arm64 ?
components-local-app-app-windows-386 ?
components-ws-proxy-app ?
installer-raw-app ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
components/ws-proxy/pkg/proxy/cookies.go
components/ws-proxy/pkg/proxy/auth.go
installer/pkg/components/ws-manager/rolebinding.go
components/local-app/pkg/auth/pkce.go
installer/pkg/common/storage.go
components/ws-proxy/pkg/proxy/infoprovider.go
installer/pkg/components/ws-manager/deployment.go
installer/pkg/components/ws-manager/tlssecret.go
installer/pkg/components/ws-manager/configmap.go
components/ws-proxy/pkg/proxy/config.go
... and 13 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a48e177...ce198b0. Read the comment docs.

@jldec
Copy link
Contributor Author

jldec commented Feb 11, 2022

/werft run

👍 started the job as gitpod-build-jldec-webhook-logs.1

@jldec jldec force-pushed the jldec/webhook-logs branch 2 times, most recently from 5ffcec2 to f797dd3 Compare February 11, 2022 14:56
@jldec jldec closed this Feb 14, 2022
@jldec jldec force-pushed the jldec/webhook-logs branch from f797dd3 to a48e177 Compare February 14, 2022 11:19
@roboquat roboquat added size/XS and removed size/L labels Feb 14, 2022
@jldec
Copy link
Contributor Author

jldec commented Feb 14, 2022

roboquat should not have closed this.

@jldec jldec reopened this Feb 14, 2022
@roboquat roboquat added size/M and removed size/XS labels Feb 14, 2022
@jldec
Copy link
Contributor Author

jldec commented Feb 14, 2022

/werft run

👍 started the job as gitpod-build-jldec-webhook-logs.6

@stale
Copy link

stale bot commented Feb 24, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the meta: stale This issue/PR is stale and will be closed soon label Feb 24, 2022
@stale stale bot closed this Mar 2, 2022
@jldec jldec deleted the jldec/webhook-logs branch April 26, 2022 17:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better instrument incoming webhooks via logging
2 participants