Re: Docker build simplification

On Wed, Sep 11, 2019 at 6:59 AM Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:

On Tue, Sep 10, 2019 at 11:21 PM Dave Page <dpage@pgadmin.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