Skip to content

Commit

Permalink
rds/orderable_instance: Improve performance
Browse files Browse the repository at this point in the history
  • Loading branch information
YakDriver committed Feb 9, 2024
1 parent c0a95af commit 6154e5f
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
6 changes: 3 additions & 3 deletions internal/service/rds/orderable_instance_data_source.go
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,9 @@ func dataSourceOrderableInstanceRead(ctx context.Context, d *schema.ResourceData
var diags diag.Diagnostics
conn := meta.(*conns.AWSClient).RDSConn(ctx)

input := &rds.DescribeOrderableDBInstanceOptionsInput{}
input := &rds.DescribeOrderableDBInstanceOptionsInput{
MaxRecords: aws.Int64(3412),
}

if v, ok := d.GetOk("availability_zone_group"); ok {
input.AvailabilityZoneGroup = aws.String(v.(string))
Expand Down Expand Up @@ -463,8 +465,6 @@ func dataSourceOrderableInstanceRead(ctx context.Context, d *schema.ResourceData
return sdkdiag.AppendErrorf(diags, "no RDS DB Instance Classes match the criteria; try a different search")
}

//fmt.Printf("found: %s, %s\n", aws.StringValue(found.DBInstanceClass), aws.StringValue(found.EngineVersion))

d.SetId(aws.StringValue(found.DBInstanceClass))
d.Set("availability_zone_group", found.AvailabilityZoneGroup)
var availabilityZones []string
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -242,7 +242,7 @@ func TestAccRDSOrderableInstanceDataSource_readReplicaCapable(t *testing.T) {
resource.TestCheckResourceAttr(dataSourceName, "read_replica_capable", "true"),
resource.TestCheckResourceAttr(dataSourceName, "engine", tfrds.InstanceEngineOracleEnterprise),
resource.TestCheckResourceAttr(dataSourceName, "engine_latest_version", "true"),
resource.TestMatchResourceAttr(dataSourceName, "instance_class", regexache.MustCompile(`^db\..*large$`)),
resource.TestCheckResourceAttrSet(dataSourceName, "instance_class"),
),
},
},
Expand Down

0 comments on commit 6154e5f

Please sign in to comment.