[pr] addressing feedback from @austinbyers

manage.py

@@ -404,7 +404,7 @@ def _add_metric_alarm_subparser(subparsers):
         help=ARGPARSE_SUPPRESS
     )
 
-    # Set the name of this parser to 'validate-schemas'
+    # Set the name of this parser to 'create-alarm'
     metric_alarm_parser.set_defaults(command='create-alarm')
 
     # add all the required parameters

stream_alert_cli/terraform_generate.py

@@ -432,6 +432,8 @@ def generate_cloudwatch_metric_alarms(cluster_name, cluster_dict, config):
         if 'metric_alarms' not in func_config:
             continue
 
+        # TODO: update this logic to simply use a list of maps once Terraform fixes
+        # their support for this, instead of the comma-separated string this creates
         metric_alarms = func_config['metric_alarms']
         for name, alarm_info in metric_alarms.iteritems():
             formatted_alarms.append(

terraform/modules/tf_stream_alert/main.tf

@@ -185,6 +185,7 @@ resource "aws_cloudwatch_log_metric_filter" "alert_processor_cw_metric_filters"
 // The split list is our way around poor tf support for lists of maps and is made up of:
 // <alarm_name>, <alarm_description>, <comparison_operator>, <evaluation_periods>,
 // <metric>, <period>, <statistic>, <threshold>
+// TODO: update this logic to simply use a variable that is a list of maps once Terraform fixes this
 resource "aws_cloudwatch_metric_alarm" "cw_metric_alarms" {
   count               = "${length(var.metric_alarms)}"
   alarm_name          = "${element(split(",", var.metric_alarms[count.index]), 0)}"