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

[New Resource]: DataZone Form Type #38668

Closed
wants to merge 215 commits into from

Conversation

ThomasZalewski
Copy link
Contributor

@ThomasZalewski ThomasZalewski commented Aug 2, 2024

Added support for the datazone form type resource

Description

This resource will allow practitioners to create DataZone Form Types for DataZone via Terraform.

Relations

Relates #37423

References

https://docs.aws.amazon.com/datazone/latest/APIReference/API_CreateFormType.html#datazone-CreateFormType-request-model

Output from Acceptance Testing

make testacc TESTS=TestAccDataZoneFormType PKG=datazone                           
make: Verifying source code with gofmt...
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go1.22.5 test ./internal/service/datazone/... -v -count 1 -parallel 20 -run='TestAccDataZoneFormType'  -timeout 360m
=== RUN   TestAccDataZoneFormType_basic
=== PAUSE TestAccDataZoneFormType_basic
=== RUN   TestAccDataZoneFormType_disappears
=== PAUSE TestAccDataZoneFormType_disappears
=== CONT  TestAccDataZoneFormType_basic
=== CONT  TestAccDataZoneFormType_disappears
--- PASS: TestAccDataZoneFormType_disappears (28.49s)
--- PASS: TestAccDataZoneFormType_basic (29.51s)
PASS
ok      github.com/hashicorp/terraform-provider-aws/internal/service/datazone   34.489s

