Add acceptable_codes to plan.run_sh and plan.exec

[tedim52]

Background & motivation

plan.exec has an acceptable_codes field that fails the starlark script if one of the acceptable_codes isn't set.

Desired behaviour

plan.run_sh and plan.run_python should have the field on it.

How important is this to you?

Painful; the lack of this feature makes using Kurtosis frictionful.

What area of the product does this pertain to?

Other: anything not covered by the above

[tedim52]

cc. @samlaf

[samlaf]

Not familiar at all with the kurtosis codebase, but just looking at

kurtosis/core/server/api_container/server/startosis_engine/kurtosis_instruction/tasks/run_sh.go

Line 298 in afe0d87

if createDefaultDirectoryResult.GetExitCode() != 0 {

doesn't this check for a non-zero error code and return an error? Maybe the error is not handled properly?