Skip to content

fix(roles): use HTTPS source URLs for aws-iam-role modules #2

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions roles.tf
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# patch_role — assumed by patch_execution_role to perform EC2 operations in target accounts
module "patch_role" {
source = "git@github.e.it.census.gov:terraform-modules/aws-iam-role.git?ref=tf-upgrade"
source = "git::https://github.e.it.census.gov/terraform-modules/aws-iam-role.git?ref=tf-upgrade"

role_name = local.patch_role_name
role_description = format("Assumed by %v to execute patching operations on EC2 instances in target accounts", local.patch_execution_role_name)
Expand All @@ -12,7 +12,7 @@ module "patch_role" {

# patch_execution_role — assumed by Lambda functions to orchestrate patching
module "patch_execution_role" {
source = "git@github.e.it.census.gov:terraform-modules/aws-iam-role.git?ref=tf-upgrade"
source = "git::https://github.e.it.census.gov/terraform-modules/aws-iam-role.git?ref=tf-upgrade"

role_name = local.patch_execution_role_name
role_description = format("Assumed by Lambda functions to invoke patch operations, access SQS/RDS, and assume %v in target accounts", local.patch_role_name)
Expand All @@ -26,7 +26,7 @@ module "patch_execution_role" {
# This instance profile is used by the EC2 host running p4proxy to authenticate
# to RDS via IAM and to assume patch_execution_role for orchestration tasks.
module "patch_execproxy_role" {
source = "git@github.e.it.census.gov:terraform-modules/aws-iam-role.git?ref=tf-upgrade"
source = "git::https://github.e.it.census.gov/terraform-modules/aws-iam-role.git?ref=tf-upgrade"

role_name = local.patch_execproxy_role_name
role_description = format("Applied to the EC2 host running p4proxy; allows RDS IAM auth and assumption of %v", local.patch_execution_role_name)
Expand Down