Skip to content

Commit

Permalink
feat: update azurerm and azuread provider version
Browse files Browse the repository at this point in the history
  • Loading branch information
ishabakeh committed Nov 23, 2023
1 parent 23e30c9 commit 39e4f07
Show file tree
Hide file tree
Showing 12 changed files with 41 additions and 35 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/workflow.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Check out code
uses: actions/checkout@v1
uses: actions/checkout@v2

- uses: hashicorp/setup-terraform@v1
- uses: hashicorp/setup-terraform@v3
with:
terraform_version: ^1.1

Expand Down
6 changes: 3 additions & 3 deletions main.tf
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
terraform {
required_version = ">= 1.1"
required_version = "> 1.1"
required_providers {
azurerm = {
source = "hashicorp/azurerm"
version = "3.3.0"
version = "3.81.0"
}
azuread = {
source = "hashicorp/azuread"
version = "2.18.0"
version = "2.46.0"
}
}
}
Expand Down
10 changes: 5 additions & 5 deletions modules/meshcloud-idp-lookup-service-principal/module.tf
Original file line number Diff line number Diff line change
@@ -1,21 +1,21 @@
terraform {
required_version = ">= 1.0"
required_version = "> 1.0"
required_providers {
azurerm = {
source = "hashicorp/azurerm"
version = "3.3.0"
version = "3.81.0"
}
azuread = {
source = "hashicorp/azuread"
version = "2.18.0"
version = "2.46.0"
}
}
}

data "azuread_application_published_app_ids" "well_known" {}

data "azuread_service_principal" "msgraph" {
application_id = data.azuread_application_published_app_ids.well_known.result.MicrosoftGraph
client_id = data.azuread_application_published_app_ids.well_known.result.MicrosoftGraph
}

