build-sys: updated golint to fix issues

Signed-off-by: Nagy Károly Gábriel <k@jpi.io>

Makefile

@@ -10,7 +10,7 @@ TOOLSDIR := $(CURDIR)/pkg/tools
 TMPDIR ?= $(CURDIR)/.tmp
 OUTDIR ?= $(TMPDIR)
 
-GOLANGCI_LINT_VERSION ?= v1.59.1
+GOLANGCI_LINT_VERSION ?= v1.60.0
 REVIVE_VERSION ?= v1.3.7
 
 GOLANGCI_LINT_URL ?= github.com/golangci/golangci-lint/cmd/golangci-lint@$(GOLANGCI_LINT_VERSION)

cmd/jpictl/dns.go

@@ -163,9 +163,9 @@ var dnsAddCmd = &cobra.Command{
 			addr, err := core.ParseAddr(s)
 			switch {
 			case err != nil:
-				return core.Wrap(err, s)
+				return core.Wrap(err, "cannot parse address %q", s)
 			case !addr.IsValid(), addr.IsUnspecified(), addr.IsPrivate(), addr.IsMulticast():
-				return core.Wrap(core.ErrInvalid, s)
+				return core.Wrap(core.ErrInvalid, "address %q is invalid", s)
 			default:
 				addrs = append(addrs, addr)
 			}

pkg/ceph/config_parser.go

@@ -54,7 +54,7 @@ func loadGlobalConfField(cfg *GlobalConfig, field basic.Field) error {
 		err := cfg.FSID.UnmarshalText([]byte(field.Value))
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "unmarshal failed on field %q", field.Key)
 		default:
 			return nil
 		}
@@ -64,7 +64,7 @@ func loadGlobalConfField(cfg *GlobalConfig, field basic.Field) error {
 			var addr netip.Addr
 
 			if err := addr.UnmarshalText([]byte(s)); err != nil {
-				return core.Wrap(err, field.Key)
+				return core.Wrap(err, "unmarshal failed on field %q", field.Key)
 			}
 
 			cfg.MonitorsAddr = append(cfg.MonitorsAddr, addr)
@@ -83,7 +83,7 @@ func loadGlobalConfField(cfg *GlobalConfig, field basic.Field) error {
 		err := cfg.ClusterNetwork.UnmarshalText([]byte(field.Value))
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "unmarshal failed on field %q", field.Key)
 		default:
 			return nil
 		}

pkg/cluster/cluster_scan.go

