Merge pull request #5626 from terraform-providers/d-aws_ami-owners-re…

…quired-soon data-source/aws_ami: Add note that owners argument will be required in next major version
hashicorp · Aug 21, 2018 · b313d17 · b313d17
2 parents 145a0ad + 1c66fb8
commit b313d17
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 0 deletions.
diff --git a/website/docs/d/ami.html.markdown b/website/docs/d/ami.html.markdown
@@ -11,6 +11,8 @@ description: |-
 Use this data source to get the ID of a registered AMI for use in other
 resources.
 
+~> **NOTE:** The `owners` argument will be **required** in the next major version.
+
 ## Example Usage
 
 ```hcl

diff --git a/website/docs/d/ami_ids.html.markdown b/website/docs/d/ami_ids.html.markdown
@@ -10,6 +10,8 @@ description: |-
 
 Use this data source to get a list of AMI IDs matching the specified criteria.
 
+~> **NOTE:** The `owners` argument will be **required** in the next major version.
+
 ## Example Usage
 
 ```hcl

diff --git a/website/docs/guides/version-2-upgrade.html.md b/website/docs/guides/version-2-upgrade.html.md
@@ -19,6 +19,8 @@ Upgrade topics:
 <!-- TOC depthFrom:2 depthTo:2 -->
 
 - [Provider Version Configuration](#provider-version-configuration)
+- [Data Source: aws_ami](#data-source-aws_ami)
+- [Data Source: aws_ami_ids](#data-source-aws_ami_ids)
 - [Data Source: aws_iam_role](#data-source-aws_iam_role)
 - [Data Source: aws_kms_secret](#data-source-aws_kms_secret)
 - [Data Source: aws_region](#data-source-aws_region)
@@ -70,6 +72,18 @@ provider "aws" {
 }
 ```
 
+## Data Source: aws_ami
+
+### owners Argument Now Required
+
+The `owners` argument is now required. Specifying `owner-id` or `owner-alias` under `filter` does not satisfy this requirement.
+
+## Data Source: aws_ami_ids
+
+### owners Argument Now Required
+
+The `owners` argument is now required. Specifying `owner-id` or `owner-alias` under `filter` does not satisfy this requirement.
+
 ## Data Source: aws_iam_role
 
 ### assume_role_policy_document Attribute Removal