OPTIM/MAJOR: go: Optimize struct field alignments

Author: dkorunic

pkg/annotations/cfgSnippet.go

@@ -28,18 +28,18 @@ const (
 )
 
 type CfgSnippet struct {
+	ingress  *store.Ingress
+	service  *store.Service
 	name     string
 	frontend string
 	backend  string
-	ingress  *store.Ingress
-	service  *store.Service
 }
 
 type cfgData struct {
+	status   store.Status
 	value    []string
 	updated  []string
 	disabled bool
-	status   store.Status
 }
 
 // CfgSnippetType represents type of a config snippet
@@ -82,10 +82,10 @@ func InitCfgSnippet() {
 }
 
 type ConfigSnippetOptions struct {
-	Name     string
 	Backend  *string
 	Frontend *string
 	Ingress  *store.Ingress
+	Name     string
 }
 
 // DisableConfigSnippets fills a map[cfgSnippetType]struct{} of disabled config snippet types:

pkg/controller/builder.go

@@ -43,20 +43,20 @@ import (
 )
 
 type Builder struct {
+	store                    store.K8s
 	annotations              annotations.Annotations
 	haproxyClient            api.HAProxyClient
 	gatewayManager           gateway.GatewayManager
 	haproxyProcess           process.Process
 	haproxyRules             rules.Rules
 	restClientSet            client.Client
+	updateStatusManager      status.UpdateStatusManager
 	updatePublishServiceFunc func(ingresses []*ingress.Ingress, publishServiceAddresses []string)
 	eventChan                chan k8s.SyncDataEvent
 	clientSet                *kubernetes.Clientset
-	haproxyEnv               env.Env
 	haproxyCfgFile           []byte
-	store                    store.K8s
+	haproxyEnv               env.Env
 	osArgs                   utils.OSArgs
-	updateStatusManager      status.UpdateStatusManager
 }
 
 var defaultEnv = env.Env{

pkg/controller/controller.go

@@ -40,21 +40,21 @@ var logger = utils.GetLogger()
 
 // HAProxyController is ingress controller
 type HAProxyController struct {
+	store                    store.K8s
 	gatewayManager           gateway.GatewayManager
 	annotations              annotations.Annotations
+	updateStatusManager      status.UpdateStatusManager
 	eventChan                chan k8s.SyncDataEvent
 	updatePublishServiceFunc func(ingresses []*ingress.Ingress, publishServiceAddresses []string)
 	chShutdown               chan struct{}
 	podNamespace             string
 	podPrefix                string
-	haproxy                  haproxy.HAProxy
 	updateHandlers           []UpdateHandler
-	store                    store.K8s
+	beforeUpdateHandlers     []UpdateHandler
+	haproxy                  haproxy.HAProxy
 	osArgs                   utils.OSArgs
 	auxCfgModTime            int64
 	ready                    bool
-	updateStatusManager      status.UpdateStatusManager
-	beforeUpdateHandlers     []UpdateHandler
 }
 
 // Wrapping a Native-Client transaction and commit it.

pkg/handler/quic.go

@@ -21,12 +21,12 @@ const (
 type Quic struct {
 	AddrIPv4         string
 	AddrIPv6         string
-	IPv4             bool
-	IPv6             bool
 	CertDir          string
-	QuicAnnouncePort int64
 	MaxAge           string
+	QuicAnnouncePort int64
 	QuicBindPort     int64
+	IPv4             bool
+	IPv6             bool
 }
 
 func (q *Quic) Update(k store.K8s, h haproxy.HAProxy, a annotations.Annotations) (err error) {

pkg/haproxy/api/runtime.go

@@ -19,8 +19,8 @@ type RuntimeServerData struct {
 	BackendName string
 	ServerName  string
 	IP          string
-	Port        int
 	State       string
+	Port        int
 }
 
 func (c *clientNative) ExecuteRaw(command string) (result []string, err error) {

pkg/haproxy/instance/configuration.go

@@ -42,8 +42,8 @@ func NeedAction() bool {
 }
 
 type configurationManagerImpl struct {
-	reload, restart bool
 	logger          utils.Logger
+	reload, restart bool
 }
 
 func NewConfigurationManager() *configurationManagerImpl {

pkg/k8s/main.go

@@ -76,13 +76,13 @@ type k8s struct {
 	apiExtensionsClient    *crdclientset.Clientset
 	publishSvc             *utils.NamespaceValue
 	gatewayClient          *gatewayclientset.Clientset
+	crdClient              *crdclientset.Clientset
 	podPrefix              string
 	podNamespace           string
 	whiteListedNS          []string
 	syncPeriod             time.Duration
 	cacheResyncPeriod      time.Duration
 	disableSvcExternalName bool // CVE-2021-25740
-	crdClient              *crdclientset.Clientset
 	gatewayAPIInstalled    bool
 }
 

pkg/service/service.go

@@ -35,17 +35,17 @@ var logger = utils.GetLogger()
 const cookieKey = "ohph7OoGhong"
 
 type Service struct {
-	path        *store.IngressPath
-	resource    *store.Service
-	backend     *models.Backend
-	certs       certs.Certificates
+	certs    certs.Certificates
+	path     *store.IngressPath
+	resource *store.Service
+	backend  *models.Backend
+	// ingressName      string
+	// ingressNamespace string
+	ingress     *store.Ingress
 	annotations []map[string]string
 	modeTCP     bool
 	newBackend  bool
 	standalone  bool
-	// ingressName      string
-	// ingressNamespace string
-	ingress *store.Ingress
 }
 
 // New returns a Service instance to handle the k8s IngressPath resource given in params.

pkg/status/updatestatus.go

@@ -17,9 +17,9 @@ type UpdateStatusManager interface {
 }
 
 type UpdateStatusManagerImpl struct {
-	updateIngresses   []*ingress.Ingress
 	client            *kubernetes.Clientset
 	ingressClass      string
+	updateIngresses   []*ingress.Ingress
 	emptyIngressClass bool
 }
 

pkg/store/store.go

@@ -31,11 +31,11 @@ type K8s struct {
 	SecretsProcessed             map[string]struct{}
 	BackendsProcessed            map[string]struct{}
 	GatewayClasses               map[string]*GatewayClass
+	HaProxyPods                  map[string]struct{}
+	BackendsWithNoConfigSnippets map[string]struct{}
 	GatewayControllerName        string
 	PublishServiceAddresses      []string
-	HaProxyPods                  map[string]struct{}
 	UpdateAllIngresses           bool
-	BackendsWithNoConfigSnippets map[string]struct{}
 }
 
 type NamespacesWatch struct {