Skip to content

Commit

Permalink
Merge pull request #21738 from hashicorp/elasticsearch-domain-tags-test
Browse files Browse the repository at this point in the history
r/elasticsearch_domin: use tags as `KeyValueTags` type instead of as `[]*elasticsearchservice.Tag` in `UpdateTags`
  • Loading branch information
anGie44 authored Nov 11, 2021
2 parents 8b3c18a + 7a5bb2a commit 46a70eb
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .changelog/21738.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:bug
resource/aws_elasticsearch_domain: Fix tagging on creation
```
2 changes: 1 addition & 1 deletion internal/service/elasticsearch/domain.go
Original file line number Diff line number Diff line change
Expand Up @@ -626,7 +626,7 @@ func resourceDomainCreate(d *schema.ResourceData, meta interface{}) error {
// whilst the operation is being performed), we still get the required tags on
// the resources.
if len(tags) > 0 {
if err := UpdateTags(conn, d.Id(), nil, Tags(tags.IgnoreAWS())); err != nil {
if err := UpdateTags(conn, d.Id(), nil, tags); err != nil {
return fmt.Errorf("error adding Elasticsearch Cluster (%s) tags: %s", d.Id(), err)
}
}
Expand Down

0 comments on commit 46a70eb

Please sign in to comment.