Re: Build failed in Jenkins: pgadmin4-master-python33 #507

public inbox for [email protected]  
help / color / mirror / Atom feed

From: Dave Page <[email protected]>
To: Khushboo Vashi <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: Build failed in Jenkins: pgadmin4-master-python33 #507
Date: Thu, 8 Mar 2018 10:05:42 +0000
Message-ID: <CA+OCxozhh9caZ-Y1BNsqsSRgY45Wa6ChiPiWajQDZjGZE_ykTg@mail.gmail.com> (raw)
In-Reply-To: <CAFOhELcpCQb775=M5PZL9uY+2kpBba6Z-8tpV5TAucQzU8NSCg@mail.gmail.com>
References: <[email protected]>
	<CA+OCxoxrPgB_mv_-50z9d2hj7iHQyeMJGhXtEkGfFMYf4_Ep+w@mail.gmail.com>
	<CAFOhELfx_KfUmFc+McR+maHJgLfsi+z+CMfnn3VPgdv539XtuQ@mail.gmail.com>
	<CA+OCxoziegBKj165ysM86fWL3m1CFN5+hnVBDjx3D9eeFHH6=Q@mail.gmail.com>
	<CAFOhELc+=eod5gtxQvRw=G2t=BQWb=2SgHOxbLrA2wxEROutzQ@mail.gmail.com>
	<CA+OCxox9OOr-UNS3WvMLB4ayZF+xvVjBbZH7JEi-C=fWQ2tZuw@mail.gmail.com>
	<CAFOhELe+5kX67EWH7nZc8965doOLanvqORKiqGX0BUyh42Hb-A@mail.gmail.com>
	<CA+OCxoy3EQogDYScsTSKas6SX0x=L8SW2RZ1Cd5hA=J7Y7itOA@mail.gmail.com>
	<CA+OCxoyMfKvF6jxFtR7w4Zi5=aUVMtcsXtJw_0ouDRgQRQtr8Q@mail.gmail.com>
	<CAFOhELddhg2i+_Hc152ojpejFp97B=bMMLqfNMoL99AbL1tAXA@mail.gmail.com>
	<CAFOhELed+dfW1r3_hfOc1LgxHEkr_8T9423aTr=RhUuNm6FL-w@mail.gmail.com>
	<CAFOhELc577sDgVxCNYuvC9S=5+qer2So2JW9FXEtb3FLpFPcGA@mail.gmail.com>
	<CA+OCxozG5+wHq1LQao8Tj6qQ9c+TD85HG1=+pQynDkFWy86jRA@mail.gmail.com>
	<CA+OCxoz0R+UoyOEh_4__TyyZ9G_nkOi3uHofMs=HjGc3mzavtw@mail.gmail.com>
	<CAFOhELcpCQb775=M5PZL9uY+2kpBba6Z-8tpV5TAucQzU8NSCg@mail.gmail.com>

On Thu, Mar 8, 2018 at 9:47 AM, Khushboo Vashi <
[email protected]> wrote:

>
>
> On Thu, Mar 8, 2018 at 3:08 PM, Dave Page <[email protected]> wrote:
>
>> Well, it looks like Python 3 might be happy now. 2.6 and 2.7 aren't
>> though (which is odd, as it passed on my local 2.7).
>>
>> I am looking into this. Btw, which command do you use to run tests?
>

make check-python



