doc: add global maxRetry option

[dongjiang1989]

add global jobRetry option

ref: #3838

[hwdef]

/lgtm

[dongjiang1989]

Hi @hwdef When will volcano/apis be released? Use new apis version to generate new CRD.

[hwdef]

sorry I do not know about this.
Please ask @Monokaix

[Monokaix]

Hi @hwdef When will volcano/apis be released? Use new apis version to generate new CRD.

Can you tell me which CRD are you going to use?

[dongjiang1989]

Hi @hwdef When will volcano/apis be released? Use new apis version to generate new CRD.

Can you tell me which CRD are you going to use?

ref：volcano-sh/apis#145

[hwdef]

/lgtm cancel

[hwdef]

I prefer the following definition

type Flow struct {
	// +kubebuilder:validation:MinLength=1
	// +required
	Name string `json:"name"`
	// +optional
	DependsOn *DependsOn `json:"dependsOn,omitempty"`
	// +optional
	MaxRetry *MaxRetry `json:"maxRetry,omitempty"`
}

[volcano-sh-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign hwdef
You can assign the PR to them by writing /assign @hwdef in a comment when ready.

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[dongjiang1989]

I prefer the following definition

type Flow struct {
	// +kubebuilder:validation:MinLength=1
	// +required
	Name string `json:"name"`
	// +optional
	DependsOn *DependsOn `json:"dependsOn,omitempty"`
	// +optional
	MaxRetry *MaxRetry `json:"maxRetry,omitempty"`
}

I agree. Change it to design done.
Thanks @hwdef

[hwdef]

@googs1025
What do you think of this?

[googs1025]

@googs1025 What do you think of this?

I'm ok with this.

[googs1025]

I think we should squash commit to one

[dongjiang1989]

I think we should squash commit to one

Thanks @googs1025 @hwdef
Squash commits done.
Please re-check it.

[dongjiang1989]

@hwdef Please add label to merge it, thanks

[dongjiang1989]

cc @Monokaix @googs1025 PTAL, recheck it

[hwdef]

/lgtm
/ok-to-test

[hwdef]

/lgtm

[hwdef]

/lgtm

[Monokaix]

We should also implement this in controller first?

[dongjiang1989]

The next PR to add this feature.

This PR to patch MaxRetry from jobflow to jobTemplate