goreleaser bug

Author: SivaanandM

spectrocloud/kubevirt/schema/virtualmachineinstance/domain_spec.go

@@ -2,13 +2,13 @@ package virtualmachineinstance
 
 import (
 	"fmt"
+	"math"
 
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
 	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/validation"
 	"k8s.io/apimachinery/pkg/api/resource"
 	kubevirtapiv1 "kubevirt.io/api/core/v1"
 
-	"github.com/spectrocloud/terraform-provider-spectrocloud/spectrocloud/constants"
 	"github.com/spectrocloud/terraform-provider-spectrocloud/spectrocloud/kubevirt/utils"
 )
 
@@ -289,19 +289,19 @@ func expandCPU(cpu map[string]interface{}) (kubevirtapiv1.CPU, error) {
 	}
 
 	if v, ok := cpu["cores"].(int); ok {
-		if v < 0 || v > constants.UInt32MaxValue {
+		if v < 0 || uint64(v) > math.MaxUint32 {
 			return result, fmt.Errorf("cores value %d is out of range for uint32", v)
 		}
 		result.Cores = uint32(v)
 	}
 	if v, ok := cpu["sockets"].(int); ok {
-		if v < 0 || v > constants.UInt32MaxValue {
+		if v < 0 || uint64(v) > math.MaxUint32 {
 			return result, fmt.Errorf("sockets value %d is out of range for uint32", v)
 		}
 		result.Sockets = uint32(v)
 	}
 	if v, ok := cpu["threads"].(int); ok {
-		if v < 0 || v > constants.UInt32MaxValue {
+		if v < 0 || uint64(v) > math.MaxUint32 {
 			return result, fmt.Errorf("threads value %d is out of range for uint32", v)
 		}
 		result.Threads = uint32(v)

spectrocloud/resource_cluster_vsphere.go

@@ -5,6 +5,7 @@ import (
 	"errors"
 	"fmt"
 	"log"
+	"math"
 	"sort"
 	"strings"
 	"time"
@@ -15,7 +16,6 @@ import (
 	"github.com/spectrocloud/palette-sdk-go/api/models"
 	"github.com/spectrocloud/palette-sdk-go/client"
 
-	"github.com/spectrocloud/terraform-provider-spectrocloud/spectrocloud/constants"
 	"github.com/spectrocloud/terraform-provider-spectrocloud/spectrocloud/schemas"
 	"github.com/spectrocloud/terraform-provider-spectrocloud/types"
 )
@@ -855,7 +855,7 @@ func toMachinePoolVsphere(machinePool interface{}) (*models.V1VsphereMachinePool
 	memoryInt := ins["memory_mb"].(int)
 	cpuInt := ins["cpu"].(int)
 
-	if diskSizeInt > constants.Int32MaxValue || memoryInt > constants.Int64MaxValue || cpuInt > constants.Int32MaxValue {
+	if uint64(diskSizeInt) > math.MaxInt32 || uint64(memoryInt) > math.MaxInt64 || uint64(cpuInt) > math.MaxInt32 {
 		return nil, fmt.Errorf("instance type values out of range: disk_size_gb=%d, memory_mb=%d, cpu=%d", diskSizeInt, memoryInt, cpuInt)
 	}
 
@@ -866,7 +866,7 @@ func toMachinePoolVsphere(machinePool interface{}) (*models.V1VsphereMachinePool
 	}
 
 	countInt := m["count"].(int)
-	if countInt > constants.Int32MaxValue {
+	if uint64(countInt) > math.MaxInt32 {
 		return nil, fmt.Errorf("count value %d is out of range for int32", countInt)
 	}
 
@@ -875,15 +875,15 @@ func toMachinePoolVsphere(machinePool interface{}) (*models.V1VsphereMachinePool
 
 	if m["min"] != nil {
 		minInt := m["min"].(int)
-		if minInt > constants.Int32MaxValue {
+		if uint64(minInt) > math.MaxInt32 {
 			return nil, fmt.Errorf("min value %d is out of range for int32", minInt)
 		}
 		min = SafeInt32(minInt)
 	}
 
 	if m["max"] != nil {
 		maxInt := m["max"].(int)
-		if maxInt > constants.Int32MaxValue {
+		if uint64(maxInt) > math.MaxInt32 {
 			return nil, fmt.Errorf("max value %d is out of range for int32", maxInt)
 		}
 		max = SafeInt32(maxInt)
@@ -915,13 +915,13 @@ func toMachinePoolVsphere(machinePool interface{}) (*models.V1VsphereMachinePool
 		if m["node_repave_interval"] != nil {
 			nodeRepaveInterval = m["node_repave_interval"].(int)
 		}
-		if nodeRepaveInterval > constants.Int32MaxValue {
+		if uint64(nodeRepaveInterval) > math.MaxInt32 {
 			return nil, fmt.Errorf("node_repave_interval value %d is out of range for int32", nodeRepaveInterval)
 		}
 		mp.PoolConfig.NodeRepaveInterval = SafeInt32(nodeRepaveInterval)
 	} else {
 		nodeRepaveInterval := m["node_repave_interval"].(int)
-		if nodeRepaveInterval > constants.Int32MaxValue {
+		if uint64(nodeRepaveInterval) > math.MaxInt32 {
 			return nil, fmt.Errorf("node_repave_interval value %d is out of range for int32", nodeRepaveInterval)
 		}
 		err := ValidationNodeRepaveIntervalForControlPlane(nodeRepaveInterval)

spectrocloud/utils.go

@@ -27,7 +27,7 @@ func SafeUint32(value int) uint32 {
 	if value < 0 {
 		return 0
 	}
-	if value > math.MaxUint32 {
+	if uint64(value) > math.MaxUint32 {
 		return math.MaxUint32
 	}
 	return uint32(value)