Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Bug]: TopicControllerIT.shouldUpdateTopicInKafkaWhen* tests are flaky #10926

Open
fvaleri opened this issue Dec 9, 2024 · 2 comments
Open
Labels

Comments

@fvaleri
Copy link
Contributor

fvaleri commented Dec 9, 2024

Bug Description

Topic Operator integration tests that starts with shouldUpdateTopicInKafkaWhen are flaky, but they only seems to be failing on Azure with similar logs:

2024-12-07T14:01:08.5931055Z [INFO] Results:
2024-12-07T14:01:08.5931548Z [INFO] 
2024-12-07T14:01:08.5931910Z [ERROR] Failures: 
2024-12-07T14:01:08.5932358Z [ERROR] io.strimzi.operator.topic.TopicControllerIT.shouldUpdateTopicInKafkaWhenConfigRemovedInKube
2024-12-07T14:01:08.5933312Z [ERROR]   Run 1: TopicControllerIT.shouldUpdateTopicInKafkaWhenConfigRemovedInKube:992->shouldUpdateTopicInKafkaWhenConfigChangedInKube:872 expected: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.6, index.interval.bytes=1234}> but was: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.6, index.interval.bytes=1234, unclean.leader.election.enable=true}>
2024-12-07T14:01:08.5934673Z [ERROR]   Run 2: TopicControllerIT.shouldUpdateTopicInKafkaWhenConfigRemovedInKube:992->shouldUpdateTopicInKafkaWhenConfigChangedInKube:872 expected: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.6, index.interval.bytes=1234}> but was: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.6, index.interval.bytes=1234, unclean.leader.election.enable=true}>
2024-12-07T14:01:08.5936219Z [ERROR]   Run 3: TopicControllerIT.shouldUpdateTopicInKafkaWhenConfigRemovedInKube:992->shouldUpdateTopicInKafkaWhenConfigChangedInKube:872 expected: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.6, index.interval.bytes=1234}> but was: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.6, index.interval.bytes=1234, unclean.leader.election.enable=true}>
2024-12-07T14:01:08.5937016Z [INFO] 
2024-12-07T14:01:08.5942614Z [WARNING] Flakes: 
2024-12-07T14:01:08.5943537Z [WARNING] io.strimzi.operator.topic.TopicControllerIT.shouldUpdateTopicInKafkaWhenDoubleConfigChangedInKube
2024-12-07T14:01:08.5944569Z [ERROR]   Run 1: TopicControllerIT.shouldUpdateTopicInKafkaWhenDoubleConfigChangedInKube:935->shouldUpdateTopicInKafkaWhenConfigChangedInKube:872 expected: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.1, index.interval.bytes=1234, unclean.leader.election.enable=true}> but was: <{compression.type=producer, cleanup.policy=compact, flush.ms=1234, min.cleanable.dirty.ratio=0.6, index.interval.bytes=1234, unclean.leader.election.enable=true}>
2024-12-07T14:01:08.5945802Z [INFO]   Run 2: PASS

Steps to reproduce

No response

Expected behavior

No response

Strimzi version

main

Kubernetes version

1.31

Installation method

No response

Infrastructure

No response

Configuration files and logs

No response

Additional context

No response

@fvaleri fvaleri changed the title [Bug]: Test cases TopicControllerIT.shouldUpdateTopicInKafkaWhen* are flaky [Bug]: TopicControllerIT.shouldUpdateTopicInKafkaWhen* tests are flaky Dec 9, 2024
@fvaleri
Copy link
Contributor Author

fvaleri commented Dec 9, 2024

Trying to address in #10921.

@ppatierno
Copy link
Member

Triaged on 12/12/2024: Fede is trying to address the flakyness with a PR and we should fix it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants