Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate IoT services to AWS Go SDKv2 #37152

Open
wants to merge 29 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
29 commits
Select commit Hold shift + click to select a range
3521154
feat: migrated iot services to SDKv2
DanielRieske Apr 29, 2024
c845db3
chore: linter findings
DanielRieske Apr 29, 2024
ce0b150
feat: refactored unused params and fixed ci findings
DanielRieske Apr 29, 2024
e0943c3
fix: fix ci finding
DanielRieske Apr 29, 2024
3aa6eb1
chore: added changelog
DanielRieske Apr 29, 2024
7b72b62
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 7, 2024
3fa49c4
chore: fix merge conflicts
DanielRieske May 7, 2024
d65ba94
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 9, 2024
7830143
chore: fix mod
DanielRieske May 9, 2024
331e32b
chore: cleantidy
DanielRieske May 9, 2024
7f9e834
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 11, 2024
3a7296d
chore: fix undefined
DanielRieske May 11, 2024
8ecd6f8
fix: semgrep finding
DanielRieske May 11, 2024
855ddfc
feat: added `pre_provisioning_hook` update with tests
DanielRieske May 12, 2024
d3b8443
fmt: fix formatting
DanielRieske May 12, 2024
17916f6
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 12, 2024
f99fd27
fix: added copyright headers
DanielRieske May 12, 2024
47f6ef9
chore: updated zip
DanielRieske May 12, 2024
bb7dc16
chore: changed changelog
DanielRieske May 12, 2024
75564b6
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 14, 2024
aebedfd
fix: fix semgrep findings
DanielRieske May 14, 2024
2d97312
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 14, 2024
354dfef
fix: semgrep findings
DanielRieske May 15, 2024
68427d6
fix: tag test
DanielRieske May 15, 2024
f076429
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 16, 2024
772f840
fix: semgrep finding
DanielRieske May 16, 2024
59eeda5
chore: fix literals
DanielRieske May 16, 2024
537f247
Merge branch 'main' into f/add-tags-iot-authorizer-resource
DanielRieske May 24, 2024
d900f33
chore: `make clean-tidy`
DanielRieske May 24, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 7 additions & 0 deletions .changelog/37152.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,7 @@
```release-note:enhancement
resource/aws_iot_authorizer: Add `tags` argument
```

