s3-connector-for-apache-kafka icon indicating copy to clipboard operation
s3-connector-for-apache-kafka copied to clipboard

Use default or custom provider chain for s3 authentication

Open steephengeorge opened this issue 2 years ago • 13 comments

steephengeorge avatar Nov 27 '22 19:11 steephengeorge

Seems that this is likely to address Aiven-Open/commons-for-apache-kafka-connect#275

nikmmd avatar Nov 29 '22 16:11 nikmmd

Dear team, this is a very expected update. Is it planned to be merged? Thanks!

mikitakandratsiuk avatar Dec 22 '22 15:12 mikitakandratsiuk

@ivanyu my team require this change in order to deploy in a production environment. What is needed to have this merged?

ampem avatar Aug 03 '23 09:08 ampem

Hi! Sorry for it stayed unattended, I'll get someone from our team to review.

ivanyu avatar Aug 03 '23 09:08 ivanyu

We will also probably need this change for deploying s3 sink connectors on eks with a service account role.

SubhraB avatar Aug 07 '23 17:08 SubhraB

@ivanyu Please let us now ETA for getting this merged and released.

SubhraB avatar Aug 09 '23 18:08 SubhraB

I know this has been dormant for a while.

Our team needs this change. @steephengeorge if you are no longer available to make the change requested by @ivanyu, @ivanyu are you ok if someone from my team here @ComplyAdvantage takes on this PR and re-submits with your requested changes?

markallanson avatar Jan 10 '24 15:01 markallanson

I will do it today. Sorry for the delay . Sent from my iPhoneOn Jan 10, 2024, at 7:24 AM, Mark Allanson @.***> wrote: I know this has been dormant for a while. Our team needs this change. @steephengeorge if you are no longer available to make the change requested by @ivanyu, @ivanyu are you ok if someone from my team here @ComplyAdvantage takes on this PR and re-submits with your requested changes?

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

steephengeorge avatar Jan 10 '24 15:01 steephengeorge

Thanks @steephengeorge - greatly appreciated! IT looks like there's some merge conflicts with the target branch that need resolving too though :(

markallanson avatar Jan 10 '24 16:01 markallanson

@markallanson this is taken care.

steephengeorge avatar Jan 10 '24 17:01 steephengeorge

@jeqo I will get someone on my team to pick up this change as soon as we can - we'll take what's been done here so far (thanks @steephengeorge for what you've done so far), apply your suggested changes and raise another PR with the changes.

markallanson avatar Jan 26 '24 15:01 markallanson

@jeqo how close do you think we are here? Is it just the readme left to update?

markallanson avatar Apr 03 '24 15:04 markallanson

We moved to confluent based s3 connector, works fine.

SubhraB avatar Apr 03 '24 15:04 SubhraB

Closing, Aiven-Open/s3-connector-for-apache-kafka#355 merged.

jjaakola-aiven avatar May 24 '24 11:05 jjaakola-aiven