medic / cht-sync

Data synchronization between CouchDB and PostgreSQL for the purpose of analytics.
GNU General Public License v3.0
2 stars 3 forks source link

Remove host dependency on node #72

Closed witash closed 3 months ago

witash commented 4 months ago

The docs currently refer to running the project with node on the host. The node script is mostly issuing docker compose commands with different arguments based on environment and some other things which are either not relevant anymore or only for dev (creating logstash file which is now in source control, zipping/unzip data directory for local couchdb)

This node script could be replaced with a bash script which sets the arguments for docker-compose. Similar to https://github.com/medic/cht-interoperability/blob/main/startup.sh. And the bash script is only for convenience; if the dev and other stuff is removed, its possible to run by using the docker compose commands directly

this would remove the requirement that node be installed on the host. maybe not a big requirement but node is not very lightweight and fewer dependencies is always better. It undermines the portability benefits of docker-compose a little bit whenever new requirements for the host are added; although unlikely that this script would run differently on different version of node/npm, it is still possible.