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

Add support for other arguments in aws_dms_endpoint kinesis_settings #20083

Closed
ksagle77 opened this issue Jul 6, 2021 · 5 comments · Fixed by #20084 or #20904
Closed

Add support for other arguments in aws_dms_endpoint kinesis_settings #20083

ksagle77 opened this issue Jul 6, 2021 · 5 comments · Fixed by #20084 or #20904
Labels
enhancement Requests to existing resources that expand the functionality or scope.
Milestone

Comments

@ksagle77
Copy link

ksagle77 commented Jul 6, 2021

Description

Currently aws/dms_endpoint kinesis_settings only supports message_format, service_access_role_arn, and stream_arn. I would like to add support for IncludeNullAndEmpty. I am not sure whether there would be a way to query DMS to discover available settings, or whether IncludeNullAndEmpty should be hard coded in to the existing kinesis settings block.

New or Affected Resource(s)

Potential Terraform Configuration

provider "aws" {
  region                  = "us-east-2"
  profile                 = "aws"
}

resource "aws_iam_role" "this" {  
  name               = "dms-role"
  assume_role_policy = <<EOF
{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Sid": "",
      "Effect": "Allow",
      "Principal": {
        "Service": [
          "dms.amazonaws.com"
        ]
      },
      "Action": "sts:AssumeRole"
    }
  ]
}
EOF
}

resource "aws_kinesis_stream" "this" {
  name             = "dms-kinesis"
  shard_count      = 1
}

resource "aws_dms_endpoint" "target" {
  endpoint_id   = "test-kinesis"
  endpoint_type = "target"
  engine_name   = "kinesis"
  ssl_mode      = "none"
  kinesis_settings {
    stream_arn               = aws_kinesis_stream.this.arn
    message_format           = "json"
    service_access_role_arn  = aws_iam_role.this.arn
    include_null_and_empty   = true
  }  
}

References

  • #0000
@ksagle77 ksagle77 added the enhancement Requests to existing resources that expand the functionality or scope. label Jul 6, 2021
@github-actions github-actions bot added needs-triage Waiting for first response or review from a maintainer. service/databasemigrationservice service/iam Issues and PRs that pertain to the iam service. service/kinesis Issues and PRs that pertain to the kinesis service. labels Jul 6, 2021
@gdavison gdavison removed needs-triage Waiting for first response or review from a maintainer. service/iam Issues and PRs that pertain to the iam service. service/kinesis Issues and PRs that pertain to the kinesis service. labels Jul 6, 2021
@vikatskhay
Copy link

Would it be possible to add support for IncludeTransactionDetails as well?

@ksagle77
Copy link
Author

ksagle77 commented Jul 9, 2021

In addition to IncludeNullAndEmpty and IncludeTransactionDetails, it makes sense to add support for the other available kinesis settings:

IncludeControlDetails
IncludePartitionValue
IncludeTableAlterOperations
PartitionIncludeSchemaTable

Will update the PR to reflect that.

@anGie44 anGie44 changed the title kinesis_settings Add support for other arguments in aws_dms_endpoint kinesis_settings Aug 18, 2021
@ewbankkit

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions
Copy link

github-actions bot commented Jun 2, 2022

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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 2, 2022
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.
Projects
None yet
4 participants