data-source/aws_kms_alias: Improve comment about aliases missing Targ…

…etKeyId
hashicorp · Jan 31, 2018 · 19b34df · 19b34df
1 parent fac79d5
commit 19b34df
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/aws/data_source_aws_kms_alias.go b/aws/data_source_aws_kms_alias.go
@@ -64,7 +64,8 @@ func dataSourceAwsKmsAliasRead(d *schema.ResourceData, meta interface{}) error {
 	d.SetId(time.Now().UTC().String())
 	d.Set("arn", alias.AliasArn)
 
-	// AWS service aliases do not return TargetKeyId:
+	// Some aliases do not return TargetKeyId (e.g. aliases for AWS services or
+	// aliases not associated with a Customer Managed Key (CMK))
 	// https://docs.aws.amazon.com/kms/latest/APIReference/API_ListAliases.html
 	if alias.TargetKeyId != nil {
 		aliasARN, err := arn.Parse(*alias.AliasArn)