cp-helm-charts icon indicating copy to clipboard operation
cp-helm-charts copied to clipboard

[cp-kafka-connect] Add ServiceAccount

Open niqdev opened this issue 5 years ago • 9 comments

What changes were proposed in this pull request?

Add ServiceAccount for cp-kafka-connect.

We are using a custom kafkaConfigProvider to retrieve the credentials via SSM to access and pull the data from Oracle using kafka-connect-jdbc. To give fine-grained permissions we need to bind an IAM role to the service account.

How was this patch tested?

Tested using helm on AWS EKS

niqdev avatar Oct 25 '19 17:10 niqdev

It looks like @niqdev hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence. Wikipedia

You can read and sign our full Contributor License Agreement here.

Once you've signed reply with [clabot:check] to prove it.

Appreciation of efforts,

clabot

ghost avatar Oct 25 '19 17:10 ghost

[clabot:check]

niqdev avatar Oct 25 '19 17:10 niqdev

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

Always at your service,

clabot

ghost avatar Oct 25 '19 17:10 ghost

@gAmUssA @mohnishbasha any chance you could review this PR or is it possible to assign it to someone? We are pointing to my fork in work and I'd like to switch back to the official chart. Thanks a lot

niqdev avatar Nov 27 '19 17:11 niqdev

Bumping this up, can someone please take a look?

sirocchj avatar Jan 16 '20 16:01 sirocchj

@niqdev @globulon @gAmUssA any chances to merge it?

alexnikitchuk avatar Dec 21 '21 13:12 alexnikitchuk

Bumping! Any news about it?

ops-hummus avatar Apr 05 '22 10:04 ops-hummus

Any news ? We need this ASAP

denzhel avatar Jan 03 '24 08:01 denzhel

Update anyone?

J00MZ avatar Jan 03 '24 12:01 J00MZ