Merge pull request #58437 from tossmilestone/scheduler-golint
Automatic merge from submit-queue. 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>. Enable golinting for scheduler packages. **What this PR does / why we need it**: Enable golinting for scheduler packages **Which issue(s) this PR fixes**: Fixes #58234 **Special notes for your reviewer**: - `pkg/scheduler/api` and `pkg/scheduler/api/v1` are not removed from `hack/.golint_failures`, because there are auto-generated go files by `deepcopy-gen` in the package, which have golint errors and are not suggested manually edited. - Please help to refine the comments if there are error or inaccurate descriptions. Thanks! **Release note**: ```release-note Enable golint for `pkg/scheduler` and fix the golint errors in it. ```
This commit is contained in:
@@ -334,7 +334,7 @@ func TestSchedulerExtender(t *testing.T) {
|
||||
FilterVerb: filter,
|
||||
PrioritizeVerb: prioritize,
|
||||
Weight: 3,
|
||||
EnableHttps: false,
|
||||
EnableHTTPS: false,
|
||||
},
|
||||
{
|
||||
URLPrefix: es2.URL,
|
||||
@@ -342,14 +342,14 @@ func TestSchedulerExtender(t *testing.T) {
|
||||
PrioritizeVerb: prioritize,
|
||||
BindVerb: bind,
|
||||
Weight: 4,
|
||||
EnableHttps: false,
|
||||
EnableHTTPS: false,
|
||||
},
|
||||
{
|
||||
URLPrefix: es3.URL,
|
||||
FilterVerb: filter,
|
||||
PrioritizeVerb: prioritize,
|
||||
Weight: 10,
|
||||
EnableHttps: false,
|
||||
EnableHTTPS: false,
|
||||
NodeCacheCapable: true,
|
||||
},
|
||||
},
|
||||
|
Reference in New Issue
Block a user