apply golangcilint

Author: sp-yduck

cloud/scope/machine.go

@@ -147,7 +147,7 @@ func (m *MachineScope) GetBiosUUID() *string {
 	if err != nil {
 		return nil
 	}
-	return pointer.StringPtr(parsed.ID())
+	return pointer.String(parsed.ID())
 }
 
 func (m *MachineScope) GetProviderID() string {
@@ -194,7 +194,7 @@ func (m *MachineScope) SetProviderID(uuid string) error {
 	if err != nil {
 		return err
 	}
-	m.ProxmoxMachine.Spec.ProviderID = pointer.StringPtr(providerid.String())
+	m.ProxmoxMachine.Spec.ProviderID = pointer.String(providerid.String())
 	return nil
 }
 
@@ -207,7 +207,7 @@ func (m *MachineScope) SetReady() {
 }
 
 func (m *MachineScope) SetFailureMessage(v error) {
-	m.ProxmoxMachine.Status.FailureMessage = pointer.StringPtr(v.Error())
+	m.ProxmoxMachine.Status.FailureMessage = pointer.String(v.Error())
 }
 
 func (m *MachineScope) SetFailureReason(v capierrors.MachineStatusError) {

cloud/services/compute/instance/reconcile.go

@@ -4,7 +4,6 @@ import (
 	"context"
 	"fmt"
 	"regexp"
-	"strconv"
 	"strings"
 
 	"github.com/pkg/errors"
@@ -37,7 +36,9 @@ func (s *Service) Reconcile(ctx context.Context) error {
 	}
 
 	log.Info(fmt.Sprintf("Reconciled instance: bios-uuid=%s", *uuid))
-	s.scope.SetProviderID(*uuid)
+	if err := s.scope.SetProviderID(*uuid); err != nil {
+		return err
+	}
 	s.scope.SetInstanceStatus(infrav1.InstanceStatus(instance.Status))
 	// s.scope.SetAddresses()
 	return nil
@@ -127,7 +128,7 @@ func getBiosUUID(vm *vm.VirtualMachine) (*string, error) {
 	if err != nil {
 		return nil, err
 	}
-	return pointer.StringPtr(uuid), nil
+	return pointer.String(uuid), nil
 }
 
 func convertSMBiosToUUID(smbios string) (string, error) {
@@ -140,29 +141,6 @@ func convertSMBiosToUUID(smbios string) (string, error) {
 	return strings.Split(match, "=")[1], nil
 }
 
-// will be abolished
-func (s *Service) getInstanceFromInstanceID(instanceID string) (*vm.VirtualMachine, error) {
-	vmid, err := strconv.Atoi(instanceID)
-	if err != nil {
-		return nil, err
-	}
-	nodes, err := s.client.Nodes()
-	if err != nil {
-		return nil, err
-	}
-	if len(nodes) == 0 {
-		return nil, errors.New("proxmox nodes not found")
-	}
-	for _, node := range nodes {
-		vm, err := node.VirtualMachine(vmid)
-		if err != nil {
-			continue
-		}
-		return vm, nil
-	}
-	return nil, api.ErrNotFound
-}
-
 func (s *Service) getInstanceFromBiosUUID(uuid string) (*vm.VirtualMachine, error) {
 	nodes, err := s.client.Nodes()
 	if err != nil {