Skip to content

MINOR: Port streams broker compatibility fix#8203

Merged
guozhangwang merged 1 commit intoapache:2.5from
vvcephei:john-2.5-system-test-fix
Mar 2, 2020
Merged

MINOR: Port streams broker compatibility fix#8203
guozhangwang merged 1 commit intoapache:2.5from
vvcephei:john-2.5-system-test-fix

Conversation

@vvcephei
Copy link
Copy Markdown
Contributor

@vvcephei vvcephei commented Mar 2, 2020

Ports system test fix from #7997

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@vvcephei vvcephei requested a review from guozhangwang March 2, 2020 17:18
@vvcephei
Copy link
Copy Markdown
Contributor Author

vvcephei commented Mar 2, 2020

@guozhangwang , can you take a look at this port?

cc @mumrah

@vvcephei
Copy link
Copy Markdown
Contributor Author

vvcephei commented Mar 2, 2020

The relevant test now passes:

================================================================================
SESSION REPORT (ALL TESTS)
ducktape version: 0.7.6
session_id:       2020-03-02--004
run time:         5 minutes 57.602 seconds
tests run:        13
passed:           13
failed:           0
ignored:          0
================================================================================
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=0.11.0.3
status:     PASS
run time:   30.172 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=1.0.2
status:     PASS
run time:   29.243 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=1.1.1
status:     PASS
run time:   29.772 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=2.0.1
status:     PASS
run time:   30.618 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=2.1.1
status:     PASS
run time:   29.691 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=2.2.2
status:     PASS
run time:   29.307 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=2.3.1
status:     PASS
run time:   29.905 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_compatible_brokers_eos_disabled.broker_version=2.4.0
status:     PASS
run time:   32.071 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_fail_fast_on_incompatible_brokers.broker_version=0.10.0.1
status:     PASS
run time:   24.712 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_fail_fast_on_incompatible_brokers.broker_version=0.10.1.1
status:     PASS
run time:   24.120 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_fail_fast_on_incompatible_brokers.broker_version=0.10.2.2
status:     PASS
run time:   22.070 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_fail_fast_on_incompatible_brokers_if_eos_enabled.broker_version=0.10.1.1
status:     PASS
run time:   22.442 seconds
--------------------------------------------------------------------------------
test_id:    kafkatest.tests.streams.streams_broker_compatibility_test.StreamsBrokerCompatibility.test_fail_fast_on_incompatible_brokers_if_eos_enabled.broker_version=0.10.2.2
status:     PASS
run time:   22.548 seconds
--------------------------------------------------------------------------------

Copy link
Copy Markdown
Contributor

@guozhangwang guozhangwang left a comment

Choose a reason for hiding this comment

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

LGTM!

@guozhangwang guozhangwang merged commit 3f99f9d into apache:2.5 Mar 2, 2020
@vvcephei vvcephei deleted the john-2.5-system-test-fix branch March 2, 2020 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants