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.

Sign up for GitHub

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

Jump to bottom

Add "ES_APPLICATION_LOGS" log_type for aws_elasticsearch_domain #5411

Closed
smelchior opened this issue Aug 1, 2018 · 3 comments · Fixed by #5474
Closed

Add "ES_APPLICATION_LOGS" log_type for aws_elasticsearch_domain #5411

smelchior opened this issue Aug 1, 2018 · 3 comments · Fixed by #5474
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/elasticsearch Issues and PRs that pertain to the elasticsearch service.
Milestone
v1.31.0

Comments

@smelchior
Copy link
Contributor

smelchior commented Aug 1, 2018 •
edited
Loading

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

AWS now allows to export the application logs of ES to Cloudwatch. I want to configure this with terraform.

New or Affected Resource(s)

  • aws_elasticsearch_domain

Potential Terraform Configuration

resource "aws_elasticsearch_domain" "es" {
.....
log_publishing_options {
log_type = "ES_APPLICATION_LOGS"
cloudwatch_log_group_arn  = "/...."
}
....
}

References
@bflad bflad added enhancement Requests to existing resources that expand the functionality or scope. service/elasticsearch Issues and PRs that pertain to the elasticsearch service. labels Aug 1, 2018
This was referenced Aug 8, 2018
Merged
Closed
@bflad bflad added this to the v1.31.0 milestone Aug 8, 2018
@bflad
Copy link
Contributor

bflad commented Aug 8, 2018

Support for this has been merged in and will release with version 1.31.0 of the AWS provider, likely later today. 👍

@bflad
Copy link
Contributor

bflad commented Aug 9, 2018

This has been released in version 1.31.0 of the AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

xbrianh pushed a commit to HumanCellAtlas/data-store that referenced this issue Oct 26, 2018
Accommodate TF changes plus minor infra updates
d442808 
New support for ES_APPLICATION_LOGS seems to have broken our infra defs.
Specifically enable it.
See hashicorp/terraform-provider-aws#5411
@xbrianh xbrianh mentioned this issue Oct 26, 2018
Merged
xbrianh added a commit to HumanCellAtlas/data-store that referenced this issue Oct 26, 2018
@xbrianh
Accommodate TF changes plus minor infra updates (#1673)
3ef6e2e 
New support for ES_APPLICATION_LOGS seems to have broken our infra defs.
Specifically enable it.
See hashicorp/terraform-provider-aws#5411
@bflad bflad mentioned this issue Apr 8, 2019
Closed
@ghost
Copy link

ghost commented Apr 4, 2020

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!

@ghost ghost locked and limited conversation to collaborators Apr 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/elasticsearch Issues and PRs that pertain to the elasticsearch service.
Projects
None yet
Milestone
v1.31.0
Development

Successfully merging a pull request may close this issue.

Add "ES_APPLICATION_LOGS" log_type for aws_elasticsearch_domain sgrodzicki/terraform-provider-aws
2 participants
@bflad @smelchior