Skip to content

K
k3s
Unverified Commit 7a273aa8 authored by Kubernetes Submit Queue's avatar Kubernetes Submit Queue Committed by GitHub
Browse files
Options

Merge pull request #60796 from ravisantoshgudimetla/extender-log-fix

Automatic merge from submit-queue (batch tested with PRs 60898, 60912, 60753, 61002, 60796). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. Change to fix scheduler extender error return message **What this PR does / why we need it**: As of now, scheduler always logs extender endpoint without verb like "filter", "prioritize" etc. With this change, we are including the verb as well while logging which helps in debugging **Which issue(s) this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close the issue(s) when PR gets merged)*: Fixes # **Special notes for your reviewer**: **Release note**: ```release-note NONE ```
parents e3ac4026 1c416b1c
Hide whitespace changes
Inline Side-by-side
Showing with 1 addition and 1 deletion
pkg/scheduler/core/extender.go
View file @ 7a273aa8
...@@ -385,7 +385,7 @@ func (h *HTTPExtender) send(action string, args interface{}, result interface{}) ...@@ -385,7 +385,7 @@ func (h *HTTPExtender) send(action string, args interface{}, result interface{})
defer resp.Body.Close() defer resp.Body.Close()
if resp.StatusCode != http.StatusOK { if resp.StatusCode != http.StatusOK {
return fmt.Errorf("Failed %v with extender at URL %v, code %v", action, h.extenderURL, resp.StatusCode) return fmt.Errorf("Failed %v with extender at URL %v, code %v", action, url, resp.StatusCode)
} }
return json.NewDecoder(resp.Body).Decode(result) return json.NewDecoder(resp.Body).Decode(result)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment