Merge pull request #26 from solo-io/disableKubeMarkers-description

Fix disableKubeMarkers description

Author: arianaw66

README.md

@@ -75,4 +75,4 @@ Other supported options are:
 *  `int_native`
     *   when set to `true`, the native openapi schemas will be used for Integer types instead of Solo wrappers that add Kubernetes extension headers to the schema to treat int as strings.
 *  `disable_kube_markers`
-    *   when set to `true`, kubebuilder markers and validations such as PreserveUnknownFields, Required, default, and all CEL rules will be omitted from the OpenAPI schema. The Type marker will be maintained.
+    *   when set to `true`, kubebuilder markers and validations such as PreserveUnknownFields, MinItems, default, and all CEL rules will be omitted from the OpenAPI schema. The Type and Required markers will be maintained.

changelog/v0.2.4/disableKubeMarkers-description.yaml

@@ -0,0 +1,6 @@
+changelog:
+  - type: FIX
+    issueLink: https://github.com/solo-io/gloo-mesh-enterprise/issues/17005
+    resolvesIssue: false
+    description: |
+      Fixes the description of the disable_kube_markers option to correctly state which markers are respected and which are disabled.

openapiGenerator.go

@@ -118,8 +118,8 @@ type openapiGenerator struct {
 
 	markerRegistry *markers.Registry
 
-	// If set to true, kubebuilder markers and validations such as PreserveUnknownFields, Required, default, and all CEL rules will be omitted from the OpenAPI schema.
-	// The Type marker will be maintained.
+	// If set to true, kubebuilder markers and validations such as PreserveUnknownFields, MinItems, default, and all CEL rules will be omitted from the OpenAPI schema.
+	// The Type and Required markers will be maintained.
 	disableKubeMarkers bool
 }