strowk
strowk
Not quite sure how to reproduce this. But probably the issue is caused by this change- https://github.com/testcontainers/testcontainers-go/pull/1971 This bit: > If it's too low the http connection to get the...
Hm, actually I an not so sure if explicitly 0.28 is an issue. I tried couple versions and I see different results: in 0.28 there is plenty of useful info...
I have bisected the problem up to version 0.32.0 . It was fine in 0.31.0 . The specific reasoning for this inconvenient (at least for me) design decision comes from...
For comparison: on my machine currently, `TestKafka` in kafka_native runs in 3.6 seconds, while original module test runs in 16.6 seconds. On big codebases and weaker machines this could translate...
Sounds ok, though not sure what would interface look like
@mabrarov , if I understand correctly some earlier concerns from maintainers were that removing confluent image from existing module would be not backwards compatible change for users that rely on...
@mabrarov Ooh, thanks for clarification, I did not know that there separate apache/kafka is somehow significantly better than the native one. This does mean that it would be better for...
@mabrarov , I suppose that deriving what to do from parsing the image is a common answer in codebase as I can see: https://github.com/testcontainers/testcontainers-go/blob/main/modules/elasticsearch/version.go https://github.com/testcontainers/testcontainers-go/blob/ec51c6715cca41d30daacc7519118111a15ea623/modules/localstack/localstack.go#L32 https://github.com/testcontainers/testcontainers-go/blob/ec51c6715cca41d30daacc7519118111a15ea623/modules/redpanda/redpanda.go#L439 Though it is usually...
@mabrarov , I have made an attempt at this based on a very simple logic that looks like this: ```go const ( apacheKafkaImagePrefix = "apache/kafka" confluentincImagePrefix = "confluentinc/" dockerIoPrefix =...
Hey @mdelapenya , I have made a lot of changes to this: - new stuff is all added into existing "kafka" module, "kafka_native" is removed - user just needs to...