Skip to content

Commit 2516ed4

Browse files
authored
Double the default job resources because more is happening in the containers (#976)
1 parent 62b4919 commit 2516ed4

File tree

1 file changed

+10
-5
lines changed

1 file changed

+10
-5
lines changed

server/Tingle.Dependabot/Models/Management/UpdateJobResources.cs

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@ namespace Tingle.Dependabot.Models.Management;
55

66
public class UpdateJobResources
77
{
8+
// the minimum is 0.25vCPU and 0.5GB but we need more because a lot is happening in the container
9+
private static readonly UpdateJobResources Default = new(cpu: 0.5, memory: 1);
10+
811
public UpdateJobResources() { } // required for deserialization
912

1013
public UpdateJobResources(double cpu, double memory)
@@ -33,14 +36,16 @@ public static UpdateJobResources FromEcosystem(string ecosystem)
3336
{
3437
return ecosystem switch
3538
{
36-
"npm" => new(cpu: 1, memory: 2),
37-
"yarn" => new(cpu: 1, memory: 2),
38-
"pnpm" => new(cpu: 1, memory: 2),
39-
"terraform" => new(cpu: 0.5, memory: 1),
40-
_ => new(cpu: 0.25, memory: 0.5), // the minimum
39+
"npm" => Default * 2,
40+
"yarn" => Default * 2,
41+
"pnpm" => Default * 2,
42+
_ => Default,
4143
};
4244
}
4345

46+
public static UpdateJobResources operator *(UpdateJobResources resources, double factor) => new(resources.Cpu * factor, resources.Memory * factor);
47+
public static UpdateJobResources operator /(UpdateJobResources resources, double factor) => new(resources.Cpu / factor, resources.Memory / factor);
48+
4449
public static implicit operator AppContainerResources(UpdateJobResources resources)
4550
{
4651
return new() { Cpu = resources.Cpu, Memory = $"{resources.Memory}Gi", };

0 commit comments

Comments
 (0)