From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Andrew Coleman <penguincoder(at)gmail(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: change to Docker entrypoint.sh |
Date: | 2019-06-27 14:57:23 |
Message-ID: | CA+OCxoyRVG=-67ih7FYU8AazmJLZCTQzewhAqjUUoexv5Q-Hrw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Thu, Jun 27, 2019 at 8:12 AM Andrew Coleman <penguincoder(at)gmail(dot)com>
wrote:
>
>
> On Fri, Jun 21, 2019 at 4:24 AM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>> Hi
>>
>> On Thu, Jun 20, 2019 at 5:31 PM Andrew Coleman <penguincoder(at)gmail(dot)com>
>> wrote:
>>
>>> Hi All!
>>>
>>> I am new to this, so I apologize beforehand if it's not entirely kosher,
>>> but I have a proposed change to the Dockerfile entrypoint.sh to allow for
>>> configurable servers.json filename path.
>>>
>>> The reasoning behind this is that I have a Kubernetes deployment of
>>> pgadmin where I build a ConfigMap sort of dynamically. I can make the file
>>> appear in /pgadmin4, but it removes the contents of the directory for me
>>> (helpful!). Instead, I put it into a separate directory and set a new
>>> environment variable that defaults to the old pathname
>>> (backwards-compatible!) but allows me to set a custom environment variable
>>> in my custom Helm chart for my deployment.
>>>
>>
>> pgAdmin removes the content of the directory? I'm not sure how it would
>> do that - it doesn't delete anything like that.
>>
>
> pgAdmin does not do that. It behaves as you would expect. Kubernetes
> removes the contents for me. It basically allows one of two options:
>
> file available at /pgadmin4/servers.json/servers.json and pgadmin works,
> but never loads the file
> file available at /pgadmin4/servers.json and nothing works because the
> only file in /pgadmin4 is servers.json
>
OK, thanks. Patch applied!
>
>
>>
>>>
>>> The code is here:
>>>
>>> https://github.com/penguincoder/pgadmin4/commit/0865400385ac0d03bf69d183e8cd9aaec6dcf318
>>>
>>> Is this something that anyone is interested in?
>>>
>>
>> Sure.
>>
>>
>>> Does it meet your coding standards?
>>>
>>
>> Looks fine to me.
>>
>>
>>> What do I do next?
>>>
>>> I would love to move back to the "official" pgadmin container.
>>>
>>
>> Please raise a feature request at
>> https://redmine.postgresql.org/projects/pgadmin4/issues/new and attach
>> the patch (just use git diff against a checkout of the master branch). You
>> can send the patch here, but we need the feature request in the tracker
>> anyway, so it's probably easier to attach it there.
>>
>> Thanks!
>>
>>
>> --
>> 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
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2019-06-27 15:15:52 | Re: passwords enabled by default |
Previous Message | Dave Page | 2019-06-27 14:56:40 | pgAdmin 4 commit: Allow the path to /pgadmin4/servers.json to be overri |