Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RUN-16744 Support KWOK #70

Merged

Conversation

gshaibi
Copy link
Contributor

@gshaibi gshaibi commented Mar 31, 2024

This PR presents KWOK minimal support for scale tests on fake GPU nodes.
This support does not guarantee metric correctness.

@gshaibi gshaibi changed the base branch from main to shaibi/RUN-17256 March 31, 2024 11:00
Base automatically changed from shaibi/RUN-17256 to main March 31, 2024 14:58
…i/RUN-16744-support-kwok-imp-daemonset-to-deployment
@gshaibi gshaibi changed the title shaibi/RUN 16744 support kwok imp daemonset to deployment RUN-16744 Support KWOK Mar 31, 2024
@gshaibi gshaibi merged commit 50bcff3 into main Apr 2, 2024
2 of 3 checks passed
@gshaibi gshaibi deleted the shaibi/RUN-16744-support-kwok-imp-daemonset-to-deployment branch April 2, 2024 08:14
Comment on lines -4 to -7
{{- if .Values.environment.openshift }}
annotations:
openshift.io/scc: hostmount-anyuid
{{- end }}
Copy link
Contributor

Choose a reason for hiding this comment

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

why no longer needed?


ReservationNs = "runai-reservation"

GpuResourceName = "nvidia.com/gpu"

// GuyTodo: Use these constants in the code
Copy link
Contributor

Choose a reason for hiding this comment

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

remove todo?

}

func NewDevicePlugin(topology *topology.NodeTopology) *DevicePlugin {
func NewDevicePlugin(topology *topology.NodeTopology, kubeClient kubernetes.Interface) Interface {
Copy link
Contributor

Choose a reason for hiding this comment

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

nice change 👍

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.

3 participants