-
Notifications
You must be signed in to change notification settings - Fork 295
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
✨ vGPU implementation #2272
✨ vGPU implementation #2272
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
@@ -0,0 +1,107 @@ | ||||
# GPU enabled clusters using vGPU | ||||
|
||||
## Overview | ||||
|
||||
You can choose to create a cluster with both worker and control plane nodes having vGPU devices attached to them. | ||||
|
||||
Before we begin, a few important things to note: | ||||
|
||||
- [NVIDIA GPU Operator](https://github.com/NVIDIA/gpu-operator) is used to expose the GPU PCI devices to the workloads running on the cluster. | ||||
- The OVA templates used for cluster creation should have the VMX version (Virtual Hardware) set to 17 or higher. This is necessary because Dynamic DirectPath I/O was introduced in this version, which enables the Assignable Hardware intelligence for passthrough devices. | ||||
- Since we need the VMX version to be >=17, this way of provisioning clusters with PCI passthrough devices works for vSphere 7.0 and above. This is the ESXi/VMX version [compatibility list](https://kb.vmware.com/s/article/2007240). | ||||
- UEFI boot mode is recommended for the OVAs used for cluster creation. | ||||
- Most of the setup is similar to [GPU enabled clusters via PCI Passthrough](https://github.com/kubernetes-sigs/cluster-api-provider-vsphere/blob/main/docs/gpu-pci.md#create-the-cluster). | ||||
|
||||
## An example GPU enabled cluster | ||||
|
||||
Let's create a CAPV cluster with vGPU enabled nodes. | ||||
|
||||
### Prerequisites | ||||
|
||||
- Refer the [NVIDIA Virtual GPU Software Quick Start Guide](https://docs.nvidia.com/grid/latest/grid-software-quick-start-guide/index.html) to download and install the vGPU software and configure vGPU licensing. | ||||
|
||||
- Ensure vGPU compatibility for your vSphere installation and the GPU devices using the [VMware Compatibility Guide - Shared Pass-through Graphics](https://www.vmware.com/resources/compatibility/search.php?deviceCategory=vgpu) | ||||
|
||||
- Enable Shared Passthrough for the GPU device on the ESXi Host | ||||
- Browse to a host in the vSphere Client navigator. | ||||
- On the **Configure** tab, expand **Hardware** and click **Graphics**. | ||||
- Under **GRAPHICS DEVICES**, select the GPU device to be used for vGPU, click **EDIT...** and select **Shared Direct**. Repeat this for additional GPU devices as needed. | ||||
- Select **HOST GRAPHICS**, click **EDIT...** and select **Shared Direct** and select a shared passthrough GPU assignment policy, for example **Group VMs on GPU until full (GPU consolidation)**. | ||||
|
||||
- Build an OVA template | ||||
We can build a custom OVA template using the [image-builder](https://github.com/kubernetes-sigs/image-builder) project. We will build a Ubuntu 20.04 OVA with UEFI boot mode. More documentation on how to use image-builder can be found in the [image-builder book](https://image-builder.sigs.k8s.io/capi/providers/vsphere.html) | ||||
- Clone the repo locally and go to the `./images/capi/` directory. | ||||
- Create a `packer-vars.json` file with the following content. | ||||
|
||||
```shell | ||||
$ cat packer-vars.json | ||||
{ | ||||
"vmx_version": 17 | ||||
} | ||||
``` | ||||
|
||||
- Run the make file target associated to ubuntu 20.04 UEFI OVA as follows: | ||||
|
||||
```shell | ||||
> PACKER_VAR_FILES=packer-vars.json make build-node-ova-vsphere-ubuntu-2004-efi | ||||
``` | ||||
|
||||
### Source the vGPU profile(s) for the GPU device | ||||
|
||||
See "2. Choosing the vGPU Profile for the Virtual Machine" at [Using GPUs with Virtual Machines on vSphere](https://blogs.vmware.com/apps/2018/09/using-gpus-with-virtual-machines-on-vsphere-part-3-installing-the-nvidia-grid-technology.html) to see what vGPU profiles are available for your GPU device. | ||||
|
||||
We are using NVIDIA Tesla V100 32GB cards for this example and will use the `grid_v100d-4c` vGPU profile for this card that allocates 4GB GPU memory to the worker node's vGPU device. | ||||
|
||||
### Create the cluster template | ||||
|
||||
```shell | ||||
$ make dev-flavors | ||||
/Applications/Xcode.app/Contents/Developer/usr/bin/make generate-flavors FLAVOR_DIR=/Users/pkatyal/.cluster-api/overrides/infrastructure-vsphere/v0.0.0 | ||||
go run ./packaging/flavorgen --output-dir /Users/pkatyal/.cluster-api/overrides/infrastructure-vsphere/v0.0.0 | ||||
``` | ||||
|
||||
Edit the generated Cluster template (`cluster-template.yaml`) to set the values for the `vgpuDevices` array. Here we are editing the VSphereMachineTemplate object for the worker nodes. This will create a worker node with a single NVIDIA 16GB vGPU device attached to the VM. | ||||
|
||||
```yaml | ||||
--- | ||||
apiVersion: infrastructure.cluster.x-k8s.io/v1beta1 | ||||
kind: VSphereMachineTemplate | ||||
metadata: | ||||
name: ${CLUSTER_NAME}-worker | ||||
namespace: '${NAMESPACE}' | ||||
spec: | ||||
template: | ||||
spec: | ||||
cloneMode: linkedClone | ||||
datacenter: '${VSPHERE_DATACENTER}' | ||||
datastore: '${VSPHERE_DATASTORE}' | ||||
diskGiB: 25 | ||||
folder: '${VSPHERE_FOLDER}' | ||||
memoryMiB: 8192 | ||||
network: | ||||
devices: | ||||
- dhcp4: true | ||||
networkName: '${VSPHERE_NETWORK}' | ||||
numCPUs: 2 | ||||
os: Linux | ||||
powerOffMode: trySoft | ||||
resourcePool: '${VSPHERE_RESOURCE_POOL}' | ||||
server: '${VSPHERE_SERVER}' | ||||
storagePolicyName: '${VSPHERE_STORAGE_POLICY}' | ||||
template: '${VSPHERE_TEMPLATE}' | ||||
thumbprint: '${VSPHERE_TLS_THUMBPRINT}' | ||||
vgpuDevices: | ||||
- profileName: "grid_v100d-4c" <============ value from above | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. To make it a valid yaml
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Would it be worth having this as envsubst parameter in some way? WDYT: would it be worth having a separate flavor for vgpu? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The same NVIDIA GPU supports multiple vGPU profiles, and the matrix expands when you add more GPU varieties to the mix. For example, in my testing, I use different profiles for different worker nodes for the same workload cluster. I don't think it's useful to have this as an envsubst parameter. |
||||
``` | ||||
|
||||
Set the required values for the other fields and the cluster template is ready for use. The similar changes can be made to a template generated using clusterctl generate cluster command as well. | ||||
|
||||
### Create the cluster | ||||
|
||||
Set the size of the GPU nodes appropriately, since the Nvidia gpu-operator requires additional CPU and memory to install the device drivers on the VMs. | ||||
|
||||
Note: For GPU nodes (PCI Passthrough or vGPU), all memory of the nodes must be reserved. CAPV will automatically do this for nodes that have a PCI Passthrough GPU or a vGPU device in the spec. See "Memory Reservation" at [Using GPUs with Virtual Machines on vSphere](https://blogs.vmware.com/apps/2018/09/using-gpus-with-virtual-machines-on-vsphere-part-2-vmdirectpath-i-o.html) | ||||
|
||||
Apply the manifest from the previous step to your management cluster to have CAPV create a workload cluster with worker nodes that have vGPUs. | ||||
|
||||
From this point on, the setup is exactly the same as [GPU enabled clusters via PCI Passthrough](https://github.com/kubernetes-sigs/cluster-api-provider-vsphere/blob/main/docs/gpu-pci.md#create-the-cluster). |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -68,7 +68,7 @@ func Clone(ctx *context.VMContext, bootstrapData []byte, format bootstrapv1.Form | |
} | ||
} | ||
if ctx.VSphereVM.Spec.CustomVMXKeys != nil { | ||
ctx.Logger.Info("applied custom vmx keys o VM clone spec") | ||
ctx.Logger.Info("applied custom vmx keys to VM clone spec") | ||
if err := extraConfig.SetCustomVMXKeys(ctx.VSphereVM.Spec.CustomVMXKeys); err != nil { | ||
return err | ||
} | ||
|
@@ -151,8 +151,16 @@ func Clone(ctx *context.VMContext, bootstrapData []byte, format bootstrapv1.Form | |
|
||
deviceSpecs = append(deviceSpecs, networkSpecs...) | ||
|
||
if err != nil { | ||
return errors.Wrapf(err, "error getting network specs for %q", ctx) | ||
if len(ctx.VSphereVM.Spec.VirtualMachineCloneSpec.PciDevices) != 0 { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This adds a second codepath which adds PciDevices to a VM. The already existing code is at https://github.com/chrischdi/cluster-api-provider-vsphere/blob/234e7751f7bb09bb438eed6b4feabbc274cae842/pkg/services/govmomi/service.go#L478 . I'd prefer only having one place to do this so either this here should get removed or the other place. And we have to take care to not break existing use cases. |
||
gpuSpecs := getGpuSpecs(ctx) | ||
ctx.Logger.V(4).Info("created gpu devices", "gpu-device-specs", gpuSpecs) | ||
deviceSpecs = append(deviceSpecs, gpuSpecs...) | ||
} | ||
|
||
if len(ctx.VSphereVM.Spec.VirtualMachineCloneSpec.VGPUDevices) != 0 { | ||
vgpuSpecs := getVgpuSpecs(ctx) | ||
ctx.Logger.V(4).Info("created vgpu devices", "vgpu-device-specs", vgpuSpecs) | ||
deviceSpecs = append(deviceSpecs, vgpuSpecs...) | ||
} | ||
|
||
numCPUs := ctx.VSphereVM.Spec.NumCPUs | ||
|
@@ -199,10 +207,10 @@ func Clone(ctx *context.VMContext, bootstrapData []byte, format bootstrapv1.Form | |
Snapshot: snapshotRef, | ||
} | ||
|
||
// For PCI devices, the memory for the VM needs to be reserved | ||
// For PCI and vGPU devices, the memory for the VM needs to be reserved | ||
// We can replace this once we have another way of reserving memory option | ||
// exposed via the API types. | ||
if len(ctx.VSphereVM.Spec.PciDevices) > 0 { | ||
if len(ctx.VSphereVM.Spec.VirtualMachineCloneSpec.PciDevices) > 0 || len(ctx.VSphereVM.Spec.VirtualMachineCloneSpec.VGPUDevices) > 0 { | ||
spec.Config.MemoryReservationLockedToMax = pointer.Bool(true) | ||
} | ||
|
||
|
@@ -453,3 +461,55 @@ func getNetworkSpecs(ctx *context.VMContext, devices object.VirtualDeviceList) ( | |
|
||
return deviceSpecs, nil | ||
} | ||
|
||
func createPCIPassThroughDevice(deviceKey int32, backingInfo types.BaseVirtualDeviceBackingInfo) types.BaseVirtualDevice { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's move this func to the very bottom of the file, because its a helper for the ones below. |
||
device := &types.VirtualPCIPassthrough{ | ||
VirtualDevice: types.VirtualDevice{ | ||
Key: deviceKey, | ||
Backing: backingInfo, | ||
}, | ||
} | ||
return device | ||
puneetkatyal marked this conversation as resolved.
Show resolved
Hide resolved
|
||
} | ||
|
||
func getGpuSpecs(ctx *context.VMContext) []types.BaseVirtualDeviceConfigSpec { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could we try to not use the There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also: this function seems to be a duplicate of https://github.com/chrischdi/cluster-api-provider-vsphere/blob/234e7751f7bb09bb438eed6b4feabbc274cae842/pkg/services/govmomi/pci/device.go#L59 ? |
||
deviceSpecs := []types.BaseVirtualDeviceConfigSpec{} | ||
deviceKey := int32(-200) | ||
|
||
for _, pciDevice := range ctx.VSphereVM.Spec.VirtualMachineCloneSpec.PciDevices { | ||
backingInfo := &types.VirtualPCIPassthroughDynamicBackingInfo{ | ||
AllowedDevice: []types.VirtualPCIPassthroughAllowedDevice{ | ||
{ | ||
VendorId: *pciDevice.VendorID, | ||
DeviceId: *pciDevice.DeviceID, | ||
}, | ||
}, | ||
} | ||
dynamicDirectPathDevice := createPCIPassThroughDevice(deviceKey, backingInfo) | ||
deviceSpecs = append(deviceSpecs, &types.VirtualDeviceConfigSpec{ | ||
Device: dynamicDirectPathDevice, | ||
Operation: types.VirtualDeviceConfigSpecOperationAdd, | ||
}) | ||
deviceKey-- | ||
} | ||
return deviceSpecs | ||
} | ||
|
||
func getVgpuSpecs(ctx *context.VMContext) []types.BaseVirtualDeviceConfigSpec { | ||
deviceSpecs := []types.BaseVirtualDeviceConfigSpec{} | ||
deviceKey := int32(-200) | ||
|
||
for _, vGPUDevice := range ctx.VSphereVM.Spec.VirtualMachineCloneSpec.VGPUDevices { | ||
backingInfo := &types.VirtualPCIPassthroughVmiopBackingInfo{ | ||
Vgpu: vGPUDevice.ProfileName, | ||
} | ||
dynamicDirectPathDevice := createPCIPassThroughDevice(deviceKey, backingInfo) | ||
deviceSpecs = append(deviceSpecs, &types.VirtualDeviceConfigSpec{ | ||
Device: dynamicDirectPathDevice, | ||
Operation: types.VirtualDeviceConfigSpecOperationAdd, | ||
}) | ||
ctx.Logger.V(4).Info("created vGPU device", "vgpu-profile", vGPUDevice.ProfileName) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Should we have a smilar message for the gpu case? |
||
deviceKey-- | ||
} | ||
return deviceSpecs | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,6 @@ | ||
apiVersion: kustomize.config.k8s.io/v1beta1 | ||
kind: Kustomization | ||
resources: | ||
- ../base | ||
patchesStrategicMerge: | ||
- vgpu-device-template.yaml |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
--- | ||
apiVersion: infrastructure.cluster.x-k8s.io/v1beta1 | ||
kind: VSphereMachineTemplate | ||
metadata: | ||
name: ${CLUSTER_NAME}-worker | ||
namespace: ${NAMESPACE} | ||
spec: | ||
template: | ||
spec: | ||
vgpuDevices: | ||
- profileName: ${PROFILE_NAME} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's omit the stdout output