feat(scheduler): make node lock timeout configurable

[Kevinz857]

Summary

This PR makes the HAMi scheduler's node lock timeout configurable, allowing operators to adjust the timeout duration based on their cluster characteristics and scheduling requirements.

Changes Made

1. Configuration Layer (pkg/scheduler/config/config.go)

   • Added NodeLockTimeout int configuration parameter
   • Default value set to 1 minute to maintain backward compatibility

2. Command Line Interface (cmd/scheduler/main.go)

   • Added --node-lock-timeout flag to scheduler command line options
   • Flag accepts timeout value in minutes
   • Integrated timeout initialization in the start() function

3. Runtime Behavior (pkg/util/nodelock/nodelock.go)

   • The global NodeLockTimeout variable is now set from configuration
   • Maintains existing lock/unlock logic and timeout checking mechanism

Motivation

Previously, the node lock timeout was hardcoded to 1 minute. In some cluster environments, this timeout may be too long, causing unnecessary delays in scheduling operations. By making this configurable, operators can:

• Reduce scheduling latency in fast clusters by setting shorter timeouts
• Increase timeout for slower clusters to avoid premature lock releases
• Fine-tune scheduler behavior based on workload characteristics

Backward Compatibility

• Default timeout remains 1 minute, ensuring no behavior change for existing deployments
• Existing configurations continue to work without modification
• New flag is optional and uses sensible defaults

Default timeout is 5 minute

./scheduler

Custom set 30s

./scheduler --node-lock-timeout=30s

Custom set 2mins

./scheduler --node-lock-timeout=2m

[archlitchi]

have you tested on your environment?

[Kevinz857]

have you tested on your environment?
@archlitchi Yes, I have verified in our own production environment, and so far this lock time is working properly

[wawa0210]

/lgtm

[codecov]

Codecov Report

Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
pkg/util/nodelock/nodelock.go	33.33%	1 Missing and 1 partial ⚠️

Flag	Coverage Δ
unittests	52.95% <33.33%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
pkg/util/nodelock/nodelock.go	53.92% <33.33%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[hami-robott]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Kevinz857
Once this PR has been reviewed and has the lgtm label, please ask for approval from wawa0210. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[archlitchi]

/lgtm