Skip to content

Eliranw/run 34335 - Add KWOK deployment for DRA resources#144

Merged
eliranw merged 2 commits intomainfrom
eliranw/RUN-34335
Dec 17, 2025
Merged

Eliranw/run 34335 - Add KWOK deployment for DRA resources#144
eliranw merged 2 commits intomainfrom
eliranw/RUN-34335

Conversation

@eliranw
Copy link
Copy Markdown
Contributor

@eliranw eliranw commented Dec 15, 2025

No description provided.

@eliranw eliranw changed the base branch from main to eliranw/RUN-34332-add-dra December 15, 2025 08:42
Base automatically changed from eliranw/RUN-34332-add-dra to main December 16, 2025 09:05
- Add kwok-dra-plugin component that watches topology ConfigMaps and creates ResourceSlices
- Add Helm templates for kwok-dra-plugin deployment with RBAC
- Add integration tests for KWOK node GPU scheduling
- Update README with KWOK integration documentation
- Add KWOK stages and test manifests for integration testing
@eliranw eliranw changed the title Eliranw/run 34335 - Add KWOK deployment for DRA resour es Eliranw/run 34335 - Add KWOK deployment for DRA resources Dec 17, 2025
// ResourceSliceHandler handles ResourceSlice operations for KWOK nodes
type ResourceSliceHandler struct {
kubeClient kubernetes.Interface
clusterTopology *topology.ClusterTopology
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

looks like this is never used

- Removed the topology package from the KWOK DRA plugin, simplifying the ConfigMap reconciler setup.
- Updated ResourceSliceHandler to eliminate the clusterTopology parameter, streamlining its initialization.
- Adjusted related tests to reflect the removal of the topology dependency.
@eliranw eliranw merged commit f000471 into main Dec 17, 2025
7 checks passed
@eliranw eliranw deleted the eliranw/RUN-34335 branch December 17, 2025 14:46
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