Skip to content

Commit

Permalink
Merge pull request #58 from cerebrotech/steved-iam-fix
Browse files Browse the repository at this point in the history
Revert "remove docker-registry service account binding"
  • Loading branch information
Saahil Dhulla authored Apr 28, 2020
2 parents f3f7231 + 2edaf25 commit 978943c
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 0 deletions.
8 changes: 8 additions & 0 deletions service-accounts.tf
Original file line number Diff line number Diff line change
Expand Up @@ -76,3 +76,11 @@ resource "google_project_iam_member" "platform_monitoring" {
role = "roles/monitoring.metricWriter"
member = "serviceAccount:${google_service_account.platform.email}"
}

resource "google_service_account_iam_binding" "platform_docker_registry" {
service_account_id = google_service_account.platform.name
role = "roles/iam.workloadIdentityUser"
members = [
"serviceAccount:${var.project}.svc.id.goog[${var.platform_namespace}/docker-registry]",
]
}
6 changes: 6 additions & 0 deletions variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -194,3 +194,9 @@ variable "platform_node_type" {
type = string
default = "n1-standard-8"
}

variable "platform_namespace" {
type = string
description = "Platform namespace that is used for generating the service account binding for docker-registry"
default = "domino-platform"
}

0 comments on commit 978943c

Please sign in to comment.