Skip to content

Commit

Permalink
better namings for roles and policy
Browse files Browse the repository at this point in the history
  • Loading branch information
ryantanjunming committed Aug 16, 2023
1 parent e93c0c3 commit 2fead6f
Showing 1 changed file with 6 additions and 6 deletions.
12 changes: 6 additions & 6 deletions modules/firehose/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ resource "aws_s3_bucket_public_access_block" "firehose_bucket_bucket_access" {

resource "aws_iam_role" "firehose_to_coralogix" {
tags = local.tags
name = "${var.firehose_stream}-iam"
name = "${var.firehose_stream}-role"
assume_role_policy = jsonencode({
"Version" = "2012-10-17",
"Statement" = [
Expand All @@ -94,7 +94,7 @@ resource "aws_iam_role" "firehose_to_coralogix" {
]
})
inline_policy {
name = "${var.firehose_stream}-iam"
name = "${var.firehose_stream}-policy"
policy = jsonencode({
"Version" = "2012-10-17",
"Statement" = [
Expand Down Expand Up @@ -329,14 +329,14 @@ data "aws_iam_policy_document" "lambda_assume_role" {

resource "aws_iam_role" "lambda_iam_role" {
count = var.metric_enable == true ? 1 : 0
name = "${local.lambda_processor_name}-iam"
name = "${local.lambda_processor_name}-role"
tags = local.tags
assume_role_policy = data.aws_iam_policy_document.lambda_assume_role[count.index].json
}

resource "aws_iam_role_policy" "lambda_iam_policy" {
count = var.metric_enable == true ? 1 : 0
name = "${local.lambda_processor_name}-iam"
name = "${local.lambda_processor_name}-policy"
role = aws_iam_role.lambda_iam_role[count.index].id
policy = <<EOF
{
Expand Down Expand Up @@ -485,7 +485,7 @@ resource "aws_kinesis_firehose_delivery_stream" "coralogix_stream_metrics" {
resource "aws_iam_role" "metric_streams_to_firehose_role" {
tags = local.tags
count = var.enable_cloudwatch_metricstream && var.metric_enable ? 1 : 0
name = "${local.cloud_watch_metric_stream_name}-iam"
name = "${local.cloud_watch_metric_stream_name}-role"
assume_role_policy = <<EOF
{
"Version": "2012-10-17",
Expand All @@ -505,7 +505,7 @@ EOF

resource "aws_iam_role_policy" "metric_streams_to_firehose_policy" {
count = var.enable_cloudwatch_metricstream && var.metric_enable ? 1 : 0
name = "${local.cloud_watch_metric_stream_name}-iam"
name = "${local.cloud_watch_metric_stream_name}-policy"
role = aws_iam_role.metric_streams_to_firehose_role[0].id
policy = <<EOF
{
Expand Down

0 comments on commit 2fead6f

Please sign in to comment.