From cb5f9186a0e6d5ff9223064c2ba7c7ac9bad0b30 Mon Sep 17 00:00:00 2001 From: Matt McLoughlin Date: Fri, 10 Nov 2023 09:58:56 -0800 Subject: [PATCH 1/6] test lower limits --- .../scripts/helm/templates/tes-deployment.yaml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml b/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml index 1d082d81..91a74da5 100644 --- a/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml +++ b/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml @@ -112,11 +112,11 @@ spec: - containerPort: {{ .Values.service.tesPort }} resources: requests: - cpu: "1.5" - memory: "3072Mi" + cpu: "0.5" + memory: "1Gi" limits: - cpu: "8.0" - memory: "8192Mi" + cpu: "0.5" + memory: "1Gi" securityContext: allowPrivilegeEscalation: false capabilities: From 1c3f964c9851f879e8dc996c7f6ad95f13fac660 Mon Sep 17 00:00:00 2001 From: Matt McLoughlin Date: Fri, 10 Nov 2023 11:40:18 -0800 Subject: [PATCH 2/6] update values --- .../scripts/helm/templates/tes-deployment.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml b/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml index 91a74da5..b578e841 100644 --- a/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml +++ b/src/deploy-cromwell-on-azure/scripts/helm/templates/tes-deployment.yaml @@ -115,8 +115,8 @@ spec: cpu: "0.5" memory: "1Gi" limits: - cpu: "0.5" - memory: "1Gi" + cpu: "1" + memory: "2Gi" securityContext: allowPrivilegeEscalation: false capabilities: From bcc6bea7c24ed152b163cbd9ec839c9079d80953 Mon Sep 17 00:00:00 2001 From: Matt McLoughlin Date: Fri, 10 Nov 2023 11:51:01 -0800 Subject: [PATCH 3/6] reduce values --- .../scripts/helm/templates/cromwell-deployment.yaml | 8 ++++---- .../scripts/helm/templates/triggerservice-deployment.yaml | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/src/deploy-cromwell-on-azure/scripts/helm/templates/cromwell-deployment.yaml b/src/deploy-cromwell-on-azure/scripts/helm/templates/cromwell-deployment.yaml index 036cc28e..c4284c01 100644 --- a/src/deploy-cromwell-on-azure/scripts/helm/templates/cromwell-deployment.yaml +++ b/src/deploy-cromwell-on-azure/scripts/helm/templates/cromwell-deployment.yaml @@ -40,11 +40,11 @@ spec: - containerPort: {{ .Values.service.cromwellPort }} resources: requests: - cpu: "1.0" - memory: "1024Mi" + cpu: "1" + memory: "1.5Gi" limits: - cpu: "8.0" - memory: "4096Mi" + cpu: "2" + memory: "3Gi" securityContext: allowPrivilegeEscalation: false volumeMounts: diff --git a/src/deploy-cromwell-on-azure/scripts/helm/templates/triggerservice-deployment.yaml b/src/deploy-cromwell-on-azure/scripts/helm/templates/triggerservice-deployment.yaml index 337ac000..5bd77686 100644 --- a/src/deploy-cromwell-on-azure/scripts/helm/templates/triggerservice-deployment.yaml +++ b/src/deploy-cromwell-on-azure/scripts/helm/templates/triggerservice-deployment.yaml @@ -51,10 +51,10 @@ spec: resources: requests: cpu: "0.5" - memory: "256Mi" + memory: "0.5Gi" limits: - cpu: "2.0" - memory: "1024Mi" + cpu: "1" + memory: "1Gi" securityContext: allowPrivilegeEscalation: false restartPolicy: Always From 860bc551a83e83241aa5672bcb2d55004273871f Mon Sep 17 00:00:00 2001 From: Matt McLoughlin Date: Fri, 10 Nov 2023 11:59:22 -0800 Subject: [PATCH 4/6] smaller vm size and count --- src/deploy-cromwell-on-azure/Configuration.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/deploy-cromwell-on-azure/Configuration.cs b/src/deploy-cromwell-on-azure/Configuration.cs index e6380abf..c42d630f 100644 --- a/src/deploy-cromwell-on-azure/Configuration.cs +++ b/src/deploy-cromwell-on-azure/Configuration.cs @@ -34,7 +34,7 @@ public abstract class UserAccessibleConfiguration public string SubscriptionId { get; set; } public string RegionName { get; set; } public string MainIdentifierPrefix { get; set; } = "coa"; - public string VmSize { get; set; } = "Standard_D4s_v3"; + public string VmSize { get; set; } = "Standard_D2s_v3"; public string VnetAddressSpace { get; set; } = "10.1.0.0/16"; // 10.1.0.0 - 10.1.255.255, 65536 IPs // Address space for CoA services. public string VmSubnetAddressSpace { get; set; } = "10.1.0.0/24"; // 10.1.0.0 - 10.1.0.255, 256 IPs @@ -54,7 +54,7 @@ public abstract class UserAccessibleConfiguration public string AksCoANamespace { get; set; } = "coa"; public bool ManualHelmDeployment { get; set; } public string HelmBinaryPath { get; set; } = OperatingSystem.IsWindows() ? @"C:\ProgramData\chocolatey\bin\helm.exe" : "/usr/local/bin/helm"; - public int AksPoolSize { get; set; } = 2; + public int AksPoolSize { get; set; } = 1; public bool? CrossSubscriptionAKSDeployment { get; set; } = null; public bool Silent { get; set; } public bool DeleteResourceGroupOnFailure { get; set; } From 6380943e2d89115162de1d43826fca9d75d3bade Mon Sep 17 00:00:00 2001 From: Matt McLoughlin Date: Fri, 10 Nov 2023 13:35:52 -0800 Subject: [PATCH 5/6] increase VM size again --- src/deploy-cromwell-on-azure/Configuration.cs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/deploy-cromwell-on-azure/Configuration.cs b/src/deploy-cromwell-on-azure/Configuration.cs index c42d630f..d1fe7046 100644 --- a/src/deploy-cromwell-on-azure/Configuration.cs +++ b/src/deploy-cromwell-on-azure/Configuration.cs @@ -34,7 +34,7 @@ public abstract class UserAccessibleConfiguration public string SubscriptionId { get; set; } public string RegionName { get; set; } public string MainIdentifierPrefix { get; set; } = "coa"; - public string VmSize { get; set; } = "Standard_D2s_v3"; + public string VmSize { get; set; } = "Standard_E2s_v5"; public string VnetAddressSpace { get; set; } = "10.1.0.0/16"; // 10.1.0.0 - 10.1.255.255, 65536 IPs // Address space for CoA services. public string VmSubnetAddressSpace { get; set; } = "10.1.0.0/24"; // 10.1.0.0 - 10.1.0.255, 256 IPs From e8020bacd1e66645f84e88ebcda62abb9fe52e78 Mon Sep 17 00:00:00 2001 From: Matt McLoughlin Date: Fri, 10 Nov 2023 15:01:07 -0800 Subject: [PATCH 6/6] smaller VM --- src/deploy-cromwell-on-azure/Configuration.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/deploy-cromwell-on-azure/Configuration.cs b/src/deploy-cromwell-on-azure/Configuration.cs index d1fe7046..48713c7f 100644 --- a/src/deploy-cromwell-on-azure/Configuration.cs +++ b/src/deploy-cromwell-on-azure/Configuration.cs @@ -34,7 +34,7 @@ public abstract class UserAccessibleConfiguration public string SubscriptionId { get; set; } public string RegionName { get; set; } public string MainIdentifierPrefix { get; set; } = "coa"; - public string VmSize { get; set; } = "Standard_E2s_v5"; + public string VmSize { get; set; } = "Standard_D2s_v3"; public string VnetAddressSpace { get; set; } = "10.1.0.0/16"; // 10.1.0.0 - 10.1.255.255, 65536 IPs // Address space for CoA services. public string VmSubnetAddressSpace { get; set; } = "10.1.0.0/24"; // 10.1.0.0 - 10.1.0.255, 256 IPs @@ -54,7 +54,7 @@ public abstract class UserAccessibleConfiguration public string AksCoANamespace { get; set; } = "coa"; public bool ManualHelmDeployment { get; set; } public string HelmBinaryPath { get; set; } = OperatingSystem.IsWindows() ? @"C:\ProgramData\chocolatey\bin\helm.exe" : "/usr/local/bin/helm"; - public int AksPoolSize { get; set; } = 1; + public int AksPoolSize { get; set; } = 2; public bool? CrossSubscriptionAKSDeployment { get; set; } = null; public bool Silent { get; set; } public bool DeleteResourceGroupOnFailure { get; set; }