Merge pull request #39 from Roblox/env_path

Don't override $(PATH) environment variable.

Author: shishir-a412ed

containerd/driver.go

@@ -358,6 +358,9 @@ func (d *Driver) StartTask(cfg *drivers.TaskConfig) (*drivers.TaskHandle, *drive
 	// Setup environment variables.
 	var env []string
 	for key, val := range cfg.Env {
+		if skipOverride(key) {
+			continue
+		}
 		env = append(env, fmt.Sprintf("%s=%s", key, val))
 	}
 
@@ -407,6 +410,17 @@ func (d *Driver) StartTask(cfg *drivers.TaskConfig) (*drivers.TaskHandle, *drive
 	return handle, nil, nil
 }
 
+// skipOverride determines whether the environment variable (key) needs an override or not.
+func skipOverride(key string) bool {
+	skipOverrideList := []string{"PATH"}
+	for _, k := range skipOverrideList {
+		if key == k {
+			return true
+		}
+	}
+	return false
+}
+
 // RecoverTask recreates the in-memory state of a task from a TaskHandle.
 func (d *Driver) RecoverTask(handle *drivers.TaskHandle) error {
 	if handle == nil {