s3-connector-for-apache-kafka
s3-connector-for-apache-kafka copied to clipboard
Use default or custom provider chain for s3 authentication
Seems that this is likely to address Aiven-Open/commons-for-apache-kafka-connect#275
Dear team, this is a very expected update. Is it planned to be merged? Thanks!
@ivanyu my team require this change in order to deploy in a production environment. What is needed to have this merged?
Hi! Sorry for it stayed unattended, I'll get someone from our team to review.
We will also probably need this change for deploying s3 sink connectors on eks with a service account role.
@ivanyu Please let us now ETA for getting this merged and released.
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?
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: @.***>
Thanks @steephengeorge - greatly appreciated! IT looks like there's some merge conflicts with the target branch that need resolving too though :(
@markallanson this is taken care.
@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.
@jeqo how close do you think we are here? Is it just the readme left to update?
We moved to confluent based s3 connector, works fine.
Closing, Aiven-Open/s3-connector-for-apache-kafka#355 merged.