Francisco Vila

[issue #275] fix waiting for consumers to be ready

1

Following the issue #275 already closed, I believe that it is better to throw an exception when the timeout is reached instead of logging a warning. I got an issue...

update run-example script to use helm to install strimzi

1

### Type of change - Bugfix ### Description Fixes #1212. We need to refactor `run-example.sh` script to use helm and select the version of strimzi that we are installing instead...

I really like the direction this is headed. Thank you. We should probably refactor run-example.sh to install strimzi using Helm too as all the arguments you make apply to it...

String return type is not good. It means the test needs to know about the format of the output created by the consumer in order to separate the different records....

kroxylicious-admin user permissions for Record Encryption filter

3

**Suggestion / Problem** Following the official [documentation](https://kroxylicious.io/docs/v0.6.0/#administrator-actor-2) of Kroxylicious, when implementing system tests, I found that we cannot delete the KEK created to keep everything clean, so I need to...

We should push this into `KroxyliciousSteps`. _Originally posted by @SamBarker in https://github.com/kroxylicious/kroxylicious/pull/1391#discussion_r1704839320_ It is a good idea to move all the deployment process to the `KroxyliciousSteps` so that we can...

[System Tests] why is our test impl validating if alias exists when deleting the KEK

1

I appreciate this is just moved code, but why is our test impl validating this? Shouldn't we just forward it to the backend and let it decide? As currently we...

[System Tests] Refactor TestKmFacade to avoid duplicated code

1

This looks very like the AWS version ;) Could we have single TestKekManager which delegated to one of `[VaultLocal,VaultCloud,LocalStack,AWS]` ? I'm not entirely sure how the factory wiring would work...

[System Test] Rename `resources` package as resources is used in maven projects

1

Certainly a thought for another day but a `resources` package feels a bit odd (I think I follow the naming) as resources in maven projects is usually `src/[main|test]/resources/` and filled...

[System Tests] Refactor `Kube` bifurcation for `TestKmsFacade` implementation

5

Rightly or wrongly, for Vault we've got two implementation of each facade, one where the container runs with Docker (`VaultTestKmsFacade`) and one where the container runs as pod (`KubeVaultTestKmsFacade`). In...