Skip to content

Commit f6735fa

Browse files
authored
bug fix: podSets should not be required (#59)
If a Component does not contain any PodSpecTemplates, then PodSet is not required.
1 parent 9833c86 commit f6735fa

File tree

2 files changed

+1
-2
lines changed

2 files changed

+1
-2
lines changed

api/v1beta2/appwrapper_types.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ type AppWrapperSpec struct {
3434
// AppWrapperComponent describes a wrapped resource
3535
type AppWrapperComponent struct {
3636
// PodSets contained in the component
37-
PodSets []AppWrapperPodSet `json:"podSets"`
37+
PodSets []AppWrapperPodSet `json:"podSets,omitempty"`
3838

3939
// PodSetInfos assigned to the Component by Kueue
4040
PodSetInfos []AppWrapperPodSetInfo `json:"podSetInfos,omitempty"`

config/crd/bases/workload.codeflare.dev_appwrappers.yaml

-1
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,6 @@ spec:
127127
x-kubernetes-embedded-resource: true
128128
x-kubernetes-preserve-unknown-fields: true
129129
required:
130-
- podSets
131130
- template
132131
type: object
133132
type: array

0 commit comments

Comments
 (0)