Skip to content

MINOR: Upgrade ducktape to version 0.8.1#9933

Merged
kkonstantine merged 1 commit intoapache:trunkfrom
stan-is-hate:bump_ducktape_to_081
Jan 23, 2021
Merged

MINOR: Upgrade ducktape to version 0.8.1#9933
kkonstantine merged 1 commit intoapache:trunkfrom
stan-is-hate:bump_ducktape_to_081

Conversation

@stan-is-hate
Copy link
Copy Markdown
Contributor

Sister PR to #9932
Ducktape 0.8.1 was updated to include the following changes/fixes from 0.7.x branch:

  • Junit reporting support
  • fix for an issue where unicode characters in exception message would cause test runner to hang on py27.

Please merge this to 2.7 also. 2.6 and below are on ducktape 0.7.x branch and #9932 is about those branches.

@stan-is-hate
Copy link
Copy Markdown
Contributor Author

@andrewegel @omkreddy

Copy link
Copy Markdown
Contributor

@kkonstantine kkonstantine left a comment

Choose a reason for hiding this comment

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

The update LGTM

Again here I assume you've run tests both locally and on a jenkins builder?

@kkonstantine kkonstantine changed the title bump ducktape to version 0.8.1 MINOR: Upgrade ducktape to version 0.8.1 Jan 23, 2021
@kkonstantine
Copy link
Copy Markdown
Contributor

A few flaky test failures are not relevant to the changes.
Merging to trunk and 2.7

@kkonstantine kkonstantine merged commit 91d6c55 into apache:trunk Jan 23, 2021
@kkonstantine kkonstantine added the tests Test fixes (including flaky tests) label Jan 23, 2021
kkonstantine pushed a commit that referenced this pull request Jan 23, 2021
ducktape 0.8.1 was updated to include the following changes/fixes from 0.7.x branch:
* Junit reporting support
* fix for an issue where unicode characters in exception message would cause test runner to hang on py27.

Reviewers: Konstantine Karantasis <k.karantasis@gmail.com>
@stan-is-hate
Copy link
Copy Markdown
Contributor Author

Thanks for review and backport! Changes to ducktape were simply a bugfix to move a bunch of code into the finally block to make sure it executes, so they shouldn't affect test results.

ijuma added a commit to ijuma/kafka that referenced this pull request Jan 26, 2021
…e-allocations-lz4

* apache-github/trunk: (562 commits)
  MINOR: remove unused code from MessageTest (apache#9961)
  MINOR: Fix visibility of Log.{unflushedMessages, addSegment} methods (apache#9966)
  KAFKA-12229: Restore original class loader in integration tests using EmbeddedConnectCluster during shutdown  (apache#9942)
  KAFKA-12190: Fix setting of file permissions on non-POSIX filesystems (apache#9947)
  MINOR: Remove `toStruct` and `fromStruct` methods from generated protocol classes (apache#9960)
  MINOR: Fix typo in Utils#toPositive (apache#9943)
  MINOR: MessageUtil: remove some deadcode (apache#9931)
  MINOR: Update zstd-jni to 1.4.8-2 (apache#9957)
  MINOR: Revert assertion in MockProducerTest (apache#9956)
  MINOR: Optimize assertions in unit tests (apache#9955)
  MINOR: Tag `RaftEventSimulationTest` as `integration` and tweak it (apache#9925)
  MINOR: Update to Gradle 6.8.1 (apache#9953)
  MINOR: A few small group coordinator cleanups (apache#9952)
  MINOR: Upgrade ducktape to version 0.8.1  (apache#9933)
  MINOR: fix record time in test shouldWipeOutStandbyStateDirectoryIfCheckpointIsMissing (apache#9948)
  MINOR: Restore interrupt status when closing (apache#9863)
  KAFKA-10357: Extract setup of repartition topics from Streams partition assignor (apache#9848)
  KAFKA-12212; Bump Metadata API version to remove `ClusterAuthorizedOperations` fields (KIP-700) (apache#9945)
  MINOR: log 2min processing summary of StreamThread loop (apache#9941)
  MINOR: Drop enable.metadata.quorum config (apache#9934)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Test fixes (including flaky tests)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants