From: | Khushboo Vashi <khushboo(dot)vashi(at)enterprisedb(dot)com> |
---|---|
To: | Yogesh Mahajan <yogesh(dot)mahajan(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Feature #7178 - PostgreSQL deployment on Microsoft Azure |
Date: | 2022-06-14 06:05:09 |
Message-ID: | CAFOhELexmUp_L94VuLUbuWaeq=wRUZPhz=CMFM2OJUp1wSN1sA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Yogesh,
- Spelling mistake in below error message
- Name must be more than 3 characters or more & *Shoudld* not have
capital letter
- Cluster name validation comes after all the fields are validated. It
should be done while giving input for the cluster name
- Even If the credentials are incorrect, the authenticate button gets
disabled. So, I can't update and re-authenticate
Apart from this, it looks good to me.
Thanks,
Khushboo
On Fri, Jun 10, 2022 at 8:14 PM Yogesh Mahajan <
yogesh(dot)mahajan(at)enterprisedb(dot)com> wrote:
>
> Hi,
>
> Please find the updated patch with documentation.
>
>
> On Mon, Jun 6, 2022 at 1:10 PM Khushboo Vashi <
> khushboo(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> Hi Yogesh,
>>
>> Review comments:
>>
>> - Step 2: The below statements should be in a different line, like
>> -
>> - "Azure CLI" will use the currently logged in identity through
>> Azure CLI on the local machine.
>> - "Interactive Browser" opens a browser to authenticate a user
>> interactively.
>>
>> Done.
>
>>
>> - Disable the next button once authentication is complete.
>>
>> As discussed disabled Authentication button once authentication is
> completed.
>
>> - Cluster name availability call calls the server on every field
>> change
>>
>> Done.
>
>>
>> - Availability zone needs description
>>
>> This is a generic term with cloud hence not added.
>
>>
>> - Add High availability option
>>
>> Done
>
>>
>>
>> Code:
>>
>> - check_cluster_name_availability should be using the GET method
>> instead of Post
>>
>> Done.
>
>>
>> - Fix SonarLint issues
>>
>> Done.
>
>>
>> - Do we need the cache_persistence_options as it will create the
>> persistent storage which we do not require I guess?
>>
>> Yes, it is required.
>
>>
>> - Why do we need to call _get_azure_credentials on every request?
>> Can't we store it in the session object?
>>
>> Function returns without calling again credentials if an existing client
> is present.
>
>>
>> - Use gettext wherever required in the js file
>>
>> Done.
>
>>
>> Thanks,
>> Khushboo
>>
>
> Thanks,
> Yogesh Mahajan
> EnterpriseDB
>
>
>>
>> On Wed, 1 Jun 2022, 10:11 Yogesh Mahajan, <
>> yogesh(dot)mahajan(at)enterprisedb(dot)com> wrote:
>>
>>> Hi,
>>>
>>> Please find the attached patch which provides functionality to deploy a
>>> Postgres cloud instance on Azure Postgresql.
>>>
>>> Thanks,
>>> Yogesh Mahajan
>>> EnterpriseDB
>>>
>>
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2022-06-14 07:39:22 | Re: [Patch] - Feature #7332 - pgadmin docker container doesn't work properly with secrets |
Previous Message | Yogesh Mahajan | 2022-06-14 06:05:03 | [Patch] - Feature #7332 - pgadmin docker container doesn't work properly with secrets |