Skip to content

Commit cebc0eb

Browse files
committed
change by codeewview
Signed-off-by: dongjiang <dongjiang1989@126.com>
1 parent 08fef71 commit cebc0eb

File tree

6 files changed

+14
-100
lines changed

6 files changed

+14
-100
lines changed

pkg/apis/flow/v1alpha1/jobflow_types.go

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,6 @@ type JobFlowSpec struct {
3535
Flows []Flow `json:"flows,omitempty"`
3636
// +optional
3737
JobRetainPolicy RetainPolicy `json:"jobRetainPolicy,omitempty"`
38-
// Defaults to 1.
39-
//
40-
// +kubebuilder:default:=1
41-
// +kubebuilder:validation:Minimum=1
42-
// +optional
43-
MaxRetry *int64 `json:"maxRetry,omitempty"`
4438
}
4539

4640
// Flow defines the dependent of jobs
@@ -50,13 +44,12 @@ type Flow struct {
5044
Name string `json:"name"`
5145
// +optional
5246
DependsOn *DependsOn `json:"dependsOn,omitempty"`
47+
// Defaults to 1.
48+
//
49+
// +kubebuilder:default:=1
50+
// +kubebuilder:validation:Minimum=1
5351
// +optional
54-
Patch *Patch `json:"patch,omitempty"`
55-
}
56-
57-
type Patch struct {
58-
// +optional
59-
Spec *v1alpha1.JobSpec `json:"spec,omitempty"`
52+
MaxRetry *int64 `json:"maxRetry,omitempty"`
6053
}
6154

6255
type DependsOn struct {

pkg/apis/flow/v1alpha1/zz_generated.deepcopy.go

Lines changed: 4 additions & 30 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/client/applyconfiguration/flow/v1alpha1/flow.go

Lines changed: 5 additions & 5 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/client/applyconfiguration/flow/v1alpha1/jobflowspec.go

Lines changed: 0 additions & 9 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

pkg/client/applyconfiguration/flow/v1alpha1/patch.go

Lines changed: 0 additions & 42 deletions
This file was deleted.

pkg/client/applyconfiguration/utils.go

Lines changed: 0 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)