Skip to content

Commit

Permalink
fixing tf missing var
Browse files Browse the repository at this point in the history
  • Loading branch information
ryandeivert committed Sep 18, 2019
1 parent b9aa118 commit 188874d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
2 changes: 2 additions & 0 deletions streamalert_cli/terraform/cloudwatch.py
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,7 @@ def generate_cloudwatch(cluster_name, cluster_dict, config):
return False

account_id = config['global']['account']['aws_account_id']
prefix = config['global']['account']['prefix']
cross_account_ids = cloudwatch_module.get('cross_account_ids', []) + [account_id]
excluded_regions = set(cloudwatch_module.get('excluded_regions', set()))

Expand Down Expand Up @@ -71,6 +72,7 @@ def generate_cloudwatch(cluster_name, cluster_dict, config):
'source': 'modules/tf_cloudwatch',
'region': region,
'cross_account_ids': cross_account_ids,
'prefix': prefix,
'cluster': cluster_name,
'kinesis_stream_arn': '${{module.kinesis_{}.arn}}'.format(cluster_name)
}
Expand Down
1 change: 1 addition & 0 deletions tests/unit/streamalert_cli/terraform/test_generate.py
Original file line number Diff line number Diff line change
Expand Up @@ -402,6 +402,7 @@ def test_generate_cloudwatch(self):
assert_equal(count, 14)

expected_config = {
'prefix': 'unit-testing',
'cluster': 'advanced',
'source': 'modules/tf_cloudwatch',
'region': 'eu-west-1',
Expand Down

0 comments on commit 188874d

Please sign in to comment.