tools/openapi2crd: use only required fields from plugin chain (#3007)

s-urbaniak · web-flow · commit 1715e52e4655 · 2025-12-09T15:35:07.000+01:00
diff --git a/tools/openapi2crd/pkg/plugins/read_write_properties.go b/tools/openapi2crd/pkg/plugins/read_write_properties.go
@@ -38,8 +38,7 @@ func (p *ReadWriteProperties) Process(req *PropertyProcessorRequest) error {
 		return nil
 	}
 
-	required := sets.New(req.OpenAPISchema.Required...)
-	required.Insert(req.Property.Required...)
+	required := sets.New(req.Property.Required...)
 	for name, prop := range req.OpenAPISchema.Properties {
 		if prop.Value.ReadOnly {
 			required.Delete(name)

Original file line number	Diff line number	Diff line change
`@@ -38,8 +38,7 @@ func (p ReadWriteProperties) Process(req PropertyProcessorRequest) error {`
`38`	`38`	`return nil`
`39`	`39`	`}`
`40`	`40`
`41`		`- required := sets.New(req.OpenAPISchema.Required...)`
`42`		`- required.Insert(req.Property.Required...)`
	`41`	`+ required := sets.New(req.Property.Required...)`
`43`	`42`	`for name, prop := range req.OpenAPISchema.Properties {`
`44`	`43`	`if prop.Value.ReadOnly {`
`45`	`44`	`required.Delete(name)`