Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 3 commits into from
Aug 9, 2019

Conversation

mjsax
Copy link
Member

@mjsax mjsax commented Jun 3, 2019

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
Copy link
Member Author

mjsax commented Jun 3, 2019

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
Copy link
Contributor

@abbccdda abbccdda 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
Member

ijuma commented Jun 9, 2019

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

@mjsax
Copy link
Member Author

mjsax commented Jun 24, 2019

Updated vagrant file.

@mjsax
Copy link
Member Author

mjsax commented Jun 24, 2019

@mjsax
Copy link
Member Author

mjsax commented Jun 26, 2019

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
Copy link
Member Author

mjsax commented Jul 16, 2019

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
Copy link
Member

bbejeck commented Jul 17, 2019

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

@bbejeck
Copy link
Member

bbejeck commented Jul 17, 2019

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
Copy link
Member Author

mjsax commented Jul 19, 2019

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
Copy link
Member Author

mjsax commented Jul 26, 2019

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

@mjsax
Copy link
Member Author

mjsax commented Jul 31, 2019

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

@bbejeck
Copy link
Member

bbejeck commented Aug 1, 2019

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

retest this please

@mjsax
Copy link
Member Author

mjsax commented Aug 1, 2019

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
Copy link
Member Author

mjsax commented Aug 2, 2019

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

@mjsax
Copy link
Member Author

mjsax commented Aug 2, 2019

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

mjsax commented Aug 4, 2019

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
Copy link
Member Author

mjsax commented Aug 5, 2019

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
Copy link
Member Author

mjsax commented Aug 6, 2019

@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
Copy link
Member Author

mjsax commented Aug 6, 2019

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