```release-note:bug
resource/aws_iot_provisioning_template: Fix `pre_provisioning_hook` update operation
```
2 changes: 0 additions & 2 deletions .github/labeler-issue-triage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -307,8 +307,6 @@ service/grafana:
- '((\*|-)\s*`?|(data|resource)\s+"?)aws_grafana_'
service/greengrass:
- '((\*|-)\s*`?|(data|resource)\s+"?)aws_greengrass_'
service/greengrassv2:
- '((\*|-)\s*`?|(data|resource)\s+"?)aws_greengrassv2_'
service/groundstation:
- '((\*|-)\s*`?|(data|resource)\s+"?)aws_groundstation_'
service/guardduty:
Expand Down
6 changes: 0 additions & 6 deletions .github/labeler-pr-triage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -990,12 +990,6 @@ service/greengrass:
- any-glob-to-any-file:
- 'internal/service/greengrass/**/*'
- 'website/**/greengrass_*'
service/greengrassv2:
- any:
- changed-files:
- any-glob-to-any-file:
- 'internal/service/greengrassv2/**/*'
- 'website/**/greengrassv2_*'
service/groundstation:
- any:
- changed-files:
Expand Down
4 changes: 4 additions & 0 deletions go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -97,12 +97,16 @@ require (
github.com/aws/aws-sdk-go-v2/service/fms v1.33.4
github.com/aws/aws-sdk-go-v2/service/glacier v1.22.7
github.com/aws/aws-sdk-go-v2/service/globalaccelerator v1.23.4
github.com/aws/aws-sdk-go-v2/service/greengrass v1.23.8
github.com/aws/aws-sdk-go-v2/service/groundstation v1.27.3
github.com/aws/aws-sdk-go-v2/service/healthlake v1.24.3
github.com/aws/aws-sdk-go-v2/service/iam v1.32.3
github.com/aws/aws-sdk-go-v2/service/identitystore v1.23.8
github.com/aws/aws-sdk-go-v2/service/inspector2 v1.26.3
github.com/aws/aws-sdk-go-v2/service/internetmonitor v1.14.3
github.com/aws/aws-sdk-go-v2/service/iot v1.53.7
github.com/aws/aws-sdk-go-v2/service/iotanalytics v1.22.8
github.com/aws/aws-sdk-go-v2/service/iotevents v1.23.8
github.com/aws/aws-sdk-go-v2/service/ivschat v1.12.8
github.com/aws/aws-sdk-go-v2/service/kafka v1.32.0
github.com/aws/aws-sdk-go-v2/service/kendra v1.50.4
Expand Down
8 changes: 8 additions & 0 deletions go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -214,6 +214,8 @@ github.com/aws/aws-sdk-go-v2/service/glacier v1.22.7 h1:jSoAdc5Wbl1H069xLRePzgty
github.com/aws/aws-sdk-go-v2/service/glacier v1.22.7/go.mod h1:h7fH8k8flhqe6S0QiQgknLUhdodEGoF2u5bO+7l6vQ4=
github.com/aws/aws-sdk-go-v2/service/globalaccelerator v1.23.4 h1:NR18GyCVnE1Lkpcsd97+Q6PLKuo7TDJ/nCwavr+sj+Y=
github.com/aws/aws-sdk-go-v2/service/globalaccelerator v1.23.4/go.mod h1:641tgeMVfmvuXFc3PVh4I8+Tsag3TzaE/7ojMAjSeoI=
github.com/aws/aws-sdk-go-v2/service/greengrass v1.23.8 h1:ihMyngCB4Abfkw6FpHjb55rFYVYkUB00QledLjxlqKg=
github.com/aws/aws-sdk-go-v2/service/greengrass v1.23.8/go.mod h1:IuW0bgMj3sNQt7P+/uKNBh7Ovy9p6DCAgo+jSZpF1gI=
github.com/aws/aws-sdk-go-v2/service/groundstation v1.27.3 h1:w7sdXhMRmrHLAmXULXZ2nZiSUlwcuzdXreadDjMde80=
github.com/aws/aws-sdk-go-v2/service/groundstation v1.27.3/go.mod h1:HAClmwin3MTbLdiUJxhQUc7ZQFi8CQvucP2UzZf5tgw=
github.com/aws/aws-sdk-go-v2/service/healthlake v1.24.3 h1:2UoRqM9FtHsPi1n8U8Pkr8MJ1qfhVqI2FImBk9hzp0Y=
Expand All @@ -236,6 +238,12 @@ github.com/aws/aws-sdk-go-v2/service/internal/s3shared v1.17.7 h1:uO5XR6QGBcmPyo
github.com/aws/aws-sdk-go-v2/service/internal/s3shared v1.17.7/go.mod h1:feeeAYfAcwTReM6vbwjEyDmiGho+YgBhaFULuXDW8kc=
github.com/aws/aws-sdk-go-v2/service/internetmonitor v1.14.3 h1:eUBUSeRzLSyGVlcmrr1dzhnGoAjhBYdtpK8KXVMhQRU=
github.com/aws/aws-sdk-go-v2/service/internetmonitor v1.14.3/go.mod h1:Y1IgnRxlZuTFnmdLmC3s6EXKBMsA+1PASjjXI60T6lE=
github.com/aws/aws-sdk-go-v2/service/iot v1.53.7 h1:3HhmhDT5LhvDkED/pqSzs7i2yXYZVw0jL2U3o1CbFOM=
github.com/aws/aws-sdk-go-v2/service/iot v1.53.7/go.mod h1:akpp3CWIm/1/Bgxf4wgsmRnDWgsLMDpVdk72hVB591M=
github.com/aws/aws-sdk-go-v2/service/iotanalytics v1.22.8 h1:J76DaZZAtIkCBXBysa/8jIMPLLGCDiK2gWd/aeXdLwc=
github.com/aws/aws-sdk-go-v2/service/iotanalytics v1.22.8/go.mod h1:AC1P6YtK0aLYW/WRIlwqIOrbj4OrxDUA6ikVrKY21o0=
github.com/aws/aws-sdk-go-v2/service/iotevents v1.23.8 h1:PNa0KhiuU/xDb/s9/fksgBJma9sX/CY1OC/5KbN/Lgs=
github.com/aws/aws-sdk-go-v2/service/iotevents v1.23.8/go.mod h1:Ojl4NwWGBwxtsIgkZEozcZg9W3RyFCoyj2tCB8h3DVg=
github.com/aws/aws-sdk-go-v2/service/ivschat v1.12.8 h1:sX2ZySw9Ul7Wo3NQ58nm3zZTFFgvOMALH1W4PfmsuII=
github.com/aws/aws-sdk-go-v2/service/ivschat v1.12.8/go.mod h1:eLQ2RhI4uRQjlsGLUZIicFi8GUwa8LMBZCwOZbz+rfE=
github.com/aws/aws-sdk-go-v2/service/kafka v1.32.0 h1:lAEg5YLE60fEZr3w3q8gzCJDgs6pBX8W7dg0HEXqCn8=
Expand Down
1 change: 0 additions & 1 deletion infrastructure/repository/labels-service.tf
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,6 @@ variable "service_labels" {
"glue",
"grafana",
"greengrass",
"greengrassv2",
"groundstation",
"guardduty",
"health",
Expand Down
24 changes: 12 additions & 12 deletions internal/conns/awsclient_gen.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion internal/service/greengrass/generate.go
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// Copyright (c) HashiCorp, Inc.
// SPDX-License-Identifier: MPL-2.0

//go:generate go run ../../generate/tags/main.go -ListTags -ServiceTagsMap -UpdateTags
//go:generate go run ../../generate/tags/main.go -ListTags -ServiceTagsMap -UpdateTags -AWSSDKVersion=2 -KVTValues -SkipTypesImp
//go:generate go run ../../generate/servicepackage/main.go
// ONLY generate directives and package declaration! Do not add anything else to this file.

Expand Down
40 changes: 25 additions & 15 deletions internal/service/greengrass/service_endpoints_gen_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

17 changes: 10 additions & 7 deletions internal/service/greengrass/service_package_gen.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

33 changes: 16 additions & 17 deletions internal/service/greengrass/tags_gen.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.