Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore(fvt): tweak to work across more versions #2615

Merged
merged 2 commits into from
Aug 21, 2023
Merged

chore(fvt): tweak to work across more versions #2615

merged 2 commits into from
Aug 21, 2023

Conversation

dnwe
Copy link
Collaborator

@dnwe dnwe commented Aug 21, 2023

  • update the Dockerfile and docker-compose.yml to ensure it works across
    a wider range of KAFKA_VERSIONs all on Java 11 by patching and setting
    envvars
  • ensure hostnames are set to avoid reverse lookups on older kafkas
  • switch to kafka-api-versions for HEALTHCHECK as that was available
    with the same cmdline args for all versions

dnwe added 2 commits August 21, 2023 18:23
- update the Dockerfile and docker-compose.yml to ensure it works across
  a wider range of KAFKA_VERSIONs all on Java 11 by patching and setting
  envvars
- ensure hostnames are set to avoid reverse lookups on older kafkas
- switch to kafka-api-versions for HEALTHCHECK as that was available
  with the same cmdline args for all versions

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
A couple were missing Config Version and the zstd test only wowrks on
kafka 2.1.0 and newer so needed a checkKafkaVersion call

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
@dnwe dnwe merged commit 2b54832 into main Aug 21, 2023
@dnwe dnwe deleted the dnwe/fvt branch August 21, 2023 17:55
@dnwe dnwe added the ci label Aug 21, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant