Skip to content

Commit

Permalink
remove role
Browse files Browse the repository at this point in the history
  • Loading branch information
eze-godoy committed Jan 4, 2023
1 parent 6a79448 commit f139935
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 51 deletions.
25 changes: 0 additions & 25 deletions template/security/global/base-identities/roles.tf
Original file line number Diff line number Diff line change
Expand Up @@ -29,31 +29,6 @@ module "iam_assumable_role_devops" {
tags = local.tags
}

#
# Assumable Role Cross-Account: Admin
#
module "iam_assumable_role_admin" {
source = "github.com/binbashar/terraform-aws-iam.git//modules/iam-assumable-role?ref=v4.1.0"

trusted_role_arns = [
"arn:aws:iam::${var.security_account_id}:root"
]

create_role = true
role_name = "Admin"
admin_role_policy_arn = "arn:aws:iam::aws:policy/AdministratorAccess"
attach_admin_policy = true
role_path = "/"

#
# MFA setup
#
role_requires_mfa = true
mfa_age = 43200 # Maximum CLI/API session duration in seconds between 3600 and 43200
max_session_duration = 3600 # Max age of valid MFA (in seconds) for roles which require MFA

tags = local.tags
}

#
# Assumable Role Cross-Account: Auditor Role
Expand Down
26 changes: 0 additions & 26 deletions template/shared/global/base-identities/roles.tf
Original file line number Diff line number Diff line change
Expand Up @@ -29,32 +29,6 @@ module "iam_assumable_role_devops" {
tags = local.tags
}

#
# Assumable Role Cross-Account: Admin
#
module "iam_assumable_role_admin" {
source = "github.com/binbashar/terraform-aws-iam.git//modules/iam-assumable-role?ref=v4.1.0"

trusted_role_arns = [
"arn:aws:iam::${var.security_account_id}:root"
]

create_role = true
role_name = "Admin"
admin_role_policy_arn = "arn:aws:iam::aws:policy/AdministratorAccess"
attach_admin_policy = true
role_path = "/"

#
# MFA setup
#
role_requires_mfa = true
mfa_age = 43200 # Maximum CLI/API session duration in seconds between 3600 and 43200
max_session_duration = 3600 # Max age of valid MFA (in seconds) for roles which require MFA

tags = local.tags
}

#
# Assumable Role Cross-Account: Auditor Role
#
Expand Down

0 comments on commit f139935

Please sign in to comment.