aasaru commented on a change in pull request #6: FINCN-165: Implement bash 
scripts to provisioner fineract cn services
URL: 
https://github.com/apache/fineract-cn-docker-compose/pull/6#discussion_r326286036
 
 

 ##########
 File path: bash_scripts/provision.sh
 ##########
 @@ -0,0 +1,318 @@
+#!/bin/bash
+set -e
+
+function init-variables {
+    local file="../env_variables"
+
+    CASSANDRA_REPLICATION_TYPE="Simple"
+    CASSANDRA_CONTACT_POINTS='172.16.238.68:9042' #initialized by the start-up 
script
 
 Review comment:
   I wonder why you don't use DNS names instead of IP addresses.
   You could use "cassandra-ms" instead of Cassandra IP. And similarly you 
could use "postgres-ms" instead of Postgres IP. The same goes with all other 
IP-s. If you could use DNS names instead of IP addresses then it would be more 
compatible with Mac as the Docker IP addresses don't work in Mac. Docker DNS 
addresses don't work either in Mac but a workaround for them is to add them to 
/etc/hosts and make them all point at 127.0.0.1.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to