Skip to content

Conversation

@itsomri
Copy link
Collaborator

@itsomri itsomri commented Oct 1, 2025

This PR adds a simulator to test time-aware-fairness logic over time.
A simulation has the following inputs:

  • Queues: a list of queues, with optional quota, priority and weight
  • Jobs: a list of jobs, with pod & gpu count
  • Nodes: a list of nodes with GPU counts
  • The number of cycles the simulation should run for

@itsomri itsomri force-pushed the omric/time-aware-env-tests branch 4 times, most recently from 1db0bd0 to 88c3326 Compare October 9, 2025 12:38
SiorMeir
SiorMeir previously approved these changes Oct 12, 2025
Copy link
Collaborator

@SiorMeir SiorMeir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good!

Added a few suggestion

@itsomri itsomri force-pushed the omric/time-aware-env-tests branch 5 times, most recently from c764bf5 to a0b9848 Compare October 16, 2025 11:24
@github-actions
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 73.86% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers 43.75% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics 84.78% (-0.61%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb 56.98% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/app.go 0.00% (ø) 30 0 30
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/options.go 0.00% (ø) 14 (+3) 0 14 (+3)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/main.go 0.00% (ø) 10 0 10
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/app.go 0.00% (ø) 31 0 31
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/options.go 0.00% (ø) 12 (+1) 0 12 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder/binder.go 0.00% (ø) 24 (+1) 0 24 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller/podgroupcontroller.go 0.00% (ø) 10 (+1) 0 10 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller/queuecontroller.go 0.00% (ø) 11 (+1) 0 11 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler/scheduler.go 0.00% (ø) 26 (+2) 0 26 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers/pod_group_controller.go 0.00% (ø) 27 0 27
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/queue_controller.go 86.05% (ø) 43 37 6
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics/metrics.go 84.78% (-0.61%) 92 (+3) 78 (+2) 14 (+1) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake.go 0.00% (ø) 10 (+2) 0 10 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake_with_history.go 0.00% (ø) 49 (+49) 0 49 (+49)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub.go 100.00% (ø) 27 27 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/time_aware_fairness_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub_test.go

@itsomri itsomri force-pushed the omric/time-aware-env-tests branch from 712052e to b6672d8 Compare October 17, 2025 14:03
@github-actions
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 72.73% (-1.14%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers 43.75% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics 84.78% (-0.61%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb 56.98% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/app.go 0.00% (ø) 30 0 30
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/options.go 0.00% (ø) 14 (+3) 0 14 (+3)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/main.go 0.00% (ø) 10 0 10
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/app.go 0.00% (ø) 31 0 31
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/options.go 0.00% (ø) 12 (+1) 0 12 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder/binder.go 0.00% (ø) 24 (+1) 0 24 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller/podgroupcontroller.go 0.00% (ø) 10 (+1) 0 10 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller/queuecontroller.go 0.00% (ø) 11 (+1) 0 11 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler/scheduler.go 0.00% (ø) 26 (+2) 0 26 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers/pod_group_controller.go 0.00% (ø) 27 0 27
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/queue_controller.go 86.05% (ø) 43 37 6
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics/metrics.go 84.78% (-0.61%) 92 (+3) 78 (+2) 14 (+1) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake.go 0.00% (ø) 10 (+2) 0 10 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake_with_history.go 0.00% (ø) 49 (+49) 0 49 (+49)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub.go 100.00% (ø) 27 27 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/time_aware_fairness_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub_test.go

@github-actions
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 73.86% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers 43.75% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics 84.78% (-0.61%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb 56.98% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/app.go 0.00% (ø) 30 0 30
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/options.go 0.00% (ø) 14 (+3) 0 14 (+3)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/main.go 0.00% (ø) 10 0 10
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/app.go 0.00% (ø) 31 0 31
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/options.go 0.00% (ø) 12 (+1) 0 12 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder/binder.go 0.00% (ø) 24 (+1) 0 24 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller/podgroupcontroller.go 0.00% (ø) 10 (+1) 0 10 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller/queuecontroller.go 0.00% (ø) 11 (+1) 0 11 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler/scheduler.go 0.00% (ø) 26 (+2) 0 26 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers/pod_group_controller.go 0.00% (ø) 27 0 27
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/queue_controller.go 86.05% (ø) 43 37 6
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics/metrics.go 84.78% (-0.61%) 92 (+3) 78 (+2) 14 (+1) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake.go 0.00% (ø) 10 (+2) 0 10 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake_with_history.go 0.00% (ø) 49 (+49) 0 49 (+49)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub.go 100.00% (ø) 27 27 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/time_aware_fairness_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub_test.go

@itsomri itsomri force-pushed the omric/time-aware-env-tests branch from edb2e0e to a950c2d Compare October 19, 2025 12:29
@github-actions
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 73.86% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers 43.75% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics 84.78% (-0.61%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb 56.98% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/app.go 0.00% (ø) 30 0 30
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/options.go 0.00% (ø) 14 (+3) 0 14 (+3)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/main.go 0.00% (ø) 10 0 10
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/app.go 0.00% (ø) 31 0 31
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/options.go 0.00% (ø) 12 (+1) 0 12 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder/binder.go 0.00% (ø) 24 (+1) 0 24 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller/podgroupcontroller.go 0.00% (ø) 10 (+1) 0 10 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller/queuecontroller.go 0.00% (ø) 11 (+1) 0 11 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler/scheduler.go 0.00% (ø) 26 (+2) 0 26 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers/pod_group_controller.go 0.00% (ø) 27 0 27
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/queue_controller.go 86.05% (ø) 43 37 6
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics/metrics.go 84.78% (-0.61%) 92 (+3) 78 (+2) 14 (+1) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake.go 0.00% (ø) 10 (+2) 0 10 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake_with_history.go 0.00% (ø) 49 (+49) 0 49 (+49)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub.go 100.00% (ø) 27 27 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/time_aware_fairness_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub_test.go

@itsomri itsomri force-pushed the omric/time-aware-env-tests branch 3 times, most recently from 412445d to e63b579 Compare October 20, 2025 12:16
@github-actions
Copy link

Merging this branch changes the coverage (2 decrease, 1 increase)

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 73.86% (+1.14%) 👍
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers 43.75% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 83.72% (-2.33%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics 84.78% (-0.61%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb 56.98% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/app.go 0.00% (ø) 30 0 30
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/options.go 0.00% (ø) 14 (+3) 0 14 (+3)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/main.go 0.00% (ø) 10 0 10
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/app.go 0.00% (ø) 31 0 31
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/options.go 0.00% (ø) 12 (+1) 0 12 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder/binder.go 0.00% (ø) 24 (+1) 0 24 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller/podgroupcontroller.go 0.00% (ø) 10 (+1) 0 10 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller/queuecontroller.go 0.00% (ø) 11 (+1) 0 11 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler/scheduler.go 0.00% (ø) 26 (+2) 0 26 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers/pod_group_controller.go 0.00% (ø) 27 0 27
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/queue_controller.go 83.72% (-2.33%) 43 36 (-1) 7 (+1) 👎
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics/metrics.go 84.78% (-0.61%) 92 (+3) 78 (+2) 14 (+1) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake.go 0.00% (ø) 10 (+2) 0 10 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake_with_history.go 0.00% (ø) 49 (+49) 0 49 (+49)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub.go 100.00% (ø) 27 27 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/time_aware_fairness_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub_test.go

enoodle
enoodle previously approved these changes Oct 20, 2025
gshaibi
gshaibi previously approved these changes Oct 20, 2025
enoodle
enoodle previously approved these changes Oct 22, 2025
@itsomri itsomri enabled auto-merge (squash) October 22, 2025 11:10
@itsomri itsomri merged commit 8409531 into main Oct 22, 2025
4 checks passed
@itsomri itsomri deleted the omric/time-aware-env-tests branch October 22, 2025 11:38
@github-actions
Copy link

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests 68.82% (-3.91%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers 43.75% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers 86.05% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics 84.78% (-0.61%) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb 56.98% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake 0.00% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/app.go 0.00% (ø) 30 0 30
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/app/options.go 0.00% (ø) 14 (+3) 0 14 (+3)
github.com/NVIDIA/KAI-scheduler/cmd/podgroupcontroller/main.go 0.00% (ø) 10 0 10
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/app.go 0.00% (ø) 31 0 31
github.com/NVIDIA/KAI-scheduler/cmd/queuecontroller/app/options.go 0.00% (ø) 12 (+1) 0 12 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/binder/binder.go 0.00% (ø) 24 (+1) 0 24 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/podgroupcontroller/podgroupcontroller.go 0.00% (ø) 10 (+1) 0 10 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/queuecontroller/queuecontroller.go 0.00% (ø) 11 (+1) 0 11 (+1)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/scheduler/scheduler.go 0.00% (ø) 26 (+2) 0 26 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/env-tests/utils.go 68.82% (-3.91%) 93 (+5) 64 29 (+5) 👎
github.com/NVIDIA/KAI-scheduler/pkg/podgroupcontroller/controllers/pod_group_controller.go 0.00% (ø) 27 0 27
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/queue_controller.go 86.05% (ø) 43 37 6
github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/metrics/metrics.go 84.78% (-0.61%) 92 (+3) 78 (+2) 14 (+1) 👎
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake.go 0.00% (ø) 10 (+2) 0 10 (+2)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/fake/fake_with_history.go 0.00% (ø) 49 (+49) 0 49 (+49)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub.go 100.00% (ø) 27 27 0

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Changed unit test files

  • github.com/NVIDIA/KAI-scheduler/pkg/env-tests/time_aware_fairness_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/operator/controller/integration_tests/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/queuecontroller/controllers/suite_test.go
  • github.com/NVIDIA/KAI-scheduler/pkg/scheduler/cache/usagedb/hub_test.go

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants