Skip to content

Commit f595485

Browse files
fix: adding missing App urls and static vars to RCF (#1166)
* fix: adding missing App urls and static vars to RCF and changing function worker runtime to dotnet isolated in RCF * Env var fixes * Unrelated Sql Server sku resize to meet test loads * fix: removing requierment for ServiceBusConnectionString --------- Co-authored-by: Richard Kingston <[email protected]>
1 parent e6d0295 commit f595485

File tree

5 files changed

+25
-8
lines changed

5 files changed

+25
-8
lines changed

application/CohortManager/src/Functions/CaasIntegration/receiveCaasFile/receiveCaasFileConfig.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,9 +25,8 @@ public class ReceiveCaasFileConfig
2525
public string caasfolder_STORAGE { get; set; }
2626
[Required]
2727
public string inboundBlobName { get; set; }
28-
[Required]
2928
public string ServiceBusConnectionString { get; set; }
3029
public string GetOrchestrationStatusURL { get; set; }
3130
public bool UseNewFunctions { get; set; } = false;
32-
public string ParticipantManagementQueueName {get; set;}
31+
public string ParticipantManagementQueueName { get; set; }
3332
}

infrastructure/tf-core/environments/development.tfvars

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -255,6 +255,10 @@ function_apps = {
255255
env_var_name = "PMSRemoveParticipant"
256256
function_app_key = "RemoveParticipant"
257257
},
258+
{
259+
env_var_name = "PMSUpdateParticipant"
260+
function_app_key = "UpdateParticipant"
261+
},
258262
{
259263
env_var_name = "StaticValidationURL"
260264
function_app_key = "StaticValidation"
@@ -266,10 +270,6 @@ function_apps = {
266270
{
267271
env_var_name = "ScreeningLkpDataServiceURL"
268272
function_app_key = "ScreeningLkpDataService"
269-
},
270-
{
271-
env_var_name = "UseNewFunctions"
272-
function_app_key = "false"
273273
}
274274
],
275275
storage_containers = [
@@ -288,8 +288,10 @@ function_apps = {
288288
recordThresholdForBatching = "3"
289289
batchDivisionFactor = "2"
290290
CheckTimer = "100"
291+
delayBetweenChecks = "50"
291292
DemographicURI = "https://dev-uks-durable-demographic-function.azurewebsites.net/api/DurableDemographicFunction_HttpStart/"
292293
GetOrchestrationStatusURL = "https://dev-uks-durable-demographic-function.azurewebsites.net/api/GetOrchestrationStatus"
294+
maxNumberOfChecks = "50"
293295
AllowDeleteRecords = true
294296
TopicName = "DistributeParticipantQueue"
295297
UpdateQueueName = "update-participant-queue"
@@ -1275,7 +1277,7 @@ sqlserver = {
12751277
licence_type = "LicenseIncluded"
12761278
max_gb = 30
12771279
read_scale = false
1278-
sku = "S1"
1280+
sku = "S2"
12791281
storage_account_type = "Local"
12801282
zone_redundant = false
12811283
}

infrastructure/tf-core/environments/integration.tfvars

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -253,6 +253,10 @@ function_apps = {
253253
env_var_name = "PMSRemoveParticipant"
254254
function_app_key = "RemoveParticipant"
255255
},
256+
{
257+
env_var_name = "PMSUpdateParticipant"
258+
function_app_key = "UpdateParticipant"
259+
},
256260
{
257261
env_var_name = "StaticValidationURL"
258262
function_app_key = "StaticValidation"
@@ -282,11 +286,15 @@ function_apps = {
282286
recordThresholdForBatching = "3"
283287
batchDivisionFactor = "2"
284288
CheckTimer = "100"
289+
delayBetweenChecks = "50"
285290
DemographicURI = "https://int-uks-durable-demographic-function.azurewebsites.net/api/DurableDemographicFunction_HttpStart/"
286291
GetOrchestrationStatusURL = "https://int-uks-durable-demographic-function.azurewebsites.net/api/GetOrchestrationStatus"
292+
maxNumberOfChecks = "50"
287293
AllowDeleteRecords = true
294+
TopicName = "DistributeParticipantQueue"
288295
UpdateQueueName = "update-participant-queue"
289296
maxNumberOfChecks = "50"
297+
ServiceBusConnectionString = ""
290298
UseNewFunctions = "false"
291299
}
292300

infrastructure/tf-core/environments/nft.tfvars

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -260,6 +260,10 @@ function_apps = {
260260
env_var_name = "PMSRemoveParticipant"
261261
function_app_key = "RemoveParticipant"
262262
},
263+
{
264+
env_var_name = "PMSUpdateParticipant"
265+
function_app_key = "UpdateParticipant"
266+
},
263267
{
264268
env_var_name = "StaticValidationURL"
265269
function_app_key = "StaticValidation"
@@ -289,11 +293,15 @@ function_apps = {
289293
recordThresholdForBatching = "3"
290294
batchDivisionFactor = "2"
291295
CheckTimer = "100"
296+
delayBetweenChecks = "50"
292297
DemographicURI = "https://nft-uks-durable-demographic-function.azurewebsites.net/api/DurableDemographicFunction_HttpStart/"
293298
GetOrchestrationStatusURL = "https://nft-uks-durable-demographic-function.azurewebsites.net/api/GetOrchestrationStatus"
299+
maxNumberOfChecks = "50"
294300
AllowDeleteRecords = true
301+
TopicName = "DistributeParticipantQueue"
295302
UpdateQueueName = "update-participant-queue"
296303
maxNumberOfChecks = "50"
304+
ServiceBusConnectionString = ""
297305
UseNewFunctions = "false"
298306
}
299307

infrastructure/tf-core/function_app.tf

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ resource "azurerm_role_assignment" "function_send_to_topic" {
112112
locals {
113113
app_settings_common = {
114114
DOCKER_ENABLE_CI = var.function_apps.docker_CI_enable
115-
FUNCTION_WORKER_RUNTIME = "dotnet"
115+
FUNCTION_WORKER_RUNTIME = "dotnet-isolated"
116116
REMOTE_DEBUGGING_ENABLED = var.function_apps.remote_debugging_enabled
117117
WEBSITES_ENABLE_APP_SERVICE_STORAGE = var.function_apps.enable_appsrv_storage
118118
WEBSITE_PULL_IMAGE_OVER_VNET = var.function_apps.pull_image_over_vnet

0 commit comments

Comments
 (0)