Skip to content

Commit 8792155

Browse files
committed
fix copilot comments
Signed-off-by: roytman <roytman@il.ibm.com>
1 parent bf25381 commit 8792155

1 file changed

Lines changed: 6 additions & 1 deletion

File tree

pkg/epp/framework/plugins/scheduling/profilehandler/headerphase/header_phase_profile_handler.go

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ import (
2424

2525
"sigs.k8s.io/controller-runtime/pkg/log"
2626

27+
"github.com/llm-d/llm-d-router/pkg/common/observability/logging"
2728
fwkplugin "github.com/llm-d/llm-d-router/pkg/epp/framework/interface/plugin"
2829
fwksched "github.com/llm-d/llm-d-router/pkg/epp/framework/interface/scheduling"
2930
)
@@ -181,7 +182,11 @@ func (h *HeaderPhaseProfileHandler) Pick(ctx context.Context, request *fwksched.
181182

182183
profile, ok := profiles[resolvedPhase]
183184
if !ok {
184-
log.FromContext(ctx).Error(h.noMatchError(phase), "no scheduling profile selected for request")
185+
// A missing or unrecognized header value is a per-request client issue, not a
186+
// system fault - log at DEBUG so it doesn't page anyone or drown out real
187+
// errors, matching how parseSLOHeaders logs a malformed client header
188+
// (pkg/epp/framework/plugins/requestcontrol/dataproducer/predictedlatency/plugin.go).
189+
log.FromContext(ctx).V(logging.DEBUG).Error(h.noMatchError(phase), "no scheduling profile selected for request")
185190
return map[string]fwksched.SchedulerProfile{}
186191
}
187192

0 commit comments

Comments
 (0)