BUILD/MEDIUM: deps: update client-native to v6.4.2

The models package no longer ships hand-written compare helpers: Equal and
Diff methods are now generated by go-method-gen and take GoMethodGenOptions
instead of the removed models.Options. NilSameAsEmpty is gone; treating nil
as empty is the new default, so the explicit option was dropped.

Two behavioral changes required adjustments:

- BindParams lost its Name field, which moved to the parent Bind struct,
  so bind literals in handlers, gateways and controller were restructured.

- The generated Diff no longer treats nil as equal to an empty struct
  pointer. Empty TuneOptions/TuneSslOptions injected by the controller are
  not persisted in the config file, so they kept re-appearing as a diff and
  caused a reload on every sync. Both sides are now normalized to nil when
  empty before diffing.

The CR tests had to follow the API changes: ACL parsing returns models.ACL
with an ACLName field instead of types.ACL, and the CR deploy fixtures lost
the removed http-use-htx and http_proxy fields.

The backend CRD is the only one rewritten by yq, and the CI image ships a
yq whose emitter keeps long single-line descriptions on one line, while
newer local yq versions re-fold them at 80 columns. The CRDs are committed
with the folding as produced by the CI toolchain.

Regenerated the v3 CRDs, which drops two options removed from client-native
(http-use-htx and http_proxy, both deprecated HAProxy settings) and adds the
new v6.4.2 fields; everything else is additive.

Author: oktalz

crs/api/ingress/v1/tcp.go

@@ -200,39 +200,27 @@ func (a TCPModel) Equal(b TCPModel, opt ...models.Options) bool {
 		}
 	}
 
-	if !a.Frontend.Acls.Equal(b.Frontend.Acls, models.Options{
-		NilSameAsEmpty: true,
-	}) {
+	if !a.Frontend.Acls.Equal(b.Frontend.Acls) {
 		return false
 	}
 
-	if !a.Frontend.BackendSwitchingRules.Equal(b.Frontend.BackendSwitchingRules, models.Options{
-		NilSameAsEmpty: true,
-	}) {
+	if !a.Frontend.BackendSwitchingRules.Equal(b.Frontend.BackendSwitchingRules) {
 		return false
 	}
 
-	if !a.Frontend.Captures.Equal(b.Frontend.Captures, models.Options{
-		NilSameAsEmpty: true,
-	}) {
+	if !a.Frontend.Captures.Equal(b.Frontend.Captures) {
 		return false
 	}
 
-	if !a.Frontend.Filters.Equal(b.Frontend.Filters, models.Options{
-		NilSameAsEmpty: true,
-	}) {
+	if !a.Frontend.Filters.Equal(b.Frontend.Filters) {
 		return false
 	}
 
-	if !a.Frontend.LogTargets.Equal(b.Frontend.LogTargets, models.Options{
-		NilSameAsEmpty: true,
-	}) {
+	if !a.Frontend.LogTargets.Equal(b.Frontend.LogTargets) {
 		return false
 	}
 
-	if !a.Frontend.TCPRequestRules.Equal(b.Frontend.TCPRequestRules, models.Options{
-		NilSameAsEmpty: true,
-	}) {
+	if !a.Frontend.TCPRequestRules.Equal(b.Frontend.TCPRequestRules) {
 		return false
 	}
 

crs/api/ingress/v3/backend.go

@@ -22,7 +22,7 @@ import (
 
 // +genclient
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
-// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.2.4"
+// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.4.2"
 
 // Backend is a specification for a Backend resource
 type Backend struct {

crs/api/ingress/v3/defaults.go

@@ -22,7 +22,7 @@ import (
 
 // +genclient
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
-// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.2.4"
+// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.4.2"
 
 // Defaults is a specification for a Defaults resource
 type Defaults struct {

crs/api/ingress/v3/frontend.go

@@ -22,7 +22,7 @@ import (
 
 // +genclient
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
-// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.2.4"
+// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.4.2"
 
 // Frontend is a specification for a Frontend resource
 type Frontend struct {

crs/api/ingress/v3/global.go

@@ -22,7 +22,7 @@ import (
 
 // +genclient
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
-// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.2.4"
+// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.4.2"
 // +kubebuilder:validation:XValidation:rule="!has(self.spec.default_path)", message="spec.config.default_path is set by ingress controller internally"
 // +kubebuilder:validation:XValidation:rule="!has(self.spec.master__dash__worker)", message="spec.config.master-worker is set by ingress controller internally"
 // +kubebuilder:validation:XValidation:rule="!has(self.spec.pidfile)", message="spec.config.pidfile is set by ingress controller internally"

crs/api/ingress/v3/tcp.go

@@ -18,12 +18,13 @@ package v3
 import (
 	"github.com/go-openapi/swag/jsonutils"
 	"github.com/haproxytech/client-native/v6/models"
+	"github.com/haproxytech/go-method-gen/pkg/eqdiff"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
 // +genclient
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
-// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.2.4"
+// +kubebuilder:metadata:annotations="haproxy.org/client-native=v6.4.2"
 
 // TCP is a specification for a TCP resource
 type TCP struct {
@@ -100,7 +101,7 @@ func (s *TCPService) UnmarshalBinary(b []byte) error {
 	return nil
 }
 
-func (a TCPModel) Equal(b TCPModel, opt ...models.Options) bool {
+func (a TCPModel) Equal(b TCPModel, opt ...eqdiff.GoMethodGenOptions) bool {
 	if a.Name != b.Name {
 		return false
 	}
@@ -134,6 +135,6 @@ func (a TCPModel) Equal(b TCPModel, opt ...models.Options) bool {
 	return true
 }
 
-func (s TCPService) Equal(b TCPService, opt ...models.Options) bool {
+func (s TCPService) Equal(b TCPService, opt ...eqdiff.GoMethodGenOptions) bool {
 	return s.Name == b.Name && s.Port == b.Port
 }