-
Notifications
You must be signed in to change notification settings - Fork 27
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
Containerize tileserver #93
Conversation
Dockerfile
Outdated
&& apt-get -y install libgeos-dev \ | ||
libpq-dev \ | ||
python-pil \ | ||
libmapnik2.2 \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why are these Mapnik related installs necessary?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like:
Has reference to Mapnik in error, compared to requirement file:
So update the Wiki?
Looks great! I had to add this to the diff --git a/requirements.txt b/requirements.txt
index cf3e4e9..b104d93 100644
--- a/requirements.txt
+++ b/requirements.txt
@@ -25,3 +25,4 @@ zope.dottedname==4.1.0
edtf==0.9.3
mapbox-vector-tile==1.2.0
git+https://github.com/mapzen/tilequeue@master#egg=tilequeue
+git+https://github.com/tilezen/raw_tiles@master#egg=raw_tiles When I |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The preview is a great idea!
Thanks for the suggestion on the config.yaml. I made that change and it seems to work for me. I'm not sure why it worked for me without the raw tiles requirement. |
Excited about this! I just tried pulling the latest and running
|
Oh duh I do have port 8000 already bound... continuing... |
Follow up... postgis seems to have died on initial startup attempt:
I simply tried re-running
But, any requests to preview fail with a missing hstore error:
|
Yep, this is expected for now. There's a modification to the For now, you can try my workaround:
|
I have tiles! 🎊 Thanks! |
I'm going to merge this in and start seeking feedback on the Docker info. |
This is a work in progress as I experiment with containerizing tileserver for easier development and more consistent production deploys.