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

From: Rahul Soshte <rahulsoshte360(at)gmail(dot)com>
To: Dave Page <dpage(at)pgadmin(dot)org>, 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:41:38
Message-ID: CAKyzeV0EEZxT-hs=_CrXZANZG6NyOLR2c1k+KrQ4wkf9W-6Z0A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

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
>

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-03-22 13:47:31 pgAdmin 4 commit: Fix target dependency per Rahul Soshte.
Previous Message Dave Page 2018-03-22 13:13:25 Re: v3.0 release on hold