Skip to content

MINOR: Ignore dynamic log4j log level tests#7183

Merged
ijuma merged 1 commit intoapache:trunkfrom
stanislavkozlovski:ignore-log4j-tests
Aug 9, 2019
Merged

MINOR: Ignore dynamic log4j log level tests#7183
ijuma merged 1 commit intoapache:trunkfrom
stanislavkozlovski:ignore-log4j-tests

Conversation

@stanislavkozlovski
Copy link
Copy Markdown
Contributor

@stanislavkozlovski stanislavkozlovski commented Aug 9, 2019

We recently introduced a bunch of flaky tests in the AdminClientIntegrationTest.
These tests are failing very frequently. We should ignore the tests in order to make
the build stable until we have a fix.

Copy link
Copy Markdown
Member

@ijuma ijuma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ijuma
Copy link
Copy Markdown
Member

ijuma commented Aug 9, 2019

One build passed, two failed with 2 unrelated failures each. A fix for testDescribeLogDirsRequest has just been merged and there is a PR that is meant to make RebalanceSourceConnectorsIntegrationTest more stable. Merging to trunk.

org.apache.kafka.connect.integration.RebalanceSourceConnectorsIntegrationTest.testStartTwoConnectors
kafka.server.DescribeLogDirsRequestTest.testDescribeLogDirsRequest

org.apache.kafka.connect.integration.RebalanceSourceConnectorsIntegrationTest.testReconfigConnector
kafka.server.DescribeLogDirsRequestTest.testDescribeLogDirsRequest

@ijuma ijuma merged commit 3f00649 into apache:trunk Aug 9, 2019
@stanislavkozlovski stanislavkozlovski deleted the ignore-log4j-tests branch August 9, 2019 15:11
ijuma added a commit to confluentinc/kafka that referenced this pull request Aug 9, 2019
* apache-github/trunk:
  MINOR: Ignore dynamic log4j log level tests (apache#7183)
  KAFKA-8748: Fix flaky testDescribeLogDirsRequest (apache#7182)
  KAFKA-8598: Use automatic RPC generation in RenewDelegationToken
  KAFKA-8179: Part 3, Add PartitionsLost API for resetGenerations and metadata/subscription change (apache#6884)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants