The default value for CleanPodPolicy is inconsistent.

[Syulin7]

In the comments of common, the default value is Running：

type RunPolicy struct {
	// CleanPodPolicy defines the policy to kill pods after the job completes.
	// Default to Running.
	CleanPodPolicy *CleanPodPolicy `json:"cleanPodPolicy,omitempty"`

In actuality：

• mpiJob's default cleanpod policy is None.
• mxJob's default cleanpod policy is All.
• paddleJob's default cleanpod policy is None.
• pytorchJob's default cleanpod policy is None.
• tfJob's default cleanpod policy is Running.
• XGBoostJob's default cleanpod policy is All.

Should we set all jobs default to Running, or remove the comments(// Default to Running.) in common.

[johnugeorge]

Thanks for reporting this.

Yes. This has to be consistent. I think, we can keep defaults to None IMO.

/cc @terrytangyuan @gaocegege @tenzen-y

[tenzen-y]

I agree. We should set None as default. Setting Running as default looks weird to me.

[terrytangyuan]

Sounds good