Skip to content

MINOR: Bump system test version from 2.2.0 to 2.2.1#6873

Merged
mjsax merged 3 commits into
apache:trunkfrom
mjsax:minor-add-221-to-system-tests
Aug 9, 2019
Merged

MINOR: Bump system test version from 2.2.0 to 2.2.1#6873
mjsax merged 3 commits into
apache:trunkfrom
mjsax:minor-add-221-to-system-tests

Conversation

@mjsax

@mjsax mjsax commented Jun 3, 2019

Copy link
Copy Markdown
Member

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.

Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.

Committer Checklist (excluded from commit message)

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

@mjsax

mjsax commented Jun 3, 2019

Copy link
Copy Markdown
Member Author

This PR is blocked by #6597 -- will run system tests after merged and this PR is rebased (opened this PR early for tracking purpose).

\cc @hachikuji @rhauch Do we need to update anything for core or connect?

@mjsax
mjsax force-pushed the minor-add-221-to-system-tests branch from ce6ddc0 to 3f6cf73 Compare June 4, 2019 04:34

@abbccdda abbccdda left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@ijuma

ijuma commented Jun 9, 2019

Copy link
Copy Markdown
Member

@mjsax it looks like you didn't update the vagrant file.

@mjsax

mjsax commented Jun 24, 2019

Copy link
Copy Markdown
Member Author

Updated vagrant file.

@mjsax

mjsax commented Jun 24, 2019

Copy link
Copy Markdown
Member Author

@mjsax

mjsax commented Jun 26, 2019

Copy link
Copy Markdown
Member Author

Java11 passed. Java8 build not available any longer. System test passed.

Retest this please.

Call for review @guozhangwang @bbejeck @vvcephei @ableegoldman @cadonna

@mjsax
mjsax force-pushed the minor-add-221-to-system-tests branch from 5687709 to 7fef19e Compare July 16, 2019 23:48
@mjsax

mjsax commented Jul 16, 2019

Copy link
Copy Markdown
Member Author

Rebased this to resolve merge conflicts.

Retriggered system tests: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2795/

Call for review @guozhangwang @bbejeck

@bbejeck

bbejeck commented Jul 17, 2019

Copy link
Copy Markdown
Member

system test failed due to environmental issues, kicked off another one https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2796/

@bbejeck

bbejeck commented Jul 17, 2019

Copy link
Copy Markdown
Member

java 11 scala 2.13 failed with

org.apache.kafka.connect.integration.ConnectWorkerIntegrationTest.testAddAndRemoveWorker
org.apache.kafka.connect.integration.ExampleConnectIntegrationTest.testSourceConnector

java 8, java 11 scala 2.12 passed

retest this please

@mjsax

mjsax commented Jul 19, 2019

Copy link
Copy Markdown
Member Author

All 3 builds failed -- test result are not available any longer.

Retest this please.

@mjsax
mjsax force-pushed the minor-add-221-to-system-tests branch from 7fef19e to 11bbe93 Compare July 26, 2019 23:01
@mjsax

mjsax commented Jul 26, 2019

Copy link
Copy Markdown
Member Author

Rebased to latest trunk (hope this makes the test more stable). Also triggered system tests: minor-add-221-to-system-tests

@mjsax

mjsax commented Jul 31, 2019

Copy link
Copy Markdown
Member Author

All three runs failed -- results are not available any longer. Retest this please.

@bbejeck

bbejeck commented Aug 1, 2019

Copy link
Copy Markdown
Member

Java 11/2.12 time out Java 11 Scala/2.13 and Java 8 passed

retest this please

@mjsax

mjsax commented Aug 1, 2019

Copy link
Copy Markdown
Member Author

Previous system test results are not available any longer. Retriggered: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2844/

Java 8 passed. Retest this please.

@mjsax

mjsax commented Aug 2, 2019

Copy link
Copy Markdown
Member Author

Jenkins passed. System test failed (forgot to delete one line in a setup script for system test...)

@mjsax

mjsax commented Aug 2, 2019

Copy link
Copy Markdown
Member Author

@mjsax
mjsax force-pushed the minor-add-221-to-system-tests branch from bb9936a to 5a6cd02 Compare August 4, 2019 00:59
@mjsax

mjsax commented Aug 4, 2019

Copy link
Copy Markdown
Member Author

System test failed for test including 0.10.1 and 0.10.0 versions. I believe it because of the issue fixed via #7118

Rebased to get the fix into this PR. Retriggered system tests: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2846/

@mjsax

mjsax commented Aug 5, 2019

Copy link
Copy Markdown
Member Author

Unrelated system test failures for test kafkatest.tests.client.quota_test.QuotaTest.test_quota with error Jmx tool took too long to start.

Re-triggered: https://jenkins.confluent.io/job/system-test-kafka-branch-builder/2851/

@mjsax

mjsax commented Aug 6, 2019

Copy link
Copy Markdown
Member Author

@ijuma Seems like QuotaTest is broken atm. Can we merge this PR without getting a green system test build? Or do we need to wait for a fix for QuotaTest ?

@mjsax

mjsax commented Aug 6, 2019

Copy link
Copy Markdown
Member Author

Java 11 / 2.12 passed. Test results for others not available any longer. I would just merge as-is if there are no objections.

@mjsax
mjsax merged commit e9a35fe into apache:trunk Aug 9, 2019
@mjsax
mjsax deleted the minor-add-221-to-system-tests branch August 9, 2019 21:33
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.

4 participants