search

pelias / docker

Run the Pelias geocoder in docker containers, including example projects.
MIT License
315 stars 218 forks source link

Added folder for Italy #258

Open massimogentilini opened 3 years ago

massimogentilini commented 3 years ago

Cloned another folder and set it up for Italy

:wave: I did some awesome work for the Pelias project and would love for everyone to have a look at it and provide feedback.

Here's the reason for this change :rocket:

Here's what actually got changed :clap:

Here's how others can test the changes :eyes:

orangejulius commented 3 years ago

Ciao @massimogentilini,

Thanks for this!

Looks good overall. I don't think exposing port 5601 on the Elasticsearch container does anything though, right? That's usually the port for Kibana but our Elasticsearch docker image does not include Kibana.

massimogentilini commented 3 years ago

Correct, I’ll remove it.

Il giorno 11 mar 2021, alle ore 20:04, Julian Simioni @.***> ha scritto:

 Ciao @massimogentilini,

Thanks for this!

Looks good overall. I don't think exposing port 5601 on the Elasticsearch container does anything though, right? That's usually the port for Kibana but our Elasticsearch docker image does not include Kibana.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

missinglink commented 12 months ago

Hi, sorry I merged https://github.com/pelias/docker/pull/287 first, are you happy with that or can we take some of the config in this PR and merge them to create something better?