afalko commented on a change in pull request #920: Provide an Ansible playbook 
for AWS with documentation (WIP)
URL: https://github.com/apache/incubator-pulsar/pull/920#discussion_r153579421
 
 

 ##########
 File path: ansible/provision-pulsar-aws.tf
 ##########
 @@ -0,0 +1,117 @@
+variable "public_key_path" {
+  description = <<DESCRIPTION
+Path to the SSH public key to be used for authentication.
+Ensure this keypair is added to your local SSH agent so provisioners can
+connect.
+
+Example: ~/.ssh/terraform.pub
+DESCRIPTION
+}
+
+variable "key_name" {
+  description = "Desired name of AWS key pair"
+}
+
+variable "region" {
+    default = "us-west-2"
+}
+
+variable "ami" {
+    default = "ami-9fa343e7" // RHEL-7.4
 
 Review comment:
   Why RHEL-7.4 and not Amazon Linux? Since a pulsar deployment likely needs 
fast nvram disks, driver support for them comes faster on Amazon Linux. I've 
had issues before where RHEL doesn't recognize all disks. 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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