Re: Server specific advanced config file for testsuite

From: Dave Page <dave(dot)page(at)enterprisedb(dot)com>
To: Navnath Gadakh <navnath(dot)gadakh(at)enterprisedb(dot)com>
Cc: Kanchan Mohitey <kanchan(dot)mohitey(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Server specific advanced config file for testsuite
Date: 2016-09-13 08:48:08
Message-ID: CA+OCxox=16k4tAhdZLDk_D0bqSG34=rd4QfXgcqGx=61GjL-uA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

Hi

On Mon, Sep 12, 2016 at 6:26 PM, Navnath Gadakh
<navnath(dot)gadakh(at)enterprisedb(dot)com> wrote:
> Hi Dave,
> As we know that, we are using two JSON files one is for basic
> configuration of server(test_config.json) and second for detailed
> configuration of server(test_advanced_config.json).
> In test_advanced_config.json, we give detail for database, schema,
> extension, collation etc. These details are generic I mean these details are
> not server specific.
> - Do we need to allow user to give server specific details?

I don't see any reason to do so. Honestly I don't see much need for
test_advanced_config.json at all - I can't see anyone ever customising
the tests. I certainly doubt I will.

> If yes, there should be some identification(like ‘server_type’, ‘version’)
> for each detail so that we can read server appropriate details.
> We have one approach in mind, to attach 'server_type' & 'version' to each
> detail
> So, modified test_advanced_config.json can be like this,
> {
> "add_database_data”:[
> {
> "conn_limit": -1,
> “owner”: “postres”,
> .
> .
> .
> .
> “server_type”: “pg”,
> “version”: “pg9.4”
> },
> {
> {
> "conn_limit": -1,
> “owner”: “enterprisedb”,
> .
> .
> .
> .
> “server_type”: “ppas”,
> “version”: “ppas9.6”
> }
> }],
> “schema_credentials”:
> [{"func_acl": [],
> "seq_acl": [],
> .
> .
> .
> .
> “server_type”: “pg”,
> “version”: “pg9.4”
> }]
> .
> .
> .
> .
> .
> }
>
> Any suggestions?
>
> For multiple server currently we used 'owner' to identify the server, but
> that fails if two servers have the same owner.
>
>
> --
> Thanks,
> Navnath Gadakh
> Software Engineer
> EnterpriseDB Corporation
> Mobile: +91 9975389878
>
>

--
Dave Page
VP, Chief Architect, Tools & Installers
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Dave Page 2016-09-13 08:48:44 Re: Server specific advanced config file for testsuite
Previous Message Ashesh Vashi 2016-09-13 08:37:35 Re: PGAdmin 4 1.0-rc1 bugs