Add omitempty to dnsmanagementpolicy

Thejas N · Thejas N · commit a95768e387ce · 2022-08-26T20:23:24.000+05:30
Signed-off-by: Thejas N &lt;thn@redhat.com&gt;
diff --git a/openapi/generated_openapi/zz_generated.openapi.go b/openapi/generated_openapi/zz_generated.openapi.go
diff --git a/openapi/openapi.json b/openapi/openapi.json
@@ -23723,7 +23723,7 @@
         "dnsManagementPolicy": {
           "description": "dnsManagementPolicy indicates if the lifecycle of the wildcard DNS record associated with the load balancer service will be managed by the ingress operator. It defaults to Managed. Valid values are: Managed and Unmanaged.",
           "type": "string",
-          "default": ""
+          "default": "Managed"
         },
         "providerParameters": {
           "description": "providerParameters holds desired load balancer information specific to the underlying infrastructure provider.\n\nIf empty, defaults will be applied. See specific providerParameters fields for details about their defaults.",
@@ -26408,7 +26408,7 @@
         "dnsManagementPolicy": {
           "description": "dnsManagementPolicy denotes the current policy applied on the DNS record. Records that have policy set as \"Unmanaged\" are ignored by the ingress operator.  This means that the DNS record on the cloud provider is not managed by the operator, and the \"Published\" status condition will be updated to \"Unknown\" status, since it is externally managed. Any existing record on the cloud provider can be deleted at the discretion of the cluster admin.\n\nThis field defaults to Managed. Valid values are \"Managed\" and \"Unmanaged\".",
           "type": "string",
-          "default": ""
+          "default": "Managed"
         },
         "dnsName": {
           "description": "dnsName is the hostname of the DNS record",
diff --git a/operator/v1/types_ingress.go b/operator/v1/types_ingress.go
@@ -393,7 +393,8 @@ type LoadBalancerStrategy struct {
 	// +kubebuilder:default:="Managed"
 	// +kubebuilder:validation:Optional
 	// +optional
-	DNSManagementPolicy LoadBalancerDNSManagementPolicy `json:"dnsManagementPolicy"`
+	// +default="Managed"
+	DNSManagementPolicy LoadBalancerDNSManagementPolicy `json:"dnsManagementPolicy,omitempty"`
 }
 
 // LoadBalancerDNSManagementPolicy is a policy for configuring how
diff --git a/operatoringress/v1/types.go b/operatoringress/v1/types.go
@@ -72,7 +72,8 @@ type DNSRecordSpec struct {
 	// +kubebuilder:default:="Managed"
 	// +kubebuilder:validation:Optional
 	// +optional
-	DNSManagementPolicy DNSManagementPolicy `json:"dnsManagementPolicy"`
+	// +default="Managed"
+	DNSManagementPolicy DNSManagementPolicy `json:"dnsManagementPolicy,omitempty"`
 }
 
 // DNSRecordStatus is the most recently observed status of each record.

Original file line number	Diff line number	Diff line change
`@@ -393,7 +393,8 @@ type LoadBalancerStrategy struct {`
`393`	`393`	`// +kubebuilder:default:="Managed"`
`394`	`394`	`// +kubebuilder:validation:Optional`
`395`	`395`	`// +optional`
`396`		- DNSManagementPolicy LoadBalancerDNSManagementPolicy `json:"dnsManagementPolicy"`
	`396`	`+ // +default="Managed"`
	`397`	+ DNSManagementPolicy LoadBalancerDNSManagementPolicy `json:"dnsManagementPolicy,omitempty"`
`397`	`398`	`}`
`398`	`399`
`399`	`400`	`// LoadBalancerDNSManagementPolicy is a policy for configuring how`
Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,8 @@ type DNSRecordSpec struct {`
`72`	`72`	`// +kubebuilder:default:="Managed"`
`73`	`73`	`// +kubebuilder:validation:Optional`
`74`	`74`	`// +optional`
`75`		- DNSManagementPolicy DNSManagementPolicy `json:"dnsManagementPolicy"`
	`75`	`+ // +default="Managed"`
	`76`	+ DNSManagementPolicy DNSManagementPolicy `json:"dnsManagementPolicy,omitempty"`
`76`	`77`	`}`
`77`	`78`
`78`	`79`	`// DNSRecordStatus is the most recently observed status of each record.`