Re: Little patch for Runtime build from Top Level Directory($PGADMIN_SRC) MakeFile

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Rahul Soshte <rahulsoshte360(at)gmail(dot)com>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Little patch for Runtime build from Top Level Directory($PGADMIN_SRC) MakeFile
Date: 2018-03-22 13:47:44
Message-ID: CA+OCxoxVinUUeFds7f76-Fujizq889JH+3k==a__Q_xZ_RsSqA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Rats, my bad. Fixed now - thanks.

On Thu, Mar 22, 2018 at 1:41 PM, Rahul Soshte <rahulsoshte360(at)gmail(dot)com>
wrote:

> there is a small typo in the target name "all" runtime-release needs to be
> changed runtime too.
>
> On Thu, Mar 22, 2018 at 4:50 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Thanks - patch applied with a minor change of the target name
>> "runtime-release" to "runtime" (runtime-debug remains the same).
>>
>> On Thu, Mar 22, 2018 at 9:28 AM, Rahul Soshte <rahulsoshte360(at)gmail(dot)com>
>> wrote:
>>
>>> This is my first time trying to make a Patch.
>>>
>>> I am using Ubuntu 17.10.
>>>
>>> This patch is in reference to the Feature Added by Dave Page
>>> https://redmine.postgresql.org/issues/1305
>>>
>>> I just added code to call makefile in runtime directory(
>>> $PGADMIN_SRC/runtime/Makefile ) from the top level directory's MakeFile (
>>> $PGADMIN_SRC/Makefile )
>>>
>>>
>>> I followed the checklist for submitting a patch sent by Murutuza
>>> Zabuawala
>>>
>>> <================================>
>>> Hello,
>>>
>>> Here is the common checklist to follow before sending patch to
>>> pgAdmin4-hackers group,
>>>
>>> 1) Get the latest pull from master branch.
>>>
>>> 2) Apply your patch and check if applies successfully on the latest code.
>>>
>>> 3) Check for PEP8 issues
>>> - activate virtual env
>>> - cd ../web
>>> - yarn run pep8
>>>
>>> 4) Run regression test
>>> python regression/runtests.py
>>>
>>> - To run only UI/feature tests
>>> python regression/runtests.py --pkg feature_tests
>>>
>>> - To run regression test (without UI/Feature tests)
>>> python regression/runtests.py --exclude feature_tests
>>>
>>> 5) Run Jasmine tests
>>> - activate virtual env
>>> - cd ../web
>>> yarn run test:karma-once
>>>
>>> 6) Make sure to add or update help docs and screenshot(s) if you have
>>> added any new feature or changed any existing one.
>>>
>>> --
>>> Regards,
>>> Murtuza Zabuawala
>>> EnterpriseDB: http://www.enterprisedb.com
>>> The Enterprise PostgreSQL Company
>>> <===================================================>
>>>
>>> But I am facing some issues,
>>> Step 1,2,3 and 5 had no issues
>>> But when I tried step 4 it gave some errors
>>> I executed the following command
>>>
>>> *python regression/runtests.py *
>>>
>>> I got the following errors:
>>> 1)The screenshot attached show I wasnt able to connect to the server
>>> 2) the error ouput on the command line:
>>> =============Running the test cases for 'PostgreSQL 9.4'=============
>>> Traceback (most recent call last):
>>> File "/var/www/flask/pgadmin4/pgadmin4/web/regression/python_test_utils/test_utils.py",
>>> line 128, in create_database
>>> server['sslmode']
>>> File "/var/www/flask/pgadmin4/pgadmin4/web/regression/python_test_utils/test_utils.py",
>>> line 36, in get_db_connection
>>> sslmode=sslmode
>>> File "/var/www/flask/pgadmin4/local/lib/python2.7/site-packages/psycopg2/__init__.py",
>>> line 130, in connect
>>> conn = _connect(dsn, connection_factory=connection_factory,
>>> **kwasync)
>>> OperationalError: FATAL: password authentication failed for user
>>> "postgres"
>>> FATAL: password authentication failed for user "postgres"
>>>
>>> Traceback (most recent call last):
>>> File "regression/runtests.py", line 350, in <module>
>>> server_information = test_utils.create_parent_server_node(server)
>>> File "/var/www/flask/pgadmin4/pgadmin4/web/regression/python_test_utils/test_utils.py",
>>> line 491, in create_parent_server_node
>>> server_info['sslmode']
>>> File "/var/www/flask/pgadmin4/pgadmin4/web/regression/python_test_utils/test_utils.py",
>>> line 36, in get_db_connection
>>> sslmode=sslmode
>>> File "/var/www/flask/pgadmin4/local/lib/python2.7/site-packages/psycopg2/__init__.py",
>>> line 130, in connect
>>> conn = _connect(dsn, connection_factory=connection_factory,
>>> **kwasync)
>>> psycopg2.OperationalError: FATAL: password authentication failed for
>>> user "postgres"
>>> FATAL: password authentication failed for user "postgres"
>>>
>>> Traceback (most recent call last):
>>> File "/var/www/flask/pgadmin4/pgadmin4/web/regression/python_test_utils/test_utils.py",
>>> line 619, in _cleanup
>>> connection = get_db_server(database["server_id"])
>>> File "/var/www/flask/pgadmin4/pgadmin4/web/regression/python_test_utils/test_utils.py",
>>> line 593, in get_db_server
>>> db_name, username, db_password, host, db_port, ssl_mode
>>> File "/var/www/flask/pgadmin4/pgadmin4/web/regression/python_test_utils/test_utils.py",
>>> line 36, in get_db_connection
>>> sslmode=sslmode
>>> File "/var/www/flask/pgadmin4/local/lib/python2.7/site-packages/psycopg2/__init__.py",
>>> line 130, in connect
>>> conn = _connect(dsn, connection_factory=connection_factory,
>>> **kwasync)
>>> OperationalError: FATAL: password authentication failed for user
>>> "postgres"
>>> FATAL: password authentication failed for user "postgres"
>>>
>>> I have no password authentication errors when I am connecting from the
>>> application.I had also changed password for postgres manually through the
>>> command line.
>>>
>>
>>
>>
>> --
>> 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

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Joao De Almeida Pereira 2018-03-22 14:57:09 Re: Showstopper desktop runtime issue
Previous Message Dave Page 2018-03-22 13:47:31 pgAdmin 4 commit: Fix target dependency per Rahul Soshte.