From: | Akshay Joshi <akshay(dot)joshi(at)enterprisedb(dot)com> |
---|---|
To: | Dave Page <dpage(at)pgadmin(dot)org> |
Cc: | Ashesh Vashi <ashesh(dot)vashi(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Docker build simplification |
Date: | 2019-09-13 06:59:01 |
Message-ID: | CANxoLDf3R2EGS5m-1Zk=KHWWKWiH1RFH2nsCam5sXRWte_NOJw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
Hi Dave
I have installed docker, applied your patch and run "*docker build .*".
Image has been created without REPOSITORY and TAG is this OK.
[image: Screenshot 2019-09-13 at 12.22.57 PM.png]
Rest I have tested it works fine.
On Wed, Sep 11, 2019 at 4:44 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>
> On Wed, Sep 11, 2019 at 6:59 AM Ashesh Vashi <
> ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
>
>> On Tue, Sep 10, 2019 at 11:21 PM Dave Page <dpage(at)pgadmin(dot)org> wrote:
>>
>>> The attached patch simplifies the Docker container build so that it can
>>> be done entirely using "docker build" in the root of a source tree. It also
>>> enables building the container from a tarball, not just a git checkout.
>>> This does mean that container builds may now also include any garbage in
>>> the source tree from which they're built, but conversely offers the benefit
>>> of making it easy to build a container with code changes in it without
>>> having to commit first.
>>>
>>> Akshay, can you look it over to make sure I haven't done anything
>>> obviously stupid? Feedback from others is also welcome of course.
>>>
>> Dave,
>>
>> Is there any reason - we're not using the requirements.txt from the
>> docker file, and installing the packages separately.
>>
>
> I'm not sure what you mean. It installs Python dependencies in two places;
>
> - In the documentation builder intermediate container. This doesn't use
> requirements.txt, and installs just the bare minimum required to install
> the docs.
>
> - In the final container. This does use requirements.txt.
>
> I suppose there's a minor simplification that can be made to the way I'm
> installing the requirements.txt file itself. At the moment it gets copied
> to the app builder, and then from there to the final container. I don't
> think it's actually needed in the app builder so I could just copy it from
> the host straight into the final container.
>
> Or did you mean something else?
>
> --
> Dave Page
> Blog: http://pgsnake.blogspot.com
> Twitter: @pgsnake
>
> EnterpriseDB UK: http://www.enterprisedb.com
> The Enterprise PostgreSQL Company
>
--
*Thanks & Regards*
*Akshay Joshi*
*Sr. Software Architect*
*EnterpriseDB Software India Private Limited*
*Mobile: +91 976-788-8246*
From | Date | Subject | |
---|---|---|---|
Next Message | Murtuza Zabuawala | 2019-09-13 08:10:41 | Re: [pgAdmin][RM4642] port should not be mandatory when a service is provided |
Previous Message | Akshay Joshi | 2019-09-13 06:24:24 | pgAdmin 4 commit: Ensure port and username should not be mandatory when |