@@ -51,7 +51,7 @@ func (m *Cluster) scanDirectory(opts *ScanOptions) error {
 			ok, err := m.scanSubdirectory(opts, e.Name())
 			switch {
 			case err != nil:
-				return core.Wrap(err, e.Name())
+				return core.Wrap(err, "cannot scan directory %q", e.Name())
 			case !ok:
 				m.warn(nil).
 					WithField("zone", e.Name()).
@@ -220,7 +220,7 @@ func (z *Zone) loadRegions() error {
 		// parsed
 		err = z.appendRegions(regions...)
 		if err != nil {
-			err = core.Wrap(err, filename)
+			err = core.Wrap(err, "cannot append region from file %q", filename)
 		}
 	}
 
@@ -247,7 +247,7 @@ func (z *Zone) loadClusterToken() error {
 
 	err = z.zones.setRegionClusterToken(z.Name, token)
 	if err != nil {
-		err = core.Wrap(err, filename)
+		err = core.Wrap(err, "wrong cluster token in file %q", filename)
 	}
 
 	return err

pkg/cluster/machine_rings.go

@@ -36,7 +36,7 @@ func (m *Machine) GetWireguardKeys(ringID rings.RingID) (wireguard.KeyPair, erro
 	out.PrivateKey, err = wireguard.PrivateKeyFromBase64(string(data))
 	if err != nil {
 		// bad key
-		err = core.Wrap(err, keyFile)
+		err = core.Wrap(err, "bad private key in file %q", keyFile)
 		return out, err
 	}
 
@@ -52,7 +52,7 @@ func (m *Machine) GetWireguardKeys(ringID rings.RingID) (wireguard.KeyPair, erro
 		out.PublicKey, err = wireguard.PublicKeyFromBase64(string(data))
 		if err != nil {
 			// bad key
-			err = core.Wrap(err, pubFile)
+			err = core.Wrap(err, "bad public key in %q", pubFile)
 			return out, err
 		}
 	}
@@ -227,10 +227,10 @@ func (m *Machine) applyWireguardPeerConfig(ring rings.RingID,
 	switch {
 	case !found:
 		// unknown
-		return core.Wrap(ErrUnknownNode, pc.Endpoint.Host)
+		return core.Wrap(ErrUnknownNode, "peer %q not found", pc.Endpoint.Host)
 	case ring == 1 && m.zone != peer.zone:
 		// invalid zone
-		return core.Wrap(ErrInvalidNode, peer.Name)
+		return core.Wrap(ErrInvalidNode, "peer %q invalid zone", peer.Name)
 	default:
 		// apply RingInfo
 		ri := &RingInfo{

pkg/cluster/machine_scan.go

@@ -36,12 +36,12 @@ func (m *Machine) UpdatePublicAddresses() error {
 
 func (m *Machine) init() error {
 	if err := m.setID(); err != nil {
-		return core.Wrap(err, m.Name)
+		return core.Wrap(err, "cannot set ID for machine %q", m.Name)
 	}
 
 	for _, ring := range Rings {
 		if err := m.tryReadWireguardKeys(ring.ID); err != nil {
-			return core.Wrap(err, m.Name)
+			return core.Wrap(err, "cannot read keys for machine %q", m.Name)
 		}
 	}
 

pkg/dns/manager.go

@@ -30,7 +30,7 @@ type ManagerOption func(*Manager) error
 
 func newErrorManagerOption(err error, hint string) ManagerOption {
 	return func(*Manager) error {
-		return core.Wrap(err, hint)
+		return core.Wrap(err, "%q", hint)
 	}
 }
 

pkg/wireguard/config_parser.go

@@ -65,7 +65,7 @@ func loadInterfaceConfField(cfg *InterfaceConfig, field basic.Field) error {
 		err := cfg.Address.UnmarshalText([]byte(field.Value))
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "unmarshal failed on field %q", field.Key)
 		default:
 			return nil
 		}
@@ -77,7 +77,7 @@ func loadInterfaceConfField(cfg *InterfaceConfig, field basic.Field) error {
 		err := cfg.PrivateKey.UnmarshalText([]byte(field.Value))
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "unmarshal failed on field %q", field.Key)
 		default:
 			return nil
 		}
@@ -89,7 +89,7 @@ func loadInterfaceConfField(cfg *InterfaceConfig, field basic.Field) error {
 		u64, err := strconv.ParseUint(field.Value, 10, 16)
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "field %q is not an integer", field.Key)
 		case u64 == 0:
 			return core.Wrap(fs.ErrInvalid, "invalid %q value", field.Key)
 		default:
@@ -117,7 +117,7 @@ func loadPeerConfField(cfg *PeerConfig, field basic.Field) error {
 		err := cfg.PublicKey.UnmarshalText([]byte(field.Value))
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "unmarshal failed on field %q", field.Key)
 		default:
 			return nil
 		}
@@ -129,7 +129,7 @@ func loadPeerConfField(cfg *PeerConfig, field basic.Field) error {
 		err := cfg.Endpoint.UnmarshalText([]byte(field.Value))
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "unmarshal failed on field %q", field.Key)
 		default:
 			return nil
 		}
@@ -137,7 +137,7 @@ func loadPeerConfField(cfg *PeerConfig, field basic.Field) error {
 		s, err := parseAllowedIPs(field.Value)
 		switch {
 		case err != nil:
-			return core.Wrap(err, field.Key)
+			return core.Wrap(err, "%q is not an IP", field.Key)
 		case len(s) > 0:
 			cfg.AllowedIPs = append(cfg.AllowedIPs, s...)
 			return nil