-
Notifications
You must be signed in to change notification settings - Fork 14k
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
flaky test report improvements #18212
base: trunk
Are you sure you want to change the base?
Conversation
@santhoshct some initial notes:
Thanks for continuing work on this! |
…y links to the test classes and test methods
All three successfully done. |
This looks excellent! The links will be super helpful. Looks like the "High-Priority Quarantined Tests" table is mostly redundant with the summary table at the top. Actually, with the collapsed details sections, I think we can restructure things a little bit. How about the following layout:
I think this will give the report a fairly uniform feel and make it a bit more intuitive to navigate. |
…arantine test report
I was also thinking about removing the high priority quarantine test report. the layout looks uniform. I have added it and updated the results in PR description. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@santhoshct I ran this locally and copied it here https://gist.github.com/mumrah/b476eb6abe07eb37dd297078e39ebbe0
I'd like to see each flaky method for a given test class (like how the PR description looks). I probably should have been more specific with my suggestion yesterday, but in the tables I think we have to have the test cases in there. The only thing I consider too detailed for the tables is the build scan data.
The new layout looks excellent. Just need to add the test cases back inline to the tables.
edit: btw I also notice since we've hyperlinked the whole class/test name, it makes selecting the text tricky. Let's create a column with a clickable link, e.g. "develocity
Testing
Flaky Test Report for 2024-12-17
This report was run on 2024-12-17 08:45:18 UTC
Most Problematic Tests
Found 8 tests that have been quarantined for 7 days and are still failing frequently.
Detailed Execution History
org.apache.kafka.tiered.storage.integration.OffloadAndTxnConsumeFromLeaderTest
executeTieredStorageTest(String, String)[1]
Recent Executions:
executeTieredStorageTest(String, String)[2]
Recent Executions:
kafka.api.CustomQuotaCallbackTest
testExecuteUpdateClusterMetadataMethodCallback(String, String)[1]
Recent Executions:
org.apache.kafka.tiered.storage.integration.TransactionsWithTieredStoreTest
testReadCommittedConsumerShouldNotSeeUndecidedData(String, String)[2]
Recent Executions:
testBumpTransactionalEpochWithTV2Enabled(String, String, boolean)[2]
Recent Executions:
testBumpTransactionalEpochWithTV2Enabled(String, String, boolean)[1]
Recent Executions:
testBumpTransactionalEpochWithTV2Disabled(String, String, boolean)[1]
Recent Executions:
testBumpTransactionalEpochWithTV2Disabled(String, String, boolean)[2]
Recent Executions:
org.apache.kafka.common.network.Tls13SelectorTest
testCloseOldestConnection()
Recent Executions:
org.apache.kafka.common.network.SslTransportLayerTest
testSelectorPollReadSize(Args)[1]
Recent Executions:
kafka.api.TransactionsTest
testBumpTransactionalEpochWithTV2Enabled(String, String, boolean)[1]
Recent Executions:
testBumpTransactionalEpochWithTV2Enabled(String, String, boolean)[2]
Recent Executions:
testBumpTransactionalEpochWithTV2Disabled(String, String, boolean)[1]
Recent Executions:
testBumpTransactionalEpochWithTV2Disabled(String, String, boolean)[2]
Recent Executions:
kafka.api.PlaintextConsumerTest
testCloseLeavesGroupOnInterrupt(String, String)[2]
Recent Executions:
testCoordinatorFailover(String, String)[2]
Recent Executions:
testCloseLeavesGroupOnInterrupt(String, String)[1]
Recent Executions:
testCoordinatorFailover(String, String)[1]
Recent Executions:
org.apache.kafka.clients.consumer.internals.AbstractCoordinatorTest
testWakeupAfterSyncGroupReceived()
Recent Executions:
Flaky Test Regressions
No flaky test regressions found.
Cleared Tests (Ready for Unquarantine)
Found 20 test classes with 42 test methods that have been consistently passing. These tests could be candidates for removing quarantine annotations at either class or method level.
Detailed Test Method History
kafka.integration.MetricsDuringTopicCreationDeletionTest
testMetricsDuringTopicCreateDelete(String)[1]
Recent Executions:
kafka.api.PlaintextConsumerSubscriptionTest
testRe2JPatternExpandSubscription(String, String)[1]
Recent Executions:
kafka.api.PlaintextAdminIntegrationTest
testConsumerGroupWithMemberMigration(String)[1]
Recent Executions:
kafka.api.SslAdminIntegrationTest
testListNodesFromControllersIncludingFencedBrokers(String)[1]
Recent Executions:
org.apache.kafka.streams.AutoOffsetResetTest
shouldThrowExceptionOnDurationForEarliestReset()
Recent Executions:
shouldThrowExceptionOnDurationForLastetReset()
Recent Executions:
shouldThrowExceptionOnDurationForNoneReset()
Recent Executions:
org.apache.kafka.streams.kstream.internals.KStreamImplTest
shouldProcessWithProcessorAndState()
Recent Executions:
org.apache.kafka.clients.producer.internals.TransactionManagerTest
testTransactionV2AddPartitionAndOffsets()
Recent Executions:
testMaybeAddPartitionToTransactionInTransactionV2()
Recent Executions:
testFatalErrorWhenProduceResponseWithInvalidPidMapping()
Recent Executions:
kafka.api.SslConsumerTest
testCoordinatorFailover(String, String)[1]
Recent Executions:
testCoordinatorFailover(String, String)[2]
Recent Executions:
org.apache.kafka.streams.StreamsBuilderTest
shouldWrapProcessorsForStreamStreamSelfJoinWithSharedStoreOptimization()
Recent Executions:
kafka.api.SaslSslConsumerTest
testCoordinatorFailover(String, String)[1]
Recent Executions:
testCoordinatorFailover(String, String)[2]
Recent Executions:
org.apache.kafka.clients.consumer.internals.ApplicationEventHandlerTest
testRecordApplicationEventQueueSize()
Recent Executions:
org.apache.kafka.streams.state.internals.metrics.RocksDBMetricsRecorderTest
shouldThrowIfMetricRecorderIsInitialisedWithNullTaskId()
Recent Executions:
shouldThrowIfMetricRecorderIsInitialisedWithNullMetrics()
Recent Executions:
org.apache.kafka.controller.QuorumControllerTest
testUncleanShutdownElrDisabled()
Recent Executions:
testUncleanShutdownBrokerElrEnabled()
Recent Executions:
kafka.test.api.ShareConsumerTest
testAcknowledgementCommitCallbackCallsShareConsumerWakeup(String)[1]
Recent Executions:
testAcknowledgementCommitCallbackCallsShareConsumerWakeup(String)[2]
Recent Executions:
testAcknowledgementCommitCallbackInvalidRecordStateException(String)[1]
Recent Executions:
testMultipleConsumersInGroupConcurrentConsumption(String)[1]
Recent Executions:
testExplicitAcknowledgementCommitAsync(String)[1]
Recent Executions:
org.apache.kafka.streams.integration.FineGrainedAutoResetIntegrationTest
shouldResetByDuration()
Recent Executions:
shouldFailForResetNone()
Recent Executions:
kafka.api.SaslMultiMechanismConsumerTest
testCoordinatorFailover(String, String)[1]
Recent Executions:
testCoordinatorFailover(String, String)[2]
Recent Executions:
org.apache.kafka.connect.integration.OffsetsApiIntegrationTest
testResetSinkConnectorOffsetsOverriddenConsumerGroupId()
Recent Executions:
testResetSinkConnectorOffsets()
Recent Executions:
testAlterSinkConnectorOffsetsOverriddenConsumerGroupId()
Recent Executions:
testGetSinkConnectorOffsetsDifferentKafkaClusterTargeted()
Recent Executions:
testAlterSinkConnectorOffsetsDifferentKafkaClusterTargeted()
Recent Executions:
kafka.network.DynamicConnectionQuotaTest
testDynamicConnectionQuota(String)[1]
Recent Executions:
kafka.api.SaslPlaintextConsumerTest
testCoordinatorFailover(String, String)[1]
Recent Executions:
testCoordinatorFailover(String, String)[2]
Recent Executions:
kafka.api.PlaintextConsumerTest
testCoordinatorFailover(String, String)[2]
Recent Executions:
testCoordinatorFailover(String, String)[1]
Recent Executions:
testCloseLeavesGroupOnInterrupt(String, String)[1]
Recent Executions:
testCloseLeavesGroupOnInterrupt(String, String)[2]
Recent Executions:
Committer Checklist (excluded from commit message)