Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace Alpine with a Python (3.10) base image for CKAN 2.10 and master branches #61

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

kowh-ai
Copy link
Contributor

@kowh-ai kowh-ai commented May 10, 2024

Fixes: #54

The objective is to upgrade from the alpine 2.17 base image to an official DockerHub Python 3.10 base image. We opted for a Debian 12 (bookworm slim) base image.

No alterations have been made to the current CKAN 2.9 builds.

Efforts were made to streamline the system libraries to only include essential components.

Upon successful testing and integration of this PR, several improvements will be realized:

  • Implementation of Multi-Stage Builds, assuming it reduces image size and increases image security
  • Optimization of Docker Layers and subsequent caching mechanisms
  • Enhancements to render the base image more suitable for production environments
  • Assess the feasibility of substituting Supervisor with a native Docker Compose (or Docker) approach for managing CKAN process restarts.
  • Fix: Running "ckan -c ckan.ini db upgrade" on master fails #60

I'm happy to take suggestions for better naming of files and images

@kowh-ai
Copy link
Contributor Author

kowh-ai commented May 10, 2024

@EricSoroos - if you get a chance would you please be able to take a look at this?

@wardi wardi self-assigned this May 21, 2024
@amercader amercader self-assigned this May 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants