-
Notifications
You must be signed in to change notification settings - Fork 9.2k
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: aws_imagebuilder_component #16169
Conversation
# Conflicts: # aws/provider.go
# Conflicts: # aws/provider.go
# Conflicts: # aws/provider.go
…ing set on create
fixed lint issues and terminate_instance_on_failure
…vider-aws into Dogers-image-builder
Reference: #11084 Reference: #13315 Changes: ``` * **New Data Source:** `aws_imagebuilder_component` * **New Resource:** `aws_imagebuilder_component` ``` Output from acceptance testing in AWS Commercial: ``` --- PASS: TestAccAwsImageBuilderComponent_basic (18.98s) --- PASS: TestAccAwsImageBuilderComponent_ChangeDescription (19.15s) --- PASS: TestAccAwsImageBuilderComponent_Description (19.29s) --- PASS: TestAccAwsImageBuilderComponent_disappears (15.10s) --- PASS: TestAccAwsImageBuilderComponent_KmsKeyId (23.58s) --- PASS: TestAccAwsImageBuilderComponent_Platform_Windows (19.22s) --- PASS: TestAccAwsImageBuilderComponent_SupportedOsVersions (19.15s) --- PASS: TestAccAwsImageBuilderComponent_Tags (42.96s) --- PASS: TestAccAwsImageBuilderComponent_Uri (35.66s) --- PASS: TestAccAwsImageBuilderComponentDataSource_Arn (18.51s) ``` Output from acceptance testing in AWS GovCloud (US): ``` --- PASS: TestAccAwsImageBuilderComponent_basic (24.13s) --- PASS: TestAccAwsImageBuilderComponent_ChangeDescription (23.39s) --- PASS: TestAccAwsImageBuilderComponent_Description (24.43s) --- PASS: TestAccAwsImageBuilderComponent_disappears (19.28s) --- PASS: TestAccAwsImageBuilderComponent_KmsKeyId (28.49s) --- PASS: TestAccAwsImageBuilderComponent_Platform_Windows (24.54s) --- PASS: TestAccAwsImageBuilderComponent_SupportedOsVersions (24.05s) --- PASS: TestAccAwsImageBuilderComponent_Tags (53.99s) --- PASS: TestAccAwsImageBuilderComponent_Uri (40.78s) --- PASS: TestAccAwsImageBuilderComponentDataSource_Arn (24.26s) ``` Output from sweeper in AWS Commercial: ``` 2020/11/12 16:58:32 [DEBUG] Running Sweepers for region (us-west-2): 2020/11/12 16:58:32 [DEBUG] Running Sweeper (aws_imagebuilder_component) in region (us-west-2) 2020/11/12 16:58:45 Sweeper Tests ran successfully: - aws_imagebuilder_component 2020/11/12 16:58:45 [DEBUG] Running Sweepers for region (us-east-1): 2020/11/12 16:58:45 [DEBUG] Running Sweeper (aws_imagebuilder_component) in region (us-east-1) 2020/11/12 16:58:46 Sweeper Tests ran successfully: - aws_imagebuilder_component ok github.com/terraform-providers/terraform-provider-aws/aws 18.223s ``` Output from sweeper in AWS GovCloud (US): ``` 2020/11/12 16:59:00 [DEBUG] Running Sweepers for region (us-gov-west-1): 2020/11/12 16:59:00 [DEBUG] Running Sweeper (aws_imagebuilder_component) in region (us-gov-west-1) 2020/11/12 16:59:03 Sweeper Tests ran successfully: - aws_imagebuilder_component ok github.com/terraform-providers/terraform-provider-aws/aws 5.004s ```
Reference: #11084 Reference: #13485 Reference: #16169 (required for `aws_imagebuilder_component` resource implementation) Changes: ``` * **New Data Source:** `aws_imagebuilder_image_recipe` * **New Resource:** `aws_imagebuilder_image_recipe` ``` Output from acceptance testing in AWS Commercial: ``` --- PASS: TestAccAwsImageBuilderImageRecipe_basic (38.20s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_DeviceName (35.14s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_DeleteOnTermination (34.64s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Encrypted (33.35s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Iops (36.82s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_KmsKeyId (35.17s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_SnapshotId (57.27s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeSize (35.26s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeType (37.01s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_NoDevice (35.18s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_VirtualName (34.55s) --- PASS: TestAccAwsImageBuilderImageRecipe_Description (34.98s) --- PASS: TestAccAwsImageBuilderImageRecipe_disappears (31.67s) --- PASS: TestAccAwsImageBuilderImageRecipe_Tags (59.07s) --- PASS: TestAccAwsImageBuilderImageRecipeDataSource_Arn (19.90s) ``` Output from acceptance testing in AWS GovCloud (US): ``` --- PASS: TestAccAwsImageBuilderImageRecipe_basic (25.73s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_DeviceName (39.43s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_DeleteOnTermination (39.12s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Encrypted (39.21s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Iops (39.20s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_KmsKeyId (39.29s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_SnapshotId (59.68s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeSize (39.29s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeType (38.94s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_NoDevice (38.10s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_VirtualName (39.09s) --- PASS: TestAccAwsImageBuilderImageRecipe_Description (38.84s) --- PASS: TestAccAwsImageBuilderImageRecipe_disappears (34.09s) --- PASS: TestAccAwsImageBuilderImageRecipe_Tags (68.88s) --- PASS: TestAccAwsImageBuilderImageRecipeDataSource_Arn (24.71s) ``` Output from sweeper in AWS Commercial: ``` 2020/11/16 15:31:54 [DEBUG] Running Sweepers for region (us-west-2): 2020/11/16 15:31:54 [DEBUG] Running Sweeper (aws_imagebuilder_image_recipe) in region (us-west-2) 2020/11/16 15:31:56 Sweeper Tests ran successfully: - aws_imagebuilder_image_recipe 2020/11/16 15:31:56 [DEBUG] Running Sweepers for region (us-east-1): 2020/11/16 15:31:56 [DEBUG] Running Sweeper (aws_imagebuilder_image_recipe) in region (us-east-1) 2020/11/16 15:31:57 Sweeper Tests ran successfully: - aws_imagebuilder_image_recipe ok github.com/terraform-providers/terraform-provider-aws/aws 6.199s ``` Output from sweeper in AWS GovCloud (US): ``` 2020/11/16 15:32:04 [DEBUG] Running Sweepers for region (us-gov-west-1): 2020/11/16 15:32:04 [DEBUG] Running Sweeper (aws_imagebuilder_image_recipe) in region (us-gov-west-1) 2020/11/16 15:32:06 Sweeper Tests ran successfully: - aws_imagebuilder_image_recipe ok github.com/terraform-providers/terraform-provider-aws/aws 4.799s ```
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
A couple cases of fmt.Errorf("...%s", ..., err)
Commercial partition:
--- PASS: TestAccAwsImageBuilderComponent_disappears (42.98s)
--- PASS: TestAccAwsImageBuilderComponent_KmsKeyId (56.76s)
--- PASS: TestAccAwsImageBuilderComponent_SupportedOsVersions (57.21s)
--- PASS: TestAccAwsImageBuilderComponentDataSource_Arn (57.05s)
--- PASS: TestAccAwsImageBuilderComponent_basic (57.16s)
--- PASS: TestAccAwsImageBuilderComponent_Description (58.27s)
--- PASS: TestAccAwsImageBuilderComponent_ChangeDescription (58.43s)
--- PASS: TestAccAwsImageBuilderComponent_Platform_Windows (58.96s)
--- PASS: TestAccAwsImageBuilderComponent_Uri (59.75s)
--- PASS: TestAccAwsImageBuilderComponent_Tags (85.60s)
GovCloud partition:
--- PASS: TestAccAwsImageBuilderComponent_disappears (19.97s)
--- PASS: TestAccAwsImageBuilderComponent_Description (26.93s)
--- PASS: TestAccAwsImageBuilderComponentDataSource_Arn (27.72s)
--- PASS: TestAccAwsImageBuilderComponent_Platform_Windows (28.48s)
--- PASS: TestAccAwsImageBuilderComponent_SupportedOsVersions (29.18s)
--- PASS: TestAccAwsImageBuilderComponent_ChangeDescription (29.06s)
--- PASS: TestAccAwsImageBuilderComponent_KmsKeyId (29.20s)
--- PASS: TestAccAwsImageBuilderComponent_basic (29.30s)
--- PASS: TestAccAwsImageBuilderComponent_Uri (29.53s)
--- PASS: TestAccAwsImageBuilderComponent_Tags (42.20s)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixing the ones I see
This has been released in version 3.16.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks! |
Reference: #11084 Reference: #13485 Reference: #16169 (required for `aws_imagebuilder_component` resource implementation) Changes: ``` * **New Data Source:** `aws_imagebuilder_image_recipe` * **New Resource:** `aws_imagebuilder_image_recipe` ``` Output from acceptance testing in AWS Commercial: ``` --- PASS: TestAccAwsImageBuilderImageRecipe_basic (38.20s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_DeviceName (35.14s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_DeleteOnTermination (34.64s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Encrypted (33.35s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Iops (36.82s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_KmsKeyId (35.17s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_SnapshotId (57.27s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeSize (35.26s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeType (37.01s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_NoDevice (35.18s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_VirtualName (34.55s) --- PASS: TestAccAwsImageBuilderImageRecipe_Description (34.98s) --- PASS: TestAccAwsImageBuilderImageRecipe_disappears (31.67s) --- PASS: TestAccAwsImageBuilderImageRecipe_Tags (59.07s) --- PASS: TestAccAwsImageBuilderImageRecipeDataSource_Arn (19.90s) ``` Output from acceptance testing in AWS GovCloud (US): ``` --- PASS: TestAccAwsImageBuilderImageRecipe_basic (25.73s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_DeviceName (39.43s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_DeleteOnTermination (39.12s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Encrypted (39.21s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_Iops (39.20s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_KmsKeyId (39.29s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_SnapshotId (59.68s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeSize (39.29s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_Ebs_VolumeType (38.94s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_NoDevice (38.10s) --- PASS: TestAccAwsImageBuilderImageRecipe_BlockDeviceMapping_VirtualName (39.09s) --- PASS: TestAccAwsImageBuilderImageRecipe_Description (38.84s) --- PASS: TestAccAwsImageBuilderImageRecipe_disappears (34.09s) --- PASS: TestAccAwsImageBuilderImageRecipe_Tags (68.88s) --- PASS: TestAccAwsImageBuilderImageRecipeDataSource_Arn (24.71s) ``` Output from sweeper in AWS Commercial: ``` 2020/11/16 15:31:54 [DEBUG] Running Sweepers for region (us-west-2): 2020/11/16 15:31:54 [DEBUG] Running Sweeper (aws_imagebuilder_image_recipe) in region (us-west-2) 2020/11/16 15:31:56 Sweeper Tests ran successfully: - aws_imagebuilder_image_recipe 2020/11/16 15:31:56 [DEBUG] Running Sweepers for region (us-east-1): 2020/11/16 15:31:56 [DEBUG] Running Sweeper (aws_imagebuilder_image_recipe) in region (us-east-1) 2020/11/16 15:31:57 Sweeper Tests ran successfully: - aws_imagebuilder_image_recipe ok github.com/terraform-providers/terraform-provider-aws/aws 6.199s ``` Output from sweeper in AWS GovCloud (US): ``` 2020/11/16 15:32:04 [DEBUG] Running Sweepers for region (us-gov-west-1): 2020/11/16 15:32:04 [DEBUG] Running Sweeper (aws_imagebuilder_image_recipe) in region (us-gov-west-1) 2020/11/16 15:32:06 Sweeper Tests ran successfully: - aws_imagebuilder_image_recipe ok github.com/terraform-providers/terraform-provider-aws/aws 4.799s ``` Co-authored-by: Dogers <Dogers@users.noreply.github.com> Co-authored-by: Jakub Kania <jakubkania@blackcat.rocks> Co-authored-by: Bill Schneider <bill.schneider@optum.com> Co-authored-by: Graham Davison <gdavison@hashicorp.com>
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks! |
Community Note
Reference: #11084
Closes #13315
Reference: #13485
Release note for CHANGELOG:
Output from acceptance testing in AWS Commercial:
Output from acceptance testing in AWS GovCloud (US):
Output from sweeper in AWS Commercial:
Output from sweeper in AWS GovCloud (US):