Skip to content

Commit

Permalink
🧹 add log message for required namespace for aws applicationautoscaling
Browse files Browse the repository at this point in the history
  • Loading branch information
vjeffrey committed Jan 22, 2024
1 parent 4dea7a2 commit 5736487
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions providers/aws/resources/aws_applicationautoscaling.go
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,9 @@ func (a *mqlAwsApplicationautoscalingTarget) id() (string, error) {
func (a *mqlAwsApplicationAutoscaling) scalableTargets() ([]interface{}, error) {
conn := a.MqlRuntime.Connection.(*connection.AwsConnection)
namespace := a.Namespace.Data
if namespace == "" {
return nil, errors.New("namespace required for application autoscaling query. please specify one of [comprehend, rds, sagemaker, appstream, elasticmapreduce, dynamodb, lambda, ecs, cassandra, ec2, neptune, kafka, custom-resource, elasticache]")
}

res := []interface{}{}
poolOfJobs := jobpool.CreatePool(a.getTargets(conn, aatypes.ServiceNamespace(namespace)), 5)
Expand Down

0 comments on commit 5736487

Please sign in to comment.