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

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

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



##########
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:
       I second the concern around moving this note into the Azure section; that would further increase chances of overlooking it, and potentially confuse Azure users. Perhaps, just add emphasis for this step of using the correct AMI, using Markdown syntax, within the EC2 section itself?




----------------------------------------------------------------
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