Merge pull request #452 from ekristen/fix-gamelift-supported-regions

fix(gamelift): skip unsupported regions

resources/gamelift-build.go

@@ -24,12 +24,22 @@ func init() {
 	})
 }
 
-type GameLiftBuildLister struct{}
+type GameLiftBuildLister struct {
+	GameLift
+}
 
 func (l *GameLiftBuildLister) List(_ context.Context, o interface{}) ([]resource.Resource, error) {
 	opts := o.(*nuke.ListerOpts)
 	var resources []resource.Resource
 
+	if !l.IsSupportedRegion(opts.Region.Name) {
+		opts.Logger.
+			WithField("resource", GameLiftBuildResource).
+			WithField("region", opts.Region.Name).
+			Debug("region not supported")
+		return resources, nil
+	}
+
 	svc := gamelift.New(opts.Session)
 
 	params := &gamelift.ListBuildsInput{}

resources/gamelift-fleet.go

@@ -23,12 +23,22 @@ func init() {
 	})
 }
 
-type GameLiftFleetLister struct{}
+type GameLiftFleetLister struct {
+	GameLift
+}
 
 func (l *GameLiftFleetLister) List(_ context.Context, o interface{}) ([]resource.Resource, error) {
 	opts := o.(*nuke.ListerOpts)
 	var resources []resource.Resource
 
+	if !l.IsSupportedRegion(opts.Region.Name) {
+		opts.Logger.
+			WithField("resource", GameLiftFleetResource).
+			WithField("region", opts.Region.Name).
+			Debug("region not supported")
+		return resources, nil
+	}
+
 	svc := gamelift.New(opts.Session)
 
 	params := &gamelift.ListFleetsInput{}

resources/gamelift-mm-config.go

@@ -25,12 +25,22 @@ func init() {
 	})
 }
 
-type GameLiftMatchmakingConfigurationLister struct{}
+type GameLiftMatchmakingConfigurationLister struct {
+	GameLift
+}
 
 func (l *GameLiftMatchmakingConfigurationLister) List(_ context.Context, o interface{}) ([]resource.Resource, error) {
 	opts := o.(*nuke.ListerOpts)
 	var resources []resource.Resource
 
+	if !l.IsSupportedRegion(opts.Region.Name) {
+		opts.Logger.
+			WithField("resource", GameLiftMatchmakingConfigurationResource).
+			WithField("region", opts.Region.Name).
+			Debug("region not supported")
+		return resources, nil
+	}
+
 	svc := gamelift.New(opts.Session)
 
 	params := &gamelift.DescribeMatchmakingConfigurationsInput{}

resources/gamelift-mm-rule.go

@@ -24,12 +24,22 @@ func init() {
 	})
 }
 
-type GameLiftMatchmakingRuleSetLister struct{}
+type GameLiftMatchmakingRuleSetLister struct {
+	GameLift
+}
 
 func (l *GameLiftMatchmakingRuleSetLister) List(_ context.Context, o interface{}) ([]resource.Resource, error) {
 	opts := o.(*nuke.ListerOpts)
 	var resources []resource.Resource
 
+	if !l.IsSupportedRegion(opts.Region.Name) {
+		opts.Logger.
+			WithField("resource", GameLiftMatchmakingRuleSetResource).
+			WithField("region", opts.Region.Name).
+			Debug("region not supported")
+		return resources, nil
+	}
+
 	svc := gamelift.New(opts.Session)
 
 	params := &gamelift.DescribeMatchmakingRuleSetsInput{}

resources/gamelift-queue.go

@@ -23,12 +23,22 @@ func init() {
 	})
 }
 
-type GameLiftQueueLister struct{}
+type GameLiftQueueLister struct {
+	GameLift
+}
 
 func (l *GameLiftQueueLister) List(_ context.Context, o interface{}) ([]resource.Resource, error) {
 	opts := o.(*nuke.ListerOpts)
 	var resources []resource.Resource
 
+	if !l.IsSupportedRegion(opts.Region.Name) {
+		opts.Logger.
+			WithField("resource", GameLiftQueueResource).
+			WithField("region", opts.Region.Name).
+			Debug("region not supported")
+		return resources, nil
+	}
+
 	svc := gamelift.New(opts.Session)
 
 	params := &gamelift.DescribeGameSessionQueuesInput{}

resources/gamelift.go

@@ -0,0 +1,23 @@
+package resources
+
+import "slices"
+
+type GameLift struct{}
+
+func (g *GameLift) IsSupportedRegion(region string) bool {
+	// ref: https://docs.aws.amazon.com/gamelift/latest/developerguide/gamelift-regions.html
+	// there are fewer unsupported, so doing the inverse
+	// unsupported are regions that only support "Remote location for multi-location fleets"
+	// note: we do not currently filter down to the local zone
+	unsupportedRegions := []string{
+		"af-south-1",
+		"ap-east-1",
+		"ap-northeast-3",
+		"eu-north-1",
+		"eu-south-1",
+		"eu-west-3",
+		"me-south-1",
+	}
+
+	return !slices.Contains(unsupportedRegions, region)
+}