Skip to content

Conversation

@itsomri
Copy link
Collaborator

@itsomri itsomri commented Dec 7, 2025

Description

This PR fixes a bug where the topology tool would not load the topology objects from the snapshot.

Checklist

  • Self-reviewed
  • Added/updated tests (if needed)
  • Updated documentation (if needed)

@itsomri itsomri force-pushed the omric/snapshot-topology branch from 5883baa to b5bf4ad Compare December 7, 2025 09:57
@github-actions
Copy link

github-actions bot commented Dec 7, 2025

Merging this branch will not change overall coverage

Impacted Packages Coverage Δ 🤖
github.com/NVIDIA/KAI-scheduler/cmd/snapshot-tool 0.00% (ø)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/actions/common 23.42% (ø)

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/NVIDIA/KAI-scheduler/cmd/snapshot-tool/main.go 0.00% (ø) 117 (+4) 0 117 (+4)
github.com/NVIDIA/KAI-scheduler/pkg/scheduler/actions/common/allocate.go 0.00% (ø) 120 0 120

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.

@itsomri itsomri merged commit 26e965f into main Dec 7, 2025
5 checks passed
@itsomri itsomri deleted the omric/snapshot-topology branch December 7, 2025 10:52
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.

2 participants