showwin and others added 30 commits January 9, 2024 15:37
This commit has the provider consume default_tags from environment
variables with the form `TF_AWS_DEFAULT_TAGS_<tag_name>=<tag_value>`. If
the same tag is defined in both an environment variable and the
Terraform config, the Terraform takes precedence.
ewbankkit and others added 7 commits August 7, 2024 09:15
Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.23.0 to 0.24.0.
- [Release notes](https://github.com/golang/tools/releases)
- [Commits](golang/tools@v0.23.0...v0.24.0)

---
updated-dependencies:
- dependency-name: golang.org/x/tools
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.25.0 to 0.26.0.
- [Commits](golang/crypto@v0.25.0...v0.26.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…etypes (#38729)

Bumps [github.com/hashicorp/terraform-plugin-framework-timetypes](https://github.com/hashicorp/terraform-plugin-framework-timetypes) from 0.4.0 to 0.5.0.
- [Release notes](https://github.com/hashicorp/terraform-plugin-framework-timetypes/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-framework-timetypes/blob/main/CHANGELOG.md)
- [Commits](hashicorp/terraform-plugin-framework-timetypes@v0.4.0...v0.5.0)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-framework-timetypes
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…8739)

Bumps the terraform-devex group with 1 update in the /tools/tfsdk2fw directory: [github.com/hashicorp/terraform-plugin-framework](https://github.com/hashicorp/terraform-plugin-framework).


Updates `github.com/hashicorp/terraform-plugin-framework` from 1.10.0 to 1.11.0
- [Release notes](https://github.com/hashicorp/terraform-plugin-framework/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-framework/blob/main/CHANGELOG.md)
- [Commits](hashicorp/terraform-plugin-framework@v1.10.0...v1.11.0)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-framework
  dependency-type: indirect
  update-type: version-update:semver-minor
  dependency-group: terraform-devex
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [golang.org/x/tools](https://github.com/golang/tools) from 0.23.0 to 0.24.0.
- [Release notes](https://github.com/golang/tools/releases)
- [Commits](golang/tools@v0.23.0...v0.24.0)

---
updated-dependencies:
- dependency-name: golang.org/x/tools
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…dates (#38725)

Bumps the aws-sdk-go group with 3 updates in the / directory: [github.com/aws/aws-sdk-go-v2/service/cognitoidentityprovider](https://github.com/aws/aws-sdk-go-v2), [github.com/aws/aws-sdk-go-v2/service/costoptimizationhub](https://github.com/aws/aws-sdk-go-v2) and [github.com/aws/aws-sdk-go-v2/service/workspaces](https://github.com/aws/aws-sdk-go-v2).
Bumps the aws-sdk-go group with 3 updates in the /tools/tfsdk2fw directory: [github.com/aws/aws-sdk-go-v2/service/cognitoidentityprovider](https://github.com/aws/aws-sdk-go-v2), [github.com/aws/aws-sdk-go-v2/service/costoptimizationhub](https://github.com/aws/aws-sdk-go-v2) and [github.com/aws/aws-sdk-go-v2/service/workspaces](https://github.com/aws/aws-sdk-go-v2).


Updates `github.com/aws/aws-sdk-go-v2/service/cognitoidentityprovider` from 1.41.4 to 1.42.0
- [Release notes](https://github.com/aws/aws-sdk-go-v2/releases)
- [Commits](aws/aws-sdk-go-v2@service/ecs/v1.41.4...service/s3/v1.42.0)

Updates `github.com/aws/aws-sdk-go-v2/service/costoptimizationhub` from 1.7.3 to 1.8.0
- [Release notes](https://github.com/aws/aws-sdk-go-v2/releases)
- [Changelog](https://github.com/aws/aws-sdk-go-v2/blob/v1.8.0/CHANGELOG.md)
- [Commits](aws/aws-sdk-go-v2@service/m2/v1.7.3...v1.8.0)

Updates `github.com/aws/aws-sdk-go-v2/service/workspaces` from 1.44.3 to 1.45.0
- [Release notes](https://github.com/aws/aws-sdk-go-v2/releases)
- [Commits](aws/aws-sdk-go-v2@service/ecs/v1.44.3...service/s3/v1.45.0)

Updates `github.com/aws/aws-sdk-go-v2/service/cognitoidentityprovider` from 1.41.4 to 1.42.0
- [Release notes](https://github.com/aws/aws-sdk-go-v2/releases)
- [Commits](aws/aws-sdk-go-v2@service/ecs/v1.41.4...service/s3/v1.42.0)

Updates `github.com/aws/aws-sdk-go-v2/service/costoptimizationhub` from 1.7.3 to 1.8.0
- [Release notes](https://github.com/aws/aws-sdk-go-v2/releases)
- [Changelog](https://github.com/aws/aws-sdk-go-v2/blob/v1.8.0/CHANGELOG.md)
- [Commits](aws/aws-sdk-go-v2@service/m2/v1.7.3...v1.8.0)

Updates `github.com/aws/aws-sdk-go-v2/service/workspaces` from 1.44.3 to 1.45.0
- [Release notes](https://github.com/aws/aws-sdk-go-v2/releases)
- [Commits](aws/aws-sdk-go-v2@service/ecs/v1.44.3...service/s3/v1.45.0)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go-v2/service/cognitoidentityprovider
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: aws-sdk-go
- dependency-name: github.com/aws/aws-sdk-go-v2/service/costoptimizationhub
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: aws-sdk-go
- dependency-name: github.com/aws/aws-sdk-go-v2/service/workspaces
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: aws-sdk-go
- dependency-name: github.com/aws/aws-sdk-go-v2/service/cognitoidentityprovider
  dependency-type: indirect
  update-type: version-update:semver-minor
  dependency-group: aws-sdk-go
- dependency-name: github.com/aws/aws-sdk-go-v2/service/costoptimizationhub
  dependency-type: indirect
  update-type: version-update:semver-minor
  dependency-group: aws-sdk-go
- dependency-name: github.com/aws/aws-sdk-go-v2/service/workspaces
  dependency-type: indirect
  update-type: version-update:semver-minor
  dependency-group: aws-sdk-go
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
func (r *resourceFormType) Metadata(_ context.Context, req resource.MetadataRequest, resp *resource.MetadataResponse) {
resp.TypeName = "aws_datazone_form_type"
}
func (r *resourceFormType) Schema(ctx context.Context, req resource.SchemaRequest, resp *resource.SchemaResponse) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's put the schema attributes in alphabetical order

Comment on lines 214 to 215
func (r *resourceFormType) Update(ctx context.Context, req resource.UpdateRequest, resp *resource.UpdateResponse) {
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This can be removed completely since it is embedded with framework.WithNoUpdate

jar-b and others added 10 commits August 7, 2024 10:26
…pplication-sql

s3 datasource aws_s3_object can read application/x-sql
r/aws_ecs_service: Add `force_delete` argument
Co-authored-by: Adrian Johnson <adrian.johnson@hashicorp.com>
[BUG] `aws_batch_job_definition`: add extra nil checks
…terraform-provider-aws into f/datazone-form-type-resource

Merge conflicts
Copy link

github-actions bot commented Aug 7, 2024

Thank you for your contribution! 🚀

Please note that the CHANGELOG.md file contents are handled by the maintainers during merge. This is to prevent pull request merge conflicts, especially for contributions which may not be merged immediately. Please see the Contributing Guide for additional pull request review items.

Remove any changes to the CHANGELOG.md file and commit them in this pull request to prevent delays with reviewing and potentially merging this pull request.

Copy link

github-actions bot commented Aug 7, 2024

Thank you for your contribution! 🚀

Please note that the CHANGELOG.md file contents are handled by the maintainers during merge. This is to prevent pull request merge conflicts, especially for contributions which may not be merged immediately. Please see the Contributing Guide for additional pull request review items.

Remove any changes to the CHANGELOG.md file and commit them in this pull request to prevent delays with reviewing and potentially merging this pull request.

@jar-b jar-b deleted the f/datazone-form-type-resource branch August 14, 2024 17:59
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Introduces or discusses updates to documentation. generators Relates to code generators. prioritized Part of the maintainer teams immediate focus. To be addressed within the current quarter. service/datazone Issues and PRs that pertain to the datazone service. tests PRs: expanded test coverage. Issues: expanded coverage, enhancements to test infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.