Skip to content

schedule: impl balance range scheduler #23629

schedule: impl balance range scheduler

schedule: impl balance range scheduler #23629

Triggered via pull request January 21, 2025 09:27
Status Failure
Total duration 3m 41s
Artifacts

check.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

8 errors and 1 warning
statics: pkg/schedule/schedulers/balance_range.go#L414
append to slice `ret` with non-zero initialized length (makezero)
statics: pkg/schedule/schedulers/balance_range.go#L383
exported: exported type Role should have comment or be unexported (revive)
statics: pkg/schedule/schedulers/balance_range.go#L386
exported: exported const Leader should have comment (or a comment on this block) or be unexported (revive)
statics: pkg/schedule/schedulers/balance_range.go#L175
unused-parameter: parameter 'dryRun' seems to be unused, consider removing or renaming it to match ^_ (revive)
statics: pkg/schedule/schedulers/balance_range.go#L228
empty-lines: extra empty line at the end of a block (revive)
statics: pkg/schedule/schedulers/balance_range.go#L232
unnecessary trailing newline (whitespace)
statics: pkg/schedule/schedulers/balance_range.go#L141
S1012: should use `time.Since` instead of `time.Now().Sub` (gosimple)
statics
Process completed with exit code 2.
statics
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636