[GitHub] [fluo-muchos] Manno15 commented on a change in pull request #361: Added changes to readme for user friendliness

[GitBox <gi...@apache.org>]

Manno15 commented on a change in pull request #361:
URL: https://github.com/apache/fluo-muchos/pull/361#discussion_r459351842



##########
File path: README.md
##########
@@ -74,6 +75,14 @@ sudo yum install azure-cli-2.0.69-1.el7.x86_64.rpm
 
 When running Muchos under Ubuntu 18.04, checkout these [tips](docs/azure-ubuntu-1804.md).
 
+You might also need to configure the `aws_ami` property in [muchos.props]. Muchos by default uses a free
+CentOS 7 image that is hosted in the AWS marketplace but managed by the
+CentOS organization. If you have never used this image in EC2 before, you will need to go to the
+[CentOS 7 product page][centos7] to accept the software terms. If this is not done, you will get an
+error when you try to launch your cluster. By default, the `aws_ami` property is set to an AMI in `us-east-1`.
+You will need to changes this value if a newer image has been released or if you are running in different region
+than `us-east-1`.
+

Review comment:
       To add to this, the default ami in mucho.props.example should probably be updated to the most recent ami. It has been at its current version for a while but it is still different than the default one given in our props file. 




----------------------------------------------------------------
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:
users@infra.apache.org