Skip to content

Commit

Permalink
fix key vault resource gorup error
Browse files Browse the repository at this point in the history
  • Loading branch information
luxu-ms committed Feb 29, 2024
1 parent 35a91bd commit a0ff493
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 1 deletion.
2 changes: 2 additions & 0 deletions Environments/Todo-Nodejs-Mongo-AKS/app/db.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ param collections array = [
]
param databaseName string = ''
param keyVaultName string
param keyVaultResourceGroupName string

// Because databaseName is optional in main.bicep, we make sure the database name is set here.
var defaultDatabaseName = 'Todo'
Expand All @@ -31,6 +32,7 @@ module cosmos '../core/database/cosmos/mongo/cosmos-mongo-db.bicep' = {
location: location
collections: collections
keyVaultName: keyVaultName
keyVaultResourceGroupName: keyVaultResourceGroupName
tags: tags
}
}
Expand Down
2 changes: 2 additions & 0 deletions Environments/Todo-Nodejs-Mongo-AKS/main.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ param location string = resourceGroup().location
param cosmosAccountName string = ''
param cosmosDatabaseName string = ''
param keyvaultName string = ''
param keyVaultResourceGroupName string = resourceGroup().name

var abbrs = loadJsonContent('./abbreviations.json')
var resourceToken = toLower(uniqueString(subscription().id, environmentName, location))
Expand All @@ -24,6 +25,7 @@ module cosmos './app/db.bicep' = {
location: location
tags: tags
keyVaultName: keyvaultName
keyVaultResourceGroupName: keyVaultResourceGroupName
}
}

Expand Down
8 changes: 7 additions & 1 deletion Environments/Todo-Nodejs-Mongo-AKS/manifest.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,4 +22,10 @@ parameters:
name: "keyvaultName (e.g. kv-abc123)"
description: "keyvault name that store the secret for mongo connection string"
type: string
required: true
required: true

- id: "keyvaultResourceGroupName"
name: "keyvaultResourceGroupName (e.g. rg-abc123)"
description: "keyvault group name contains the key vault above"
type: string
required: true

0 comments on commit a0ff493

Please sign in to comment.