Merge pull request #18 from sgopinath1/gpuai-5189

Remove CDI file on uninstalling package

Author: sgopinath1

cmd/amd-ctk/runtime/configure/configure.go

@@ -26,11 +26,10 @@ import (
 )
 
 const (
-	defaultRuntime                  = "docker"
-	defaultAmdRuntimeName           = "amd"
-	defaultAmdRuntimeExecutable     = "amd-container-runtime"
-	defaultAmdRuntimeExecutablePath = "/usr/bin/amd-container-runtime"
-	defaultDockerConfigFilePath     = "/etc/docker/daemon.json"
+	defaultRuntime              = "docker"
+	defaultAmdRuntimeName       = "amd"
+	defaultAmdRuntimeExecutable = "amd-container-runtime"
+	defaultDockerConfigFilePath = "/etc/docker/daemon.json"
 )
 
 type configOptions struct {

cmd/amd-ctk/runtime/engine/docker/docker.go

@@ -24,9 +24,10 @@ import (
 )
 
 const (
-	runtimesKey       = "runtimes"
-	defaultRuntimeKey = "default-runtime"
-	featuresKey       = "features"
+	runtimesKey        = "runtimes"
+	defaultRuntimeKey  = "default-runtime"
+	featuresKey        = "features"
+	defaultCDISpecPath = "/etc/cdi"
 )
 
 type dockerConfig map[string]interface{}
@@ -115,6 +116,8 @@ func (d *dockerConfig) UnsetDefaultRuntime() error {
 // an error and a do not update flag in case daemon.json doesn't need
 // to be updated
 func (d *dockerConfig) RemoveRuntime(name string) (error, bool) {
+	_ = os.RemoveAll(defaultCDISpecPath)
+
 	if d == nil {
 		return fmt.Errorf("configuration is empty"), true
 	}