pgrange / risno

real-estate search engine
GNU General Public License v3.0
2 stars 1 forks source link

Docker #18

Closed nlamirault closed 9 years ago

nlamirault commented 9 years ago

This request initialize Docker support for the web application and the Elasticsearch image. Little refactoring for project organization. We must check whether the unit tests pass correctly before accepting the merge request.

pgrange commented 9 years ago

Next time, try to avoid mixing real changes with indent or other space issues.

pgrange commented 9 years ago

Sorry, didn't want to close this :(

pgrange commented 9 years ago

Followed doc: $ make init $ ./machine create -d virtualbox risno $ eval "$(./machine env risno)" $ ./compose up

Then when connectiong to web application, we get errors since the elasticsearch indexes do not exist. So nodejs web application succeeds in connectings to elasticsearch but it can not find the indexes it needs to work.

pgrange commented 9 years ago

making a docker diff, I see that elasticsearch container write data in /src/elasticsearch/data/elasticsearch instead of /var/lib/elasticsearch

docker ps

CONTAINER ID IMAGE COMMAND CREATED STATUS PORTS NAMES 383935651aee gite_elasticsearch:latest "/src/elasticsearch/ 4 seconds ago Up 3 seconds 0.0.0.0:9200->9200/tcp, 0.0.0.0:9300->9300/tcp gite_elasticsearch_1

docker diff 383935651aee

C /src C /src/elasticsearch A /src/elasticsearch/data A /src/elasticsearch/data/elasticsearch A /src/elasticsearch/data/elasticsearch/nodes A /src/elasticsearch/data/elasticsearch/nodes/0 A /src/elasticsearch/data/elasticsearch/nodes/0/node.lock A /src/elasticsearch/logs A /src/elasticsearch/logs/elasticsearch.log A /src/elasticsearch/logs/elasticsearch_index_indexing_slowlog.log A /src/elasticsearch/logs/elasticsearch_index_search_slowlog.log C /tmp C /tmp/hsperfdata_root A /tmp/hsperfdata_root/1 C /var C /var/lib A /var/lib/elasticsearch C /var/log A /var/log/elasticsearch

pgrange commented 9 years ago

Pushed some fixes in develop branch

pgrange commented 9 years ago

Now both containers see each other but we may have an issue with elasticsearch version.

pgrange commented 9 years ago

Last issue came from elasticsearch data on my terminal. So it is OK.

Last pushed version works: I can launch both elasticsearch and webapp with compose up and it just works.

pgrange commented 9 years ago

So know the question is : what do we do with this stuff ?

pgrange commented 9 years ago

This pull request is related to issue #9

pgrange commented 9 years ago

Switching to pull request #19