fix: AWS random region for PlacementSocre API requests

Recently the region not offering Placement Score was fixed using a fixed region which offers the API, although the initial problem was solved using one single region for all the requests within the same account generates rate limit errors for API call. This commit introduces a way to randomize the region used to run the request distributing the load of the calls. Fixes #617

Signed-off-by: Adrian Riobo <[email protected]>

Author: adrianriobo

go.mod

@@ -1,8 +1,6 @@
 module github.com/redhat-developer/mapt
 
-go 1.24.1
-
-toolchain go1.25.1
+go 1.24.6
 
 require (
 	github.com/coocood/freecache v1.2.4

pkg/provider/aws/data/regions.go

@@ -12,11 +12,15 @@ import (
 
 var (
 	optInStatusFilter      string = "opt-in-status"
-	optInStatusNorRequired string = "opt-in-not-required"
-	optInStatusOptedIn     string = "opted-in"
+	OptInStatusNotRequired string = "opt-in-not-required"
+	OptInStatusOptedIn     string = "opted-in"
 )
 
 func GetRegions() ([]string, error) {
+	return GetRegionsByOptInStatus([]string{OptInStatusNotRequired, OptInStatusOptedIn})
+}
+
+func GetRegionsByOptInStatus(optInStaus []string) ([]string, error) {
 	cfg, err := config.LoadDefaultConfig(context.TODO())
 	if err != nil {
 		return nil, err
@@ -28,7 +32,7 @@ func GetRegions() ([]string, error) {
 			Filters: []ec2Types.Filter{
 				{
 					Name:   &optInStatusFilter,
-					Values: []string{optInStatusNorRequired, optInStatusOptedIn},
+					Values: optInStaus,
 				},
 			}})
 	if err != nil {

pkg/provider/aws/data/spot.go

@@ -172,9 +172,14 @@ func SpotInfo(mCtx *mc.Context, args *SpotInfoArgs) (*spot.SpotResults, error) {
 	if err != nil {
 		return nil, err
 	}
-
+	// Placement Socres
+	nOpInRegions, err := GetRegionsByOptInStatus([]string{OptInStatusNotRequired})
+	if err != nil {
+		return nil, err
+	}
 	placementScores, err := hostingPlaces.RunOnHostingPlaces(regions,
 		placementScoreArgs{
+			apiRegion:         util.RandomItemFromArray(nOpInRegions),
 			minPlacementScore: minPlacementScore(*args.SpotTolerance),
 			instanceTypes:     args.InstaceTypes,
 			capacity:          1,
@@ -368,6 +373,10 @@ func spotPricingAsync(r string, args spotPricingArgs, c chan hostingPlaces.Hosti
 }
 
 type placementScoreArgs struct {
+	// Not all regions offered the API to get
+	// data about placement score, we need set which one
+	// will be used
+	apiRegion         string
 	minPlacementScore int32
 	instanceTypes     []string
 	capacity          int32
@@ -379,15 +388,11 @@ type placementScoreResult struct {
 	azName string
 }
 
-// Every async run will use same region to request data
-// as API is not available on all regions.
-const PLACEMENT_SCORE_API_REGION = "us-east-2"
-
 // This will get placement scores grouped on map per region
 // only scores over tolerance will be added
 func placementScoresAsync(r string, args placementScoreArgs, c chan hostingPlaces.HostingPlaceData[[]placementScoreResult]) {
 	azsByRegion := describeAvailabilityZonesByRegions([]string{r})
-	cfg, err := getConfig(PLACEMENT_SCORE_API_REGION)
+	cfg, err := getConfig(args.apiRegion)
 	if err != nil {
 		hostingPlaces.SendAsyncErr(c, err)
 		return

tools/go.mod

@@ -1,8 +1,6 @@
 module github.com/redhat-developer/mapt/tools
 
-go 1.24.0
-
-toolchain go1.25.1
+go 1.24.6
 
 require github.com/golangci/golangci-lint/v2 v2.5.0