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