Merge pull request #5726 from profnandaa/fix-cdi-config-on-windows

fix: cdi: add appdefaults.CDISpecDirs for windows
moby · Feb 13, 2025 · 280ebbf · 280ebbf
2 parents 1b15816 + db8d975
commit 280ebbf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/docs/cdi.md b/docs/cdi.md
@@ -8,6 +8,9 @@ name.
 Since BuildKit 0.20.0, you can access devices using the CDI interface. This
 allows you to use devices like GPUs in your builds.
 
+> [!NOTE]
+> CDI support is yet to be tested on Windows.
+
 ## Usage
 
 To use CDI with BuildKit, you need to create the [CDI configuration file](https://github.com/cncf-tags/container-device-interface/blob/main/SPEC.md#cdi-json-specification)

diff --git a/util/appdefaults/appdefaults_windows.go b/util/appdefaults/appdefaults_windows.go
@@ -18,7 +18,7 @@ var (
 
 var (
 	UserCNIConfigPath = DefaultCNIConfigPath
-	CDISpecDirs       []string
+	CDISpecDirs       = []string{filepath.Join(os.Getenv("ProgramData"), "buildkitd", "cdi")}
 )
 
 func UserAddress() string {