> On Thu, Mar 8, 2018 at 9:13 AM, Dave Page <[email protected]> wrote:
>>
>>> Thanks, applied.
>>>
>>> On Thu, Mar 8, 2018 at 4:26 AM, Khushboo Vashi <
>>> [email protected]> wrote:
>>>
>>>>
>>>>
>>>> On Wed, Mar 7, 2018 at 11:13 PM, Khushboo Vashi <
>>>> [email protected]> wrote:
>>>>
>>>>> Hi Dave,
>>>>>
>>>>> Please find the patch to fix this.
>>>>> I have fixed this issue in the StartRunningQueryTest class, this test
>>>>> class changes the behaviour of the methods through MagicMock and due to
>>>>> this this test case is failing.
>>>>>
>>>>> Revised patch with some modification.
>>>>
>>>>> Thanks,
>>>>> Khushboo
>>>>>
>>>>>
>>>>> On Wed, Mar 7, 2018 at 10:34 PM, Khushboo Vashi <
>>>>> [email protected]> wrote:
>>>>>
>>>>>>
>>>>>>
>>>>>> On Wed, Mar 7, 2018 at 10:33 PM, Dave Page <[email protected]> wrote:
>>>>>>
>>>>>>> It still seems to be failing :-(
>>>>>>>
>>>>>>> :(
>>>>>>
>>>>>>> https://jenkins.pgadmin.org/job/pgadmin4-master-python26/las
>>>>>>> tFailedBuild/consoleFull
>>>>>>>
>>>>>>> On Wed, Mar 7, 2018 at 4:50 PM, Dave Page <[email protected]> wrote:
>>>>>>>
>>>>>>>> Thanks, applied. Lets see what happens.
>>>>>>>>
>>>>>>>> On Wed, Mar 7, 2018 at 4:37 PM, Khushboo Vashi <
>>>>>>>> [email protected]> wrote:
>>>>>>>>
>>>>>>>>> Please find the attached updated patch.
>>>>>>>>> This should work.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> On Wed, Mar 7, 2018 at 9:46 PM, Dave Page <[email protected]>
>>>>>>>>> wrote:
>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On Wed, Mar 7, 2018 at 4:13 PM, Khushboo Vashi <
>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> On Wed, Mar 7, 2018 at 9:23 PM, Dave Page <[email protected]>
>>>>>>>>>>> wrote:
>>>>>>>>>>>
>>>>>>>>>>>> Hi
>>>>>>>>>>>>
>>>>>>>>>>>> On Wed, Mar 7, 2018 at 3:10 PM, Khushboo Vashi <
>>>>>>>>>>>> [email protected]> wrote:
>>>>>>>>>>>>
>>>>>>>>>>>>> Hi Dave,
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Wed, Mar 7, 2018 at 7:34 PM, Dave Page <[email protected]>
>>>>>>>>>>>>> wrote:
>>>>>>>>>>>>>
>>>>>>>>>>>>>> Khushboo, can you look at this ASAP please? It passed tests
>>>>>>>>>>>>>> on my machine, but Jenkins isn't happy.
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> I have tried to reproduce the issue with many combination but
>>>>>>>>>>>>> couldn't reproduce it.
>>>>>>>>>>>>> With the help of logs, I have tried to fix it. Can you please
>>>>>>>>>>>>> check this fix is helpful or not?
>>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> On my laptop I now get:
>>>>>>>>>>>>
>>>>>>>>>>>> ============================================================
>>>>>>>>>>>> ==========
>>>>>>>>>>>> ERROR: runTest (pgadmin.tools.sqleditor.tests
>>>>>>>>>>>> .test_poll_query_tool.TestPollQueryTool)
>>>>>>>>>>>> When query tool polling returns messages with result data-set
>>>>>>>>>>>> ------------------------------------------------------------
>>>>>>>>>>>> ----------
>>>>>>>>>>>> Traceback (most recent call last):
>>>>>>>>>>>>   File "/Users/dpage/git/pgadmin4/web
>>>>>>>>>>>> /pgadmin/tools/sqleditor/tests/test_poll_query_tool.py", line
>>>>>>>>>>>> 104, in runTest
>>>>>>>>>>>>     response_data['data']['additional_messages'])
>>>>>>>>>>>>   File "/System/Library/Frameworks/Py
>>>>>>>>>>>> thon.framework/Versions/2.7/lib/python2.7/unittest/case.py",
>>>>>>>>>>>> line 802, in assertIn
>>>>>>>>>>>>     if member not in container:
>>>>>>>>>>>> TypeError: argument of type 'NoneType' is not iterable
>>>>>>>>>>>>
>>>>>>>>>>>> ------------------------------------------------------------
>>>>>>>>>>>> ----------
>>>>>>>>>>>>
>>>>>>>>>>>> Please provide some more details. Is this failing for
>>>>>>>>>>> particular server version?
>>>>>>>>>>> I am not getting any error with this patch and without this
>>>>>>>>>>> patch on PY2 and PY3 with a single/multiple servers.
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Python 2.7.10 on macOS Sierra, testing against PostgreSQL 9.4.
>>>>>>>>>> Here are the package versions in the venv:
>>>>>>>>>>
>>>>>>>>>> alabaster==0.7.10
>>>>>>>>>> alembic==0.9.6
>>>>>>>>>> args==0.1.0
>>>>>>>>>> Babel==2.3.4
>>>>>>>>>> backports.csv==1.0.4
>>>>>>>>>> beautifulsoup4==4.4.1
>>>>>>>>>> blinker==1.3
>>>>>>>>>> chromedriver-installer==0.0.6
>>>>>>>>>> click==6.6
>>>>>>>>>> clint==0.5.1
>>>>>>>>>> dateutils==0.6.6
>>>>>>>>>> docutils==0.13.1
>>>>>>>>>> extras==1.0.0
>>>>>>>>>> fixtures==3.0.0
>>>>>>>>>> Flask==0.11.1
>>>>>>>>>> Flask-Babel==0.11.1
>>>>>>>>>> Flask-Gravatar==0.4.2
>>>>>>>>>> Flask-HTMLmin==1.2
>>>>>>>>>> Flask-Login==0.3.2
>>>>>>>>>> Flask-Mail==0.9.1
>>>>>>>>>> Flask-Migrate==2.0.3
>>>>>>>>>> Flask-Paranoid==0.1.0
>>>>>>>>>> Flask-Principal==0.4.0
>>>>>>>>>> Flask-Script==2.0.6
>>>>>>>>>> Flask-Security==1.7.5
>>>>>>>>>> Flask-SQLAlchemy==2.1
>>>>>>>>>> Flask-WTF==0.12
>>>>>>>>>> funcsigs==1.0.2
>>>>>>>>>> html5lib==1.0b3
>>>>>>>>>> htmlmin==0.1.11
>>>>>>>>>> imagesize==0.7.1
>>>>>>>>>> importlib==1.0.3
>>>>>>>>>> itsdangerous==0.24
>>>>>>>>>> Jinja2==2.7.3
>>>>>>>>>> linecache2==1.0.0
>>>>>>>>>> Mako==1.0.7
>>>>>>>>>> MarkupSafe==0.23
>>>>>>>>>> minify==0.1.4
>>>>>>>>>> mock==2.0.0
>>>>>>>>>> passlib==1.6.2
>>>>>>>>>> pbr==3.1.1
>>>>>>>>>> pkginfo==1.4.1
>>>>>>>>>> psycopg2==2.7.3.2
>>>>>>>>>> pycodestyle==2.3.1
>>>>>>>>>> pycrypto==2.6.1
>>>>>>>>>> Pygments==2.2.0
>>>>>>>>>> pyperclip==1.5.32
>>>>>>>>>> pyrsistent==0.14.0
>>>>>>>>>> python-dateutil==2.5.0
>>>>>>>>>> python-editor==1.0.3
>>>>>>>>>> python-mimeparse==1.6.0
>>>>>>>>>> pytz==2014.10
>>>>>>>>>> requests==2.13.0
>>>>>>>>>> requests-toolbelt==0.7.1
>>>>>>>>>> selenium==3.3.3
>>>>>>>>>> simplejson==3.6.5
>>>>>>>>>> six==1.11.0
>>>>>>>>>> snowballstemmer==1.2.1
>>>>>>>>>> speaklater==1.3
>>>>>>>>>> Sphinx==1.5.3
>>>>>>>>>> SQLAlchemy==1.0.14
>>>>>>>>>> sqlparse==0.1.19
>>>>>>>>>> testscenarios==0.5.0
>>>>>>>>>> testtools==2.0.0
>>>>>>>>>> traceback2==1.4.0
>>>>>>>>>> unittest2==1.1.0
>>>>>>>>>> Werkzeug==0.9.6
>>>>>>>>>> WTForms==2.0.2
>>>>>>>>>> yuicompressor==2.4.8
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> Dave Page
>>>>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>>>>> Twitter: @pgsnake
>>>>>>>>>>
>>>>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>> --
>>>>>>>> Dave Page
>>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>>> Twitter: @pgsnake
>>>>>>>>
>>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>> --
>>>>>>> Dave Page
>>>>>>> Blog: http://pgsnake.blogspot.com
>>>>>>> Twitter: @pgsnake
>>>>>>>
>>>>>>> EnterpriseDB UK: http://www.enterprisedb.com
>>>>>>> The Enterprise PostgreSQL Company
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>
>>>
>>> --
>>> Dave Page
>>> Blog: http://pgsnake.blogspot.com
>>> Twitter: @pgsnake
>>>
>>> EnterpriseDB UK: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>>
>>
>>
>>
>> --
>> Dave Page
>> Blog: http://pgsnake.blogspot.com
>> Twitter: @pgsnake
>>
>> EnterpriseDB UK: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>
>


-- 
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

view thread (185+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: Build failed in Jenkins: pgadmin4-master-python33 #507
  In-Reply-To: <CA+OCxozhh9caZ-Y1BNsqsSRgY45Wa6ChiPiWajQDZjGZE_ykTg@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox