Skip to content

MINOR: Add 2.1 version metadata upgrade#6111

Merged
guozhangwang merged 2 commits intoapache:trunkfrom
bbejeck:MINOR_add_2_1_version_metadata_upgrade
Jan 9, 2019
Merged

MINOR: Add 2.1 version metadata upgrade#6111
guozhangwang merged 2 commits intoapache:trunkfrom
bbejeck:MINOR_add_2_1_version_metadata_upgrade

Conversation

@bbejeck
Copy link
Copy Markdown
Member

@bbejeck bbejeck commented Jan 9, 2019

Updated the test_metadata_upgrade test. To enable using the 2.1 version I needed to add config change to the StreamsUpgradeTestJobRunnerService to ensure the ductape passes proper args when starting the StreamsUpgradeTest

For testing, I ran the test_metadata_upgrade test and all versions now pass http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2019-01-09--001.1547049873--bbejeck--MINOR_add_2_1_version_metadata_upgrade--a450c68/report.html

Committer Checklist (excluded from commit message)

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

@bbejeck
Copy link
Copy Markdown
Member Author

bbejeck commented Jan 9, 2019

ping @guozhangwang, @mjsax, and @vvcephei for reviews

Copy link
Copy Markdown
Member

@mjsax mjsax 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
Copy link
Copy Markdown
Contributor

LGTM!

@guozhangwang guozhangwang merged commit b1b792d into apache:trunk Jan 9, 2019
pengxiaolong pushed a commit to pengxiaolong/kafka that referenced this pull request Jun 14, 2019
Updated the test_metadata_upgrade test. To enable using the 2.1 version I needed to add config change to the StreamsUpgradeTestJobRunnerService to ensure the ductape passes proper args when starting the StreamsUpgradeTest

For testing, I ran the test_metadata_upgrade test and all versions now pass http://confluent-kafka-branch-builder-system-test-results.s3-us-west-2.amazonaws.com/2019-01-09--001.1547049873--bbejeck--MINOR_add_2_1_version_metadata_upgrade--a450c68/report.html

Reviewers: Matthias J. Sax <matthias@confluent.io>, Guozhang Wang <wangguoz@gmail.com>
@bbejeck bbejeck deleted the MINOR_add_2_1_version_metadata_upgrade branch July 10, 2024 12:57
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.

3 participants