Hello. I fixed two system tests that fails in trunk, also.
streams_upgrade_test.py::StreamsUpgradeTest.test_version_probing_upgrade streams_static_membership_test.py Please, take a look at my PR [1] [1] https://github.com/apache/kafka/pull/9312 > 20 сент. 2020 г., в 06:11, Guozhang Wang <wangg...@gmail.com> написал(а): > > I've triggered a system test on top of your branch. > > Maybe you could also re-run the jenkins unit tests since currently all of > them fails but you've only touched on system tests, so I'd like to confirm > at least one successful run. > > On Wed, Sep 16, 2020 at 3:37 AM Nikolay Izhikov <nizhi...@apache.org> wrote: > >> Hello, Guozhang. >> >>> I can help run the test suite once your PR is cleanly rebased to verify >> the whole suite works >> >> Thank you for joining to the review. >> >> 1. PR rebased on the current trunk. >> >> 2. I triggered all tests in my private environment to verify them after >> rebase. >> Will inform you once tests passed on my environment. >> >> 3. We need a new ducktape release [1] to be able to merge PR [2]. >> For now, PR based on the ducktape trunk branch [3], not some >> specific release. >> If ducktape team need any help with the release, please, let me >> know. >> >> [1] https://github.com/confluentinc/ducktape/issues/245 >> [2] https://github.com/apache/kafka/pull/9196 >> [3] >> https://github.com/apache/kafka/pull/9196/files#diff-9235a7bdb1ca9268681c0e56f3f3609bR39 >> >>> 16 сент. 2020 г., в 07:32, Guozhang Wang <wangg...@gmail.com> >> написал(а): >>> >>> Hello Nikolay, >>> >>> I can help run the test suite once your PR is cleanly rebased to verify >> the >>> whole suite works and then I can merge (I'm trusting Ivan and Magnus here >>> for their reviews :) >>> >>> Guozhang >>> >>> On Mon, Sep 14, 2020 at 3:56 AM Nikolay Izhikov <nizhi...@apache.org> >> wrote: >>> >>>> Hello! >>>> >>>> I got 2 approvals from Ivan Daschinskiy and Magnus Edenhill. >>>> Committers, please, join the review. >>>> >>>>> 3 сент. 2020 г., в 11:06, Nikolay Izhikov <nizhikov....@gmail.com> >>>> написал(а): >>>>> >>>>> Hello! >>>>> >>>>> Just a friendly reminder. >>>>> >>>>> Patch to resolve some kind of technical debt - python2 in system tests >>>> is ready! >>>>> Can someone, please, take a look? >>>>> >>>>> https://github.com/apache/kafka/pull/9196 >>>>> >>>>>> 28 авг. 2020 г., в 11:19, Nikolay Izhikov <nizhikov....@gmail.com> >>>> написал(а): >>>>>> >>>>>> Hello! >>>>>> >>>>>> Any feedback on this? >>>>>> What I should additionally do to prepare system tests migration? >>>>>> >>>>>>> 24 авг. 2020 г., в 11:17, Nikolay Izhikov <nizhikov....@gmail.com> >>>> написал(а): >>>>>>> >>>>>>> Hello. >>>>>>> >>>>>>> PR [1] is ready. >>>>>>> Please, review. >>>>>>> >>>>>>> But, I need help with the two following questions: >>>>>>> >>>>>>> 1. We need a new release of ducktape which includes fixes [2], [3] >> for >>>> python3. >>>>>>> I created the issue in ducktape repo [4]. >>>>>>> Can someone help me with the release? >>>>>>> >>>>>>> 2. I know that some companies run system tests for the trunk on a >>>> regular bases. >>>>>>> Can someone show me some results of these runs? >>>>>>> So, I can compare failures in my PR and in the trunk. >>>>>>> >>>>>>> Results [5] of run all for my PR available in the ticket [6] >>>>>>> >>>>>>> ``` >>>>>>> SESSION REPORT (ALL TESTS) >>>>>>> ducktape version: 0.8.0 >>>>>>> session_id: 2020-08-23--002 >>>>>>> run time: 1010 minutes 46.483 seconds >>>>>>> tests run: 684 >>>>>>> passed: 505 >>>>>>> failed: 9 >>>>>>> ignored: 170 >>>>>>> ``` >>>>>>> >>>>>>> [1] https://github.com/apache/kafka/pull/9196 >>>>>>> [2] >>>> >> https://github.com/confluentinc/ducktape/commit/23bd5ab53802e3a1e1da1ddf3630934f33b02305 >>>>>>> [3] >>>> >> https://github.com/confluentinc/ducktape/commit/bfe53712f83b025832d29a43cde3de3d7803106f >>>>>>> [4] https://github.com/confluentinc/ducktape/issues/245 >>>>>>> [5] >>>> https://issues.apache.org/jira/secure/attachment/13010366/report.txt >>>>>>> [6] https://issues.apache.org/jira/browse/KAFKA-10402 >>>>>>> >>>>>>>> 14 авг. 2020 г., в 21:26, Ismael Juma <ism...@juma.me.uk> >> написал(а): >>>>>>>> >>>>>>>> +1 >>>>>>>> >>>>>>>> On Fri, Aug 14, 2020 at 7:42 AM John Roesler <vvcep...@apache.org> >>>> wrote: >>>>>>>> >>>>>>>>> Thanks Nikolay, >>>>>>>>> >>>>>>>>> No objection. This would be very nice to have. >>>>>>>>> >>>>>>>>> Thanks, >>>>>>>>> John >>>>>>>>> >>>>>>>>> On Fri, Aug 14, 2020, at 09:18, Nikolay Izhikov wrote: >>>>>>>>>> Hello. >>>>>>>>>> >>>>>>>>>>> If anyone's interested in porting it to Python 3 it would be a >> good >>>>>>>>> change. >>>>>>>>>> >>>>>>>>>> I’ve created a ticket [1] to upgrade system tests to python3. >>>>>>>>>> Does someone have any additional inputs or objections for this >>>> change? >>>>>>>>>> >>>>>>>>>> [1] https://issues.apache.org/jira/browse/KAFKA-10402 >>>>>>>>>> >>>>>>>>>> >>>>>>>>>>> 1 июля 2020 г., в 00:26, Gokul Ramanan Subramanian < >>>>>>>>> gokul24...@gmail.com> написал(а): >>>>>>>>>>> >>>>>>>>>>> Thanks Colin. >>>>>>>>>>> >>>>>>>>>>> While at the subject of system tests, there are a few times I see >>>> tests >>>>>>>>>>> timed out (even on a large machine such as m5.4xlarge EC2 with >>>> Linux). >>>>>>>>> Are >>>>>>>>>>> there any knobs that system tests provide to control timeouts / >>>>>>>>> throughputs >>>>>>>>>>> across all tests? >>>>>>>>>>> Thanks. >>>>>>>>>>> >>>>>>>>>>> On Tue, Jun 30, 2020 at 6:32 PM Colin McCabe <cmcc...@apache.org >>> >>>>>>>>> wrote: >>>>>>>>>>> >>>>>>>>>>>> Ducktape runs on Python 2. You can't use it with Python 3, as >>>> you are >>>>>>>>>>>> trying to do here. >>>>>>>>>>>> >>>>>>>>>>>> If anyone's interested in porting it to Python 3 it would be a >>>> good >>>>>>>>> change. >>>>>>>>>>>> >>>>>>>>>>>> Otherwise, using docker as suggested here seems to be the best >>>> way to >>>>>>>>> go. >>>>>>>>>>>> >>>>>>>>>>>> best, >>>>>>>>>>>> Colin >>>>>>>>>>>> >>>>>>>>>>>> On Mon, Jun 29, 2020, at 02:14, Gokul Ramanan Subramanian wrote: >>>>>>>>>>>>> Hi. >>>>>>>>>>>>> >>>>>>>>>>>>> Has anyone had luck running Kafka system tests on a Mac. I >> have a >>>>>>>>> MacOS >>>>>>>>>>>>> Mojave 10.14.6. I got Python 3.6.9 using pyenv. However, the >>>> command >>>>>>>>>>>>> *ducktape tests/kafkatest/tests* yields the following error, >>>> making >>>>>>>>> it >>>>>>>>>>>> look >>>>>>>>>>>>> like some Python incompatibility issue. >>>>>>>>>>>>> >>>>>>>>>>>>> $ ducktape tests/kafkatest/tests >>>>>>>>>>>>> Traceback (most recent call last): >>>>>>>>>>>>> File "/Users/gokusubr/.pyenv/versions/3.6.9/bin/ducktape", line >>>> 11, >>>>>>>>> in >>>>>>>>>>>>> <module> >>>>>>>>>>>>> load_entry_point('ducktape', 'console_scripts', 'ducktape')() >>>>>>>>>>>>> File >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>> >>>> >> "/Users/gokusubr/.pyenv/versions/3.6.9/lib/python3.6/site-packages/pkg_resources/__init__.py", >>>>>>>>>>>>> line 487, in load_entry_point >>>>>>>>>>>>> return get_distribution(dist).load_entry_point(group, name) >>>>>>>>>>>>> File >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>> >>>> >> "/Users/gokusubr/.pyenv/versions/3.6.9/lib/python3.6/site-packages/pkg_resources/__init__.py", >>>>>>>>>>>>> line 2728, in load_entry_point >>>>>>>>>>>>> return ep.load() >>>>>>>>>>>>> File >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>> >>>> >> "/Users/gokusubr/.pyenv/versions/3.6.9/lib/python3.6/site-packages/pkg_resources/__init__.py", >>>>>>>>>>>>> line 2346, in load >>>>>>>>>>>>> return self.resolve() >>>>>>>>>>>>> File >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>> >>>> >> "/Users/gokusubr/.pyenv/versions/3.6.9/lib/python3.6/site-packages/pkg_resources/__init__.py", >>>>>>>>>>>>> line 2352, in resolve >>>>>>>>>>>>> module = __import__(self.module_name, fromlist=['__name__'], >>>>>>>>>>>>> level=0) >>>>>>>>>>>>> File >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>> >>>> >> "/Users/gokusubr/.pyenv/versions/3.6.9/lib/python3.6/site-packages/ducktape-0.7.6-py3.6.egg/ducktape/command_line/main.py", >>>>>>>>>>>>> line 127 >>>>>>>>>>>>> print "parameters are not valid json: " + str(e.message) >>>>>>>>>>>>> ^ >>>>>>>>>>>>> SyntaxError: invalid syntax >>>>>>>>>>>>> >>>>>>>>>>>>> I followed the instructions in tests/README.md to setup a >>>> cluster of >>>>>>>>> 9 >>>>>>>>>>>>> worker machines. That worked well. When I ran *python setup.py >>>>>>>>> develop* >>>>>>>>>>>> to >>>>>>>>>>>>> install the necessary dependencies (including ducktape), I got >>>>>>>>> similar >>>>>>>>>>>>> errors to above, but the overall command completed >> successfully. >>>>>>>>>>>>> >>>>>>>>>>>>> Any help appreciated. >>>>>>>>>>>>> >>>>>>>>>>>>> Thanks. >>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>> >>>>>>>>>> >>>>>>>>> >>>>>>> >>>>>> >>>>> >>>> >>>> >>> >>> -- >>> -- Guozhang >> >> > > -- > -- Guozhang