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.

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

processing_configuration not available for AWS Elasticsearch Firehose Delivery Stream #2729

Closed
hashibot opened this issue Dec 21, 2017 · 4 comments
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/firehose Issues and PRs that pertain to the firehose service.
Milestone

Comments

@hashibot
Copy link

This issue was originally opened by @jmlw as hashicorp/terraform#16963. It was migrated here as a result of the provider split. The original body of the issue is below.


Terraform Version

Terraform v0.11.1
* provider.aws: version = "~> 1.6"
* provider.template: version = "~> 1.0"

Terraform Configuration Files

resource "aws_kinesis_firehose_delivery_stream" "elasticsearch_ingestion_stream" {
  name        = "elasticsearch_ingestion_stream"
  destination = "elasticsearch"

  kinesis_source_configuration {
    kinesis_stream_arn = "kinesis-stream-arn"
    role_arn           = "firehose_iam_role"
  }

  s3_configuration {
    role_arn           = "firehose-iam-role"
    bucket_arn         = "arn:aws:s3:::bucket"
    prefix             = "logs/logs"
    buffer_size        = 10
    buffer_interval    = 400
    compression_format = "GZIP"
  }

  elasticsearch_configuration {
    domain_arn = "domain"
    role_arn   = "role"
    index_name = "index"
    type_name  = "doc_type"
    buffering_interval = 60
    index_rotation_period = "OneDay"
    retry_duration = 900
  }
}
resource "aws_kinesis_firehose_delivery_stream" "s3_ingestion_stream" {
  name        = "s3_ingestion_stream"
  destination = "extended_s3"

  kinesis_source_configuration {
    kinesis_stream_arn = "kinesis-stream-arn"
    role_arn           = "firehose_iam_role"
  }

  extended_s3_configuration {
    role_arn                 = "firehost_iam_role"
    bucket_arn               = "arn:aws:s3:::bucket"
    prefix                   = "logs/logs"
    processing_configuration = [
      {
        enabled = "true"
        processors = [
          {
            type = "Lambda"
            parameters = [
              {
                parameter_name  = "LambdaArn"
                parameter_value = "transformer_lambda"
              }
            ]
          }
        ]
      }
    ]
  }
}

Debug Output

Not applicable

Crash Output

Not Applicable

Expected Behavior

Should be able to define lambda transformer for Elasticsearch destination the same way a processor can be defined for extended s3 destinations

Actual Behavior

processing_configuration is not a valid key for elasticsearch_configuration and is not available under any other key

@jen20 jen20 added the enhancement Requests to existing resources that expand the functionality or scope. label Dec 29, 2017
@radeksimko radeksimko added the service/firehose Issues and PRs that pertain to the firehose service. label Jan 15, 2018
@knope
Copy link

knope commented Mar 14, 2018

My team is also hitting this issue now. The method of CWL -through firehose subscription-> AWSES requires at the very minimum a lambda processor in order to decompress the CWL data.

Edit: Word on the street is this may get worked in some time soon. 💯

@knope
Copy link

knope commented Apr 18, 2018

this looks relevant #3621 ❣️

and updated doc includes: https://www.terraform.io/docs/providers/aws/r/kinesis_firehose_delivery_stream.html

  elasticsearch_configuration {
    domain_arn = "${aws_elasticsearch_domain.test_cluster.arn}"
    role_arn   = "${aws_iam_role.firehose_role.arn}"
    index_name = "test"
    type_name  = "test"

    processing_configuration = [
      {
        enabled = "true"
        processors = [
          {
            type = "Lambda"
            parameters = [
              {
                parameter_name = "LambdaArn"
                parameter_value = "${aws_lambda_function.lambda_processor.arn}:$LATEST"
              }
            ]
          }
        ]
      }
    ]
  }

@bflad bflad added this to the v1.14.0 milestone Apr 18, 2018
@bflad
Copy link
Contributor

bflad commented Apr 18, 2018

Thanks for the bump @knope! This was previously released in version 1.14.0 of the AWS provider and has been available in all releases since. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

@ghost
Copy link

ghost commented Apr 6, 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 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement Requests to existing resources that expand the functionality or scope. service/firehose Issues and PRs that pertain to the firehose service.
Projects
None yet
Development

No branches or pull requests

5 participants