modernize: simplify code by using modern constructs

build.mk

@@ -147,9 +147,11 @@ lint:
     ifeq ($(CI),true)
 		@echo "Running in GitHub Actions"
 		golangci-lint run -v
+		go run golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize@latest -test ./...
     else
 		@echo "Running in local environment"
 		golangci-lint run -v --fix
+		go run golang.org/x/tools/gopls/internal/analysis/modernize/cmd/modernize@latest -test --fix ./...
     endif
 
 .PHONY: lint-windows

cmd/cni/cni.go

@@ -29,7 +29,7 @@ func main() {
 		Add: cmdAdd,
 		Del: cmdDel,
 	}
-	about := fmt.Sprintf("CNI kube-ovn plugin %s", versions.VERSION)
+	about := "CNI kube-ovn plugin " + versions.VERSION
 	skel.PluginMainFuncs(funcs, version.All, about)
 }
 
@@ -111,7 +111,7 @@ func generateCNIResult(cniResponse *request.CniResponse, netns string) current.R
 		var netMask *net.IPNet
 		var gwStr string
 		addRoutes := len(result.Routes) == 0
-		for _, cidrBlock := range strings.Split(cniResponse.CIDR, ",") {
+		for cidrBlock := range strings.SplitSeq(cniResponse.CIDR, ",") {
 			_, netMask, _ = net.ParseCIDR(cidrBlock)
 			gwStr = ""
 			if util.CheckProtocol(cidrBlock) == kubeovnv1.ProtocolIPv4 {
@@ -227,16 +227,16 @@ func parseValueFromArgs(key, argString string) (string, error) {
 	if argString == "" {
 		return "", types.NewError(types.ErrInvalidNetworkConfig, "Invalid Configuration", "CNI_ARGS is required")
 	}
-	args := strings.Split(argString, ";")
-	for _, arg := range args {
-		if strings.HasPrefix(arg, fmt.Sprintf("%s=", key)) {
-			value := strings.TrimPrefix(arg, fmt.Sprintf("%s=", key))
+	args := strings.SplitSeq(argString, ";")
+	for arg := range args {
+		if strings.HasPrefix(arg, key+"=") {
+			value := strings.TrimPrefix(arg, key+"=")
 			if len(value) > 0 {
 				return value, nil
 			}
 		}
 	}
-	return "", types.NewError(types.ErrInvalidNetworkConfig, "Invalid Configuration", fmt.Sprintf("%s is required in CNI_ARGS", key))
+	return "", types.NewError(types.ErrInvalidNetworkConfig, "Invalid Configuration", key+" is required in CNI_ARGS")
 }
 
 func assignV4Address(ipAddress, gateway string, mask *net.IPNet) (*current.IPConfig, *types.Route) {

cmd/daemon/cniserver.go

@@ -2,7 +2,6 @@ package main
 
 import (
 	"errors"
-	"fmt"
 	"net"
 	"net/http"
 	"net/http/pprof"
@@ -81,7 +80,7 @@ func main() {
 	stopCh := ctx.Done()
 	podInformerFactory := kubeinformers.NewSharedInformerFactoryWithOptions(config.KubeClient, 0,
 		kubeinformers.WithTweakListOptions(func(listOption *v1.ListOptions) {
-			listOption.FieldSelector = fmt.Sprintf("spec.nodeName=%s", config.NodeName)
+			listOption.FieldSelector = "spec.nodeName=" + config.NodeName
 			listOption.AllowWatchBookmarks = true
 		}))
 	nodeInformerFactory := kubeinformers.NewSharedInformerFactoryWithOptions(config.KubeClient, 0,

pkg/apis/kubeovn/v1/condition.go

@@ -41,10 +41,10 @@ type Condition struct {
 	ObservedGeneration int64 `json:"observedGeneration,omitempty"`
 	// Last time the condition was probed
 	// +optional
-	LastUpdateTime metav1.Time `json:"lastUpdateTime,omitempty"`
+	LastUpdateTime metav1.Time `json:"lastUpdateTime"`
 	// Last time the condition transitioned from one status to another.
 	// +optional
-	LastTransitionTime metav1.Time `json:"lastTransitionTime,omitempty"`
+	LastTransitionTime metav1.Time `json:"lastTransitionTime"`
 }
 
 type Conditions []Condition

pkg/apis/kubeovn/v1/ip.go

@@ -18,7 +18,7 @@ type IPList struct {
 // +genclient:nonNamespaced
 type IP struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec IPSpec `json:"spec"`
 }

pkg/apis/kubeovn/v1/ippool.go

@@ -24,10 +24,10 @@ type IPPoolList struct {
 // +genclient:nonNamespaced
 type IPPool struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   IPPoolSpec   `json:"spec"`
-	Status IPPoolStatus `json:"status,omitempty"`
+	Status IPPoolStatus `json:"status"`
 }
 
 type IPPoolSpec struct {

pkg/apis/kubeovn/v1/iptables-dnat-rule.go

@@ -22,10 +22,10 @@ type IptablesDnatRuleList struct {
 // +resourceName=iptables-dnat-rules
 type IptablesDnatRule struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   IptablesDnatRuleSpec   `json:"spec"`
-	Status IptablesDnatRuleStatus `json:"status,omitempty"`
+	Status IptablesDnatRuleStatus `json:"status"`
 }
 
 type IptablesDnatRuleSpec struct {

pkg/apis/kubeovn/v1/iptables-eip.go

@@ -22,10 +22,10 @@ type IptablesEIPList struct {
 // +resourceName=iptables-eips
 type IptablesEIP struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   IptablesEIPSpec   `json:"spec"`
-	Status IptablesEIPStatus `json:"status,omitempty"`
+	Status IptablesEIPStatus `json:"status"`
 }
 type IptablesEIPSpec struct {
 	V4ip           string `json:"v4ip"`

pkg/apis/kubeovn/v1/iptables-fip-rule.go

@@ -22,10 +22,10 @@ type IptablesFIPRuleList struct {
 // +resourceName=iptables-fip-rules
 type IptablesFIPRule struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   IptablesFIPRuleSpec   `json:"spec"`
-	Status IptablesFIPRuleStatus `json:"status,omitempty"`
+	Status IptablesFIPRuleStatus `json:"status"`
 }
 type IptablesFIPRuleSpec struct {
 	EIP        string `json:"eip"`

pkg/apis/kubeovn/v1/iptables-snat-rule.go

@@ -22,10 +22,10 @@ type IptablesSnatRuleList struct {
 // +resourceName=iptables-snat-rules
 type IptablesSnatRule struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   IptablesSnatRuleSpec   `json:"spec"`
-	Status IptablesSnatRuleStatus `json:"status,omitempty"`
+	Status IptablesSnatRuleStatus `json:"status"`
 }
 
 type IptablesSnatRuleSpec struct {

pkg/apis/kubeovn/v1/ovn-dnat-rule.go

@@ -22,10 +22,10 @@ type OvnDnatRuleList struct {
 // +resourceName=ovn-dnat-rules
 type OvnDnatRule struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   OvnDnatRuleSpec   `json:"spec"`
-	Status OvnDnatRuleStatus `json:"status,omitempty"`
+	Status OvnDnatRuleStatus `json:"status"`
 }
 
 type OvnDnatRuleSpec struct {

pkg/apis/kubeovn/v1/ovn-eip.go

@@ -22,10 +22,10 @@ type OvnEipList struct {
 // +resourceName=ovn-eips
 type OvnEip struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   OvnEipSpec   `json:"spec"`
-	Status OvnEipStatus `json:"status,omitempty"`
+	Status OvnEipStatus `json:"status"`
 }
 type OvnEipSpec struct {
 	ExternalSubnet string `json:"externalSubnet"`

pkg/apis/kubeovn/v1/ovn-fip.go

@@ -22,10 +22,10 @@ type OvnFipList struct {
 // +resourceName=ovn-fips
 type OvnFip struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   OvnFipSpec   `json:"spec"`
-	Status OvnFipStatus `json:"status,omitempty"`
+	Status OvnFipStatus `json:"status"`
 }
 type OvnFipSpec struct {
 	OvnEip string `json:"ovnEip"`

pkg/apis/kubeovn/v1/ovn-snat-rule.go

@@ -22,10 +22,10 @@ type OvnSnatRuleList struct {
 // +resourceName=ovn-snat-rules
 type OvnSnatRule struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   OvnSnatRuleSpec   `json:"spec"`
-	Status OvnSnatRuleStatus `json:"status,omitempty"`
+	Status OvnSnatRuleStatus `json:"status"`
 }
 
 type OvnSnatRuleSpec struct {

pkg/apis/kubeovn/v1/provider-network.go

@@ -1,6 +1,8 @@
 package v1
 
 import (
+	"slices"
+
 	corev1 "k8s.io/api/core/v1"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
@@ -19,7 +21,7 @@ type ProviderNetworkList struct {
 // +resourceName=provider-networks
 type ProviderNetwork struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   ProviderNetworkSpec   `json:"spec"`
 	Status ProviderNetworkStatus `json:"status"`
@@ -201,7 +203,7 @@ func (s *ProviderNetworkStatus) RemoveNodeConditions(node string) bool {
 	for i := 0; i < len(s.Conditions); {
 		if s.Conditions[i].Node == node {
 			changed = true
-			s.Conditions = append(s.Conditions[:i], s.Conditions[i+1:]...)
+			s.Conditions = slices.Delete(s.Conditions, i, i+1)
 		} else {
 			i++
 		}

pkg/apis/kubeovn/v1/qos-policy.go

@@ -41,10 +41,10 @@ type QoSPolicyList struct {
 // +resourceName=qos-policies
 type QoSPolicy struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   QoSPolicySpec   `json:"spec"`
-	Status QoSPolicyStatus `json:"status,omitempty"`
+	Status QoSPolicyStatus `json:"status"`
 }
 type QoSPolicySpec struct {
 	BandwidthLimitRules QoSPolicyBandwidthLimitRules `json:"bandwidthLimitRules"`

pkg/apis/kubeovn/v1/register_test.go

@@ -81,7 +81,7 @@ func TestAddResources(t *testing.T) {
 		}
 
 		var hasTypeMeta, hasObjMeta, hasListMeta bool
-		for i := 0; i < st.NumFields(); i++ {
+		for i := range st.NumFields() {
 			v := st.Field(i)
 			if !v.Embedded() || !v.Exported() {
 				continue

pkg/apis/kubeovn/v1/security-group.go

@@ -47,7 +47,7 @@ type SecurityGroupList struct {
 // +resourceName=security-groups
 type SecurityGroup struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   SecurityGroupSpec   `json:"spec"`
 	Status SecurityGroupStatus `json:"status"`

pkg/apis/kubeovn/v1/subnet.go

@@ -33,10 +33,10 @@ type SubnetList struct {
 // +genclient:nonNamespaced
 type Subnet struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   SubnetSpec   `json:"spec"`
-	Status SubnetStatus `json:"status,omitempty"`
+	Status SubnetStatus `json:"status"`
 }
 
 type SubnetSpec struct {

pkg/apis/kubeovn/v1/switch-lb-rule.go

@@ -18,10 +18,10 @@ type SwitchLBRuleList struct {
 // +resourceName=switch-lb-rules
 type SwitchLBRule struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   SwitchLBRuleSpec   `json:"spec"`
-	Status SwitchLBRuleStatus `json:"status,omitempty"`
+	Status SwitchLBRuleStatus `json:"status"`
 }
 
 type SwitchLBRuleSpec struct {

pkg/apis/kubeovn/v1/vip.go

@@ -21,10 +21,10 @@ type VipList struct {
 // +genclient:nonNamespaced
 type Vip struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   VipSpec   `json:"spec"`
-	Status VipStatus `json:"status,omitempty"`
+	Status VipStatus `json:"status"`
 }
 
 type VipSpec struct {

pkg/apis/kubeovn/v1/vlan.go

@@ -18,7 +18,7 @@ type VlanList struct {
 // +genclient:nonNamespaced
 type Vlan struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   VlanSpec   `json:"spec"`
 	Status VlanStatus `json:"status"`

pkg/apis/kubeovn/v1/vpc-dns.go

@@ -18,10 +18,10 @@ type VpcDnsList struct {
 // +resourceName=vpc-dnses
 type VpcDns struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   VpcDNSSpec   `json:"spec"`
-	Status VpcDNSStatus `json:"status,omitempty"`
+	Status VpcDNSStatus `json:"status"`
 }
 
 type VpcDNSSpec struct {

pkg/apis/kubeovn/v1/vpc-egress-gateway.go

@@ -33,10 +33,10 @@ type VpcEgressGatewayList struct {
 // vpc egress gateway is used to forward the egress traffic from the VPC to the external network
 type VpcEgressGateway struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   VpcEgressGatewaySpec   `json:"spec"`
-	Status VpcEgressGatewayStatus `json:"status,omitempty"`
+	Status VpcEgressGatewayStatus `json:"status"`
 }
 
 // Ready returns true if the VpcEgressGateway has been processed successfully and is ready to serve traffic
@@ -68,7 +68,7 @@ type VpcEgressGatewaySpec struct {
 	ExternalIPs []string `json:"externalIPs,omitempty"`
 
 	// BFD configuration
-	BFD VpcEgressGatewayBFDConfig `json:"bfd,omitempty"`
+	BFD VpcEgressGatewayBFDConfig `json:"bfd"`
 	// egress policies
 	// at least one policy must be specified
 	Policies []VpcEgressGatewayPolicy `json:"policies,omitempty"`
@@ -116,7 +116,7 @@ type VpcEgressGatewayStatus struct {
 	Conditions  Conditions `json:"conditions,omitempty"`
 
 	// workload information
-	Workload VpcEgressWorkload `json:"workload,omitempty"`
+	Workload VpcEgressWorkload `json:"workload"`
 }
 
 type VpcEgressWorkload struct {

pkg/apis/kubeovn/v1/vpc-nat-gateway.go

@@ -23,10 +23,10 @@ type VpcNatGatewayList struct {
 // +resourceName=vpc-nat-gateways
 type VpcNatGateway struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   VpcNatGatewaySpec   `json:"spec"`
-	Status VpcNatGatewayStatus `json:"status,omitempty"`
+	Status VpcNatGatewayStatus `json:"status"`
 }
 
 type VpcNatGatewaySpec struct {

pkg/apis/kubeovn/v1/vpc.go

@@ -38,10 +38,10 @@ type VpcList struct {
 // +genclient:nonNamespaced
 type Vpc struct {
 	metav1.TypeMeta   `json:",inline"`
-	metav1.ObjectMeta `json:"metadata,omitempty"`
+	metav1.ObjectMeta `json:"metadata"`
 
 	Spec   VpcSpec   `json:"spec"`
-	Status VpcStatus `json:"status,omitempty"`
+	Status VpcStatus `json:"status"`
 }
 
 type VpcSpec struct {