Skip to content

Commit

Permalink
Merge pull request #5033 from dfe-analytical-services/EES-5283
Browse files Browse the repository at this point in the history
EES-5283 Add missing `MetaInsertBatchSize` app setting to Public API Data Processor function app
  • Loading branch information
benoutram authored Jul 8, 2024
2 parents 4bdbe8d + 78b9bf5 commit 22994d0
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,8 +27,8 @@ param appServicePlanOS string = 'Linux'
])
param functionAppRuntime string = 'dotnet'

@description('Specifies the additional setting to add to the functionapp.')
param settings object = {}
@description('Specifies the additional setting to add to the Function App')
param appSettings object = {}

@description('A set of tags with which to tag the resource in Azure')
param tagValues object
Expand Down Expand Up @@ -401,7 +401,7 @@ module functionAppSlotSettings 'appServiceSlotConfig.bicep' = {
// site is being viewed.
'SLOT_NAME'
]
commonSettings: union(settings, {
commonSettings: union(appSettings, {

// This tells the Function App where to store its "azure-webjobs-hosts" and "azure-webjobs-secrets" files.
AzureWebJobsStorage: '@Microsoft.KeyVault(VaultName=${keyVaultName};SecretName=${sharedStorageAccountModule.outputs.connectionStringSecretName})'
Expand Down
3 changes: 3 additions & 0 deletions infrastructure/templates/public-api/main.bicep
Original file line number Diff line number Diff line change
Expand Up @@ -393,6 +393,9 @@ module dataProcessorFunctionAppModule 'components/functionApp.bicep' = {
path: '/api/HealthCheck'
unhealthyMetricName: '${subscription}PublicDataProcessorUnhealthy'
}
appSettings: {
AppSettings__MetaInsertBatchSize: 1000
}
azureFileShares: [{
storageName: dataFilesFileShareModule.outputs.fileShareName
storageAccountKey: publicApiStorageAccountAccessKey
Expand Down

0 comments on commit 22994d0

Please sign in to comment.