pkg icon indicating copy to clipboard operation
pkg copied to clipboard

[webhook] logger in context should be annotated with webhook type

Open dprotaso opened this issue 5 years ago • 6 comments

Issue

When debugging some defaulting webhooks instrumenting SetDefaults on some types had the side-effect of logging invocations from the conversion webhook.

It was not obvious which webhook was doing the invocation of SetDefaults when looking at the logs.

dprotaso avatar Feb 26 '20 21:02 dprotaso

@dprotaso : Can you provide details on which type has logging in SetDefaults function?

shashwathi avatar Apr 22 '20 20:04 shashwathi

Can you provide details on which type has logging in SetDefaults function?

I was adding the logging myself while debugging something. That's where I noticed there are multiple calls originating from different webhooks endpoints and I couldn't disambiguate between them

dprotaso avatar Apr 23 '20 17:04 dprotaso

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

github-actions[bot] avatar Aug 24 '20 16:08 github-actions[bot]

/remove-lifecycle stale

dprotaso avatar Sep 18 '20 18:09 dprotaso

This issue is stale because it has been open for 90 days with no activity. It will automatically close after 30 more days of inactivity. Reopen the issue with /reopen. Mark the issue as fresh by adding the comment /remove-lifecycle stale.

github-actions[bot] avatar Dec 18 '20 01:12 github-actions[bot]

/lifecycle frozen

dprotaso avatar Mar 30 '21 18:03 dprotaso