resource "azuread_application" "meshcloud_idp_lookup" {
Expand Down Expand Up @@ -50,7 +50,7 @@ resource "azuread_application" "meshcloud_idp_lookup" {
}

resource "azuread_service_principal" "meshcloud_idp_lookup" {
application_id = azuread_application.meshcloud_idp_lookup.application_id
client_id = azuread_application.meshcloud_idp_lookup.client_id
}

resource "azuread_app_role_assignment" "meshcloud_idp_lookup" {
Expand Down
2 changes: 1 addition & 1 deletion modules/meshcloud-idp-lookup-service-principal/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ output "service_principal" {
description = "Service Principal application id and object id"
value = {
object_id = azuread_service_principal.meshcloud_idp_lookup.id
app_id = azuread_service_principal.meshcloud_idp_lookup.application_id
app_id = azuread_service_principal.meshcloud_idp_lookup.client_id
password = "Execute `terraform output idp_lookup_service_principal_password` to see the password"
}
}
Expand Down
11 changes: 6 additions & 5 deletions modules/meshcloud-metering-service-principal/module.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@
// Terraform Settings
//---------------------------------------------------------------------------
terraform {
required_version = ">= 1.0"
required_version = "> 1.0"
required_providers {
azurerm = {
source = "hashicorp/azurerm"
version = "3.3.0"
version = "3.81.0"
}
azuread = {
source = "hashicorp/azuread"
version = "2.18.0"
version = "2.46.0"
}
}
}
Expand Down Expand Up @@ -53,6 +53,7 @@ resource "azurerm_role_assignment" "meshcloud_metering" {
scope = var.scope
role_definition_name = "Cost Management Reader"
principal_id = azuread_service_principal.meshcloud_metering.id
depends_on = [azuread_service_principal.meshcloud_metering]
}


Expand All @@ -78,7 +79,7 @@ resource "azuread_application" "meshcloud_metering" {
// Create New Enterprise application and associate it with the previously created app
//---------------------------------------------------------------------------
resource "azuread_service_principal" "meshcloud_metering" {
application_id = azuread_application.meshcloud_metering.application_id
client_id = azuread_application.meshcloud_metering.client_id
feature_tags {
enterprise = true
}
Expand All @@ -92,7 +93,7 @@ resource "time_rotating" "replicator_secret_rotation" {
}

resource "azuread_application_password" "application_pw" {
application_object_id = azuread_application.meshcloud_metering.object_id
application_id = azuread_application.meshcloud_metering.id
rotate_when_changed = {
rotation = time_rotating.replicator_secret_rotation.id
}
Expand Down
2 changes: 1 addition & 1 deletion modules/meshcloud-metering-service-principal/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ output "credentials" {
description = "Service Principal application id and object id"
value = {
Enterprise_Application_Object_ID = azuread_service_principal.meshcloud_metering.id
Application_Client_ID = azuread_application.meshcloud_metering.application_id
Application_Client_ID = azuread_application.meshcloud_metering.client_id
Client_Secret = "Execute `terraform output metering_client_secret` to see the password"
}
}
Expand Down
16 changes: 10 additions & 6 deletions modules/meshcloud-replicator-service-principal/module.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,15 @@
// Terraform Settings
//---------------------------------------------------------------------------
terraform {
required_version = ">= 1.0"
required_version = "> 1.0"
required_providers {
azurerm = {
source = "hashicorp/azurerm"
version = "3.3.0"
version = "3.81.0"
}
azuread = {
source = "hashicorp/azuread"
version = "2.18.0"
version = "2.46.0"
}
}
}
Expand Down Expand Up @@ -71,7 +71,7 @@ resource "azurerm_role_definition" "meshcloud_replicator" {
data "azuread_application_published_app_ids" "well_known" {}

data "azuread_service_principal" "msgraph" {
application_id = data.azuread_application_published_app_ids.well_known.result.MicrosoftGraph
client_id = data.azuread_application_published_app_ids.well_known.result.MicrosoftGraph
}

//---------------------------------------------------------------------------
Expand Down Expand Up @@ -148,7 +148,7 @@ resource "time_rotating" "replicator_secret_rotation" {
rotation_days = 365
}
resource "azuread_application_password" "application_pw" {
application_object_id = azuread_application.meshcloud_replicator.object_id
application_id = azuread_application.meshcloud_replicator.id
rotate_when_changed = {
rotation = time_rotating.replicator_secret_rotation.id
}
Expand All @@ -158,7 +158,7 @@ resource "azuread_application_password" "application_pw" {
// Create new Enterprise Application and associate it with the previous application
//---------------------------------------------------------------------------
resource "azuread_service_principal" "meshcloud_replicator" {
application_id = azuread_application.meshcloud_replicator.application_id
client_id = azuread_application.meshcloud_replicator.client_id
feature_tags {
enterprise = true
}
Expand All @@ -176,6 +176,7 @@ resource "azurerm_role_assignment" "meshcloud_replicator" {
scope = var.scope
role_definition_id = azurerm_role_definition.meshcloud_replicator.role_definition_resource_id
principal_id = azuread_service_principal.meshcloud_replicator.id
depends_on = [azuread_service_principal.meshcloud_replicator]
}

//---------------------------------------------------------------------------
Expand All @@ -185,18 +186,21 @@ resource "azuread_app_role_assignment" "meshcloud_replicator-directory" {
app_role_id = data.azuread_service_principal.msgraph.app_role_ids["Directory.Read.All"]
principal_object_id = azuread_service_principal.meshcloud_replicator.object_id
resource_object_id = data.azuread_service_principal.msgraph.object_id
depends_on = [azuread_application.meshcloud_replicator]
}

resource "azuread_app_role_assignment" "meshcloud_replicator-group" {
app_role_id = data.azuread_service_principal.msgraph.app_role_ids["Group.ReadWrite.All"]
principal_object_id = azuread_service_principal.meshcloud_replicator.object_id
resource_object_id = data.azuread_service_principal.msgraph.object_id
depends_on = [azuread_application.meshcloud_replicator]
}

resource "azuread_app_role_assignment" "meshcloud_replicator-user" {
app_role_id = data.azuread_service_principal.msgraph.app_role_ids["User.Invite.All"]
principal_object_id = azuread_service_principal.meshcloud_replicator.object_id
resource_object_id = data.azuread_service_principal.msgraph.object_id
depends_on = [azuread_application.meshcloud_replicator]
}


Expand Down
2 changes: 1 addition & 1 deletion modules/meshcloud-replicator-service-principal/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ output "credentials" {
description = "Service Principal application id and object id"
value = {
Enterprise_Application_Object_ID = azuread_service_principal.meshcloud_replicator.id
Application_Client_ID = azuread_application.meshcloud_replicator.application_id
Application_Client_ID = azuread_application.meshcloud_replicator.client_id
Client_Secret = "Execute `terraform output replicator_client_secret` to see the password"
}
}
Expand Down
10 changes: 5 additions & 5 deletions modules/meshcloud-sso/module.tf
Original file line number Diff line number Diff line change
@@ -1,21 +1,21 @@
terraform {
required_version = ">= 1.0"
required_version = "> 1.0"
required_providers {
azurerm = {
source = "hashicorp/azurerm"
version = "3.3.0"
version = "3.81.0"
}
azuread = {
source = "hashicorp/azuread"
version = "2.18.0"
version = "2.46.0"
}
}
}

data "azuread_application_published_app_ids" "well_known" {}

data "azuread_service_principal" "msgraph" {
application_id = data.azuread_application_published_app_ids.well_known.result.MicrosoftGraph
client_id = data.azuread_application_published_app_ids.well_known.result.MicrosoftGraph
}

resource "azuread_application" "meshcloud_sso" {
Expand Down Expand Up @@ -44,5 +44,5 @@ resource "azuread_application" "meshcloud_sso" {
}

resource "azuread_application_password" "meshcloud_sso" {
application_object_id = azuread_application.meshcloud_sso.object_id
application_id = azuread_application.meshcloud_sso.id
}
2 changes: 1 addition & 1 deletion modules/meshcloud-sso/outputs.tf
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ output "app_registration" {
description = "Application registration application id and object id"
value = {
object_id = azuread_application.meshcloud_sso.object_id
app_id = azuread_application.meshcloud_sso.application_id
app_id = azuread_application.meshcloud_sso.client_id
}
}

Expand Down
9 changes: 5 additions & 4 deletions modules/uami-blueprint-user-principal/module.tf
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
terraform {
required_version = ">= 1.0"
required_version = "> 1.0"
required_providers {
azurerm = {
source = "hashicorp/azurerm"
version = "3.3.0"
version = "3.81.0"
}
azuread = {
source = "hashicorp/azuread"
version = "2.18.0"
version = "2.46.0"
}
}
}
Expand All @@ -17,7 +17,7 @@ resource "azuread_application" "uami_blueprint_principal" {
}

resource "azuread_service_principal" "uami_blueprint_principal" {
application_id = azuread_application.uami_blueprint_principal.application_id
client_id = azuread_application.uami_blueprint_principal.client_id
}

resource "azuread_service_principal_password" "service_principal_pw" {
Expand All @@ -30,6 +30,7 @@ resource "azurerm_role_assignment" "service_principal_pw" {
principal_id = azuread_service_principal.uami_blueprint_principal.id
scope = "/subscriptions/${var.subscriptions[count.index]}"
role_definition_name = "Contributor"
depends_on = [azuread_service_principal.uami_blueprint_principal]
}

# facilitate migration from v0.1.0 of the module
Expand Down
2 changes: 1 addition & 1 deletion modules/uami-blueprint-user-principal/outputs.tf
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
output "service_principal" {
value = {
object_id = azuread_service_principal.uami_blueprint_principal.id
app_id = azuread_service_principal.uami_blueprint_principal.application_id
app_id = azuread_service_principal.uami_blueprint_principal.client_id
password = "Execute `terraform output uami_blueprint_user_principal_password` to see the password"
}
}
Expand Down

0 comments on commit 39e4f07

Please sign in to comment.