Skip to content

Commit c809ba8

Browse files
authored
Merge pull request #34646 from hashicorp/td-ssoadmin-awssdkv2
ssoadmin: upgrade client to aws sdk v2
2 parents 176dd1a + 91b4f82 commit c809ba8

29 files changed

+492
-507
lines changed

go.mod

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -86,6 +86,7 @@ require (
8686
github.com/aws/aws-sdk-go-v2/service/ssm v1.44.1
8787
github.com/aws/aws-sdk-go-v2/service/ssmcontacts v1.20.1
8888
github.com/aws/aws-sdk-go-v2/service/ssmincidents v1.27.1
89+
github.com/aws/aws-sdk-go-v2/service/ssoadmin v1.23.1
8990
github.com/aws/aws-sdk-go-v2/service/sts v1.26.1
9091
github.com/aws/aws-sdk-go-v2/service/swf v1.20.1
9192
github.com/aws/aws-sdk-go-v2/service/timestreamwrite v1.23.1

go.sum

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -212,6 +212,8 @@ github.com/aws/aws-sdk-go-v2/service/ssmincidents v1.27.1 h1:1eP1sAN5FDkKwgM7B/0
212212
github.com/aws/aws-sdk-go-v2/service/ssmincidents v1.27.1/go.mod h1:3dHBotx1TNQmPzo0H62Kl2rJF21cBDUQHtQ3ajDK0H8=
213213
github.com/aws/aws-sdk-go-v2/service/sso v1.18.1 h1:V40g2daNO3l1J94JYwqfkyvQMYXi5I25fs3fNQW8iDs=
214214
github.com/aws/aws-sdk-go-v2/service/sso v1.18.1/go.mod h1:0ZWQJP/mBOUxkCvZKybZNz1XmdUKSBxoF0dzgfxtvDs=
215+
github.com/aws/aws-sdk-go-v2/service/ssoadmin v1.23.1 h1:m5TwU4+qNT3G7LfjOyAcscY+Td0L2jVZus8hd+gU7g0=
216+
github.com/aws/aws-sdk-go-v2/service/ssoadmin v1.23.1/go.mod h1:e8g8LmmUKSu5dtDpsVu1cypJzqQNkeqdgfm4n2wOnVw=
215217
github.com/aws/aws-sdk-go-v2/service/ssooidc v1.21.1 h1:uQrj7SpUNC3r55vc1CDh3qV9wJC66lz546xM9dhSo5s=
216218
github.com/aws/aws-sdk-go-v2/service/ssooidc v1.21.1/go.mod h1:oyaTk5xEAOuPXX1kCD7HmIeuLqdj3Bk5yGkqGXtGi14=
217219
github.com/aws/aws-sdk-go-v2/service/sts v1.26.1 h1:K33V7L0XDdb23FMOZySr8bon1jou5SHn1fiv7NJ1SUg=

internal/acctest/acctest.go

Lines changed: 15 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ import (
2121
ec2types "github.com/aws/aws-sdk-go-v2/service/ec2/types"
2222
"github.com/aws/aws-sdk-go-v2/service/inspector2"
2323
inspector2types "github.com/aws/aws-sdk-go-v2/service/inspector2/types"
24+
"github.com/aws/aws-sdk-go-v2/service/ssoadmin"
25+
ssoadmintypes "github.com/aws/aws-sdk-go-v2/service/ssoadmin/types"
2426
"github.com/aws/aws-sdk-go/aws"
2527
"github.com/aws/aws-sdk-go/aws/arn"
2628
"github.com/aws/aws-sdk-go/aws/endpoints"
@@ -29,7 +31,6 @@ import (
2931
"github.com/aws/aws-sdk-go/service/ec2"
3032
"github.com/aws/aws-sdk-go/service/iam"
3133
"github.com/aws/aws-sdk-go/service/outposts"
32-
"github.com/aws/aws-sdk-go/service/ssoadmin"
3334
"github.com/aws/aws-sdk-go/service/wafv2"
3435
"github.com/hashicorp/aws-sdk-go-base/v2/awsv1shim/v2/tfawserr"
3536
"github.com/hashicorp/terraform-plugin-go/tfprotov5"
@@ -1029,31 +1030,28 @@ func PreCheckOrganizationMemberAccount(ctx context.Context, t *testing.T) {
10291030
}
10301031

10311032
func PreCheckSSOAdminInstances(ctx context.Context, t *testing.T) {
1032-
conn := Provider.Meta().(*conns.AWSClient).SSOAdminConn(ctx)
1033+
conn := Provider.Meta().(*conns.AWSClient).SSOAdminClient(ctx)
10331034
input := &ssoadmin.ListInstancesInput{}
1034-
var instances []*ssoadmin.InstanceMetadata
1035+
var instances []ssoadmintypes.InstanceMetadata
10351036

1036-
err := conn.ListInstancesPagesWithContext(ctx, input, func(page *ssoadmin.ListInstancesOutput, lastPage bool) bool {
1037-
if page == nil {
1038-
return !lastPage
1037+
paginator := ssoadmin.NewListInstancesPaginator(conn, input)
1038+
for paginator.HasMorePages() {
1039+
page, err := paginator.NextPage(ctx)
1040+
if PreCheckSkipError(err) {
1041+
t.Skipf("skipping tests: %s", err)
1042+
}
1043+
if err != nil {
1044+
t.Fatalf("listing SSO Instances: %s", err)
10391045
}
10401046

1041-
instances = append(instances, page.Instances...)
1042-
1043-
return !lastPage
1044-
})
1045-
1046-
if PreCheckSkipError(err) {
1047-
t.Skipf("skipping tests: %s", err)
1047+
if page != nil {
1048+
instances = append(instances, page.Instances...)
1049+
}
10481050
}
10491051

10501052
if len(instances) == 0 {
10511053
t.Skip("skipping tests; no SSO Instances found.")
10521054
}
1053-
1054-
if err != nil {
1055-
t.Fatalf("listing SSO Instances: %s", err)
1056-
}
10571055
}
10581056

10591057
func PreCheckHasIAMRole(ctx context.Context, t *testing.T, roleName string) {

internal/conns/awsclient_gen.go

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)