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

Datapusher URL stays within docker-compose #63

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

riordan
Copy link

@riordan riordan commented May 18, 2017

Summary

This pull request changes the datapusher URL to use docker-compose style networking, so that requests to datapusher are independent of the site url.

Background

First of all - thank you for creating this incredibly well-documented way to set up CKAN. I'd been looking for a way to build a durable CKAN that was easy to reproduce, stored files on S3, and could be easily ported to a new machine without impact.

After I deployed the system following your instructions, CKAN wasn't connecting to datapusher at all. This took a long time to troubleshoot and was manifesting in 504 timeout errors every time we tried to add a new resource to a dataset and nothing entering the datastore.

Then, any resource we added that was hosted on another site was being served through the OKF Dataproxy service on Google, which frequently runs out of free bandwidth, so our data previews failed.

Anyhow, this was when it became clear that something was wrong with the datastore and the datapusher.

It appears this was also the case with prior issue #47 (datapusher isn't talking to other services)

Changes

  • rather than using the siteurl environment variable to connect to the datastore (which would have required the AWS security settings to open up the datapusher port), this uses docker-compose style networking to set up the datapusher connection. [e.g. CKAN_DATAPUSHER_URL: "${CKAN_SITE_URL}:8000" -> CKAN_DATAPUSHER_URL: "http://datapusher:8800"
  • This also changes the port that datapusher runs on to 8800, to be closer to a stock CKAN configuration.

@riordan
Copy link
Author

riordan commented May 18, 2017

(original comment updated to reflect an over-eager submission)

@Kyle-Falconer
Copy link
Contributor

Wow thanks for looking into this. I know I spent quite a while on this problem as well and never really figures it out. We will test this and let you know.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants