Keith Wall

Results 309 comments of Keith Wall

There are two problems to solve. 1. the AWS KMS SerDer currently serializes the`KeyId` into the kafka record. `KeyIds` are scoped to the AWS account. 2. record encryption's KEK selector...

## EKS >if the proxy binary is running in Amazon EKS, [IAM roles can be attached to service accounts ](https://docs.aws.amazon.com/eks/latest/userguide/iam-roles-for-service-accounts.html)but I don't see in the AWS docs a reference to...

> I don't know the reasons of using HTTP requests for integration tests, but I wonder if we could use CLI as well. WDYT? We should favour integration solutions that...

We could take a step towards this by replacing use of the CLI with REST API. The would eliminate a quite a lot of code. We could then think about...

@franvila As we now exclusively use the REST API,I not really sure that there's a problem to solve right now. I'd suggest we just close this as completed. WDYT?

@franvila Personally, I think resources as part of the package name in this context is okay. I don't think there is risk of confusion. I suggest we close this issue.

This issue might be better rolled into the other issues we have discussing dynamic config reloading.

I don't think the test failure showing in CI relates to my chnage. ``` Errors: Error: JdkHttpLoggingInterceptorTest>AbstractHttpLoggingInterceptorTest.httpLargeResponseBodyLogged:205 » Execution java.net.ProtocolException: protocol error: Frame type(114) length(6776096) exceeds MAX_FRAME_SIZE(16384) [INFO] ```

Not sure if the Quality Gate being B means this cannot be merged. I looked at the issues being flagged by Sonar: * java:S107 - constructor parameter count rule doesn't...

> I'm not sure if I'll be releasnig a 7.3.1 version tomorrow. Once that's clear I'll merge this one. I see main is now versioned 7.4 and this PR is...