Re: [pgAdmin4][Patch] RM #3355 User can not perform operation of backup,Backup all, Backup global and Maintenance DB with ssh tunneling

From: Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com>
To: Joao De Almeida Pereira <jdealmeidapereira(at)pivotal(dot)io>
Cc: pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: [pgAdmin4][Patch] RM #3355 User can not perform operation of backup,Backup all, Backup global and Maintenance DB with ssh tunneling
Date: 2018-05-29 17:19:28
Message-ID: CANxoLDepfVCj+Bc6wbhU9Sy6BxNpt60A0=yddgpK0q_KbzdmNQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi Joao

On Tue, May 29, 2018 at 9:24 PM, Joao De Almeida Pereira <
jdealmeidapereira(at)pivotal(dot)io> wrote:

> Hello Akshay,
>
> Looks like the tests failed, but it is just flakes. We assume no
> regression was made with this code, despite the fact that no tests were
> added to ensure the new code is working.
>

Thanks for reviewing it. Yes tests were failed, but it is nothing to do
with this patch. I haven't change any logic in feature test. This patch
contains the changes where instead of using host and port of the database
server it uses 'localhost' and the local bind port returned by SSH Tunnel
module.

Khushboo has send the patch where she wrote test cases for Utility
module (not committed yet) and anyways we can't replicate the SSH
Tunnelling scenario using test cases.

>
> Thanks
> Anthony && Joao
>
> On Tue, May 29, 2018 at 6:41 AM Akshay Joshi <
> akshay(dot)joshi(at)enterprisedb(dot)com> wrote:
>
>> Hi Hackers,
>>
>> Attached is the patch to fix RM #3355 User can not perform operation of
>> backup, Backup all, Backup global and Maintenance DB with ssh tunneling.
>>
>> Please review it.
>>
>> --
>> *Akshay Joshi*
>>
>> *Sr. Software Architect *
>>
>>
>>
>> *Phone: +91 20-3058-9517Mobile: +91 976-788-8246*
>>
>

--
*Akshay Joshi*

*Sr. Software Architect *

*Phone: +91 20-3058-9517Mobile: +91 976-788-8246*

In response to

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2018-05-29 19:35:39 Re: [pgadmin4][Patch]: Test cases for the backup module
Previous Message Joao De Almeida Pereira 2018-05-29 15:54:28 Re: [pgAdmin4][Patch] RM #3355 User can not perform operation of backup,Backup all, Backup global and Maintenance DB with ssh tunneling