kafka-connect-elasticsearch icon indicating copy to clipboard operation
kafka-connect-elasticsearch copied to clipboard

Feature/aws signer

Open arunx2 opened this issue 3 years ago • 9 comments

Borrowed most code from https://github.com/confluentinc/kafka-connect-elasticsearch/pull/330/ as it is old and conflict with current code.

Problem

Connecting to Amazon Managed service with IAM role authentication addressing feature request: https://github.com/confluentinc/kafka-connect-elasticsearch/issues/514

Solution

Does this solution apply anywhere else?
  • [ ] yes
  • [x] no
If yes, where?

Test Strategy

Create a Amazon ES cluster with IAM role authentication. Then create a sink connector (have the aws_access_key_id and aws_secret_access_key in the ENV variables).

Testing done:
  • [x] Unit tests
  • [ ] Integration tests
  • [ ] System tests
  • [x] Manual tests

Release Plan

merging to master. so that it can be available in next version.

yes it is safe to rollback.

arunx2 avatar Jun 01 '21 18:06 arunx2

@confluentinc It looks like @arunx2 just signed our Contributor License Agreement. :+1:

Always at your service,

clabot

ghost avatar Jun 01 '21 18:06 ghost

Hope to get this rolled out soon. Thx !!!

rldeep2889 avatar Jun 01 '21 19:06 rldeep2889

Any plan to have this PR merged ?

thibthibus avatar Dec 10 '21 11:12 thibthibus

Can we get this PR merged? We are also looking for this functionality

nomeelnoj avatar Jan 14 '22 17:01 nomeelnoj

@ Dev/Product Team, could you please give an update on this PR ? Why is it not considered for merging ?

thibthibus avatar Mar 07 '22 11:03 thibthibus

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Jun 23 '22 19:06 CLAassistant

Anything I can do to help get this merged?

gaustin avatar Dec 08 '22 20:12 gaustin

Any news about this feature? This is now necessary to use OpenSearch Serverless.

cosimomeli avatar Jun 08 '23 14:06 cosimomeli

Any news about when this will be merged?

PriyankaKarnati avatar Oct 26 '23 10:10 PriyankaKarnati