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

ckan-docker update 11/2024 #167

Open
wants to merge 34 commits into
base: ckan/update-112024
Choose a base branch
from
Open

Conversation

mjanez
Copy link
Owner

@mjanez mjanez commented Nov 22, 2024

This pull request includes various updates to the CKAN Docker setup, focusing on improving configuration, documentation, and Dockerfile settings. The most important changes include updates to environment variables, enhancements to the README documentation, and modifications to Dockerfiles for better user and permissions management.

ckan/ckan-docker repo updates

Environment Variable Updates:

  • Added CKAN_MAX_UPLOAD_SIZE_MB=100 to .env.example to set a maximum upload size.
  • Removed unnecessary DATAPUSHER_REWRITE_RESOURCES and DATAPUSHER_REWRITE_URL variables from .env.example.

Documentation Enhancements:

  • Updated DataPusher image link in README.md to point to the correct repository.
  • Replaced a warning note with a more prominent format in README.md about changing default sysadmin credentials.
  • Added detailed instructions and examples for running Docker containers and creating CKAN extensions in README.md.

Dockerfile Modifications:

  • Changed base images in ckan/Dockerfile and ckan/Dockerfile.dev to use version 2.11 without the patch number and added user ownership for copied files.

Docker Compose Improvements:

  • Added local_bin volume and updated health check configurations in docker-compose.dev.yml and docker-compose.yml.

Nginx Configuration:

  • Added custom error page error.html and increased client_max_body_size to 140M in nginx setup.

kowh-ai and others added 24 commits August 1, 2024 13:14
I think this is the correct link target...seems the ckan-base repo may have been renamed to ckan-docker-base?
add warning, remove image of text
default config for 100MB uploads
fix example extension ownership command
Update compose files to check the health of the CKAN and DataPusher container processes
Minimise root ownership of application files
@mjanez mjanez added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request ckanext-schemingdcat labels Nov 22, 2024
@mjanez mjanez self-assigned this Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ckanext-schemingdcat documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants