github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-09-13 13:14:05 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #128380 from pohly/log-TODO-logcheck

Browse Source 
apiserver: avoid TODO in public docs
This commit is contained in:
Kubernetes Prow Robot
2024-11-06 20:09:53 +00:00
committed by GitHub
parent dfba334a33 fc6e0a5799
commit 432a9af0fb
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
staging/src/k8s.io/apimachinery/pkg/util/runtime/runtime.go
View File

@@ -45,7 +45,7 @@ var PanicHandlers = []func(context.Context, interface{}){logPanic}
// //
// E.g., you can provide one or more additional handlers for something like shutting down go routines gracefully. // E.g., you can provide one or more additional handlers for something like shutting down go routines gracefully.
// //
// TODO(pohly): logcheck:context // HandleCrashWithContext should be used instead of HandleCrash in code which supports contextual logging. // Contextual logging: HandleCrashWithContext should be used instead of HandleCrash in code which supports contextual logging.
func HandleCrash(additionalHandlers ...func(interface{})) { func HandleCrash(additionalHandlers ...func(interface{})) {
if r := recover(); r != nil { if r := recover(); r != nil {
additionalHandlersWithContext := make([]func(context.Context, interface{}), len(additionalHandlers)) additionalHandlersWithContext := make([]func(context.Context, interface{}), len(additionalHandlers))
@@ -146,7 +146,7 @@ type ErrorHandler func(ctx context.Context, err error, msg string, keysAndValues
// is preferable to logging the error - the default behavior is to log but the // is preferable to logging the error - the default behavior is to log but the
// errors may be sent to a remote server for analysis. // errors may be sent to a remote server for analysis.
// //
// TODO(pohly): logcheck:context // HandleErrorWithContext should be used instead of HandleError in code which supports contextual logging. // Contextual logging: HandleErrorWithContext should be used instead of HandleError in code which supports contextual logging.
func HandleError(err error) { func HandleError(err error) {
// this is sometimes called with a nil error. We probably shouldn't fail and should do nothing instead // this is sometimes called with a nil error. We probably shouldn't fail and should do nothing instead
if err == nil { if err == nil {