Comments

client/driver/executor/executor.go

@@ -120,7 +120,7 @@ type ExecCommand struct {
 	ResourceLimits bool
 
 	// Cgroup marks whether we put the process in a cgroup. Setting this field
-	// doesn't enforce resource limits. To enforce limits, set ResoruceLimits.
+	// doesn't enforce resource limits. To enforce limits, set ResourceLimits.
 	// Using the cgroup does allow more precise cleanup of processes.
 	BasicProcessCgroup bool
 }

client/driver/raw_exec_test.go

@@ -265,6 +265,9 @@ func TestRawExecDriver_Start_Kill_Wait(t *testing.T) {
 	}
 }
 
+// This test creates a process tree such that without cgroups tracking the
+// processes cleanup of the children would not be possible. Thus the test
+// asserts that the processes get killed properly when using cgroups.
 func TestRawExecDriver_Start_Kill_Wait_Cgroup(t *testing.T) {
 	tu.ExecCompatible(t)
 	t.Parallel()