From 96c784ec79f812ebaff3b6fba2c20a0fba16c711 Mon Sep 17 00:00:00 2001 From: Silas Della Contrada Date: Thu, 4 May 2023 09:56:52 +0200 Subject: [PATCH] Add tests for hugepages resource requests/limits --- pkg/cluster/k8sres_test.go | 46 ++++++++++++++++++++++++++++++++++++-- 1 file changed, 44 insertions(+), 2 deletions(-) diff --git a/pkg/cluster/k8sres_test.go b/pkg/cluster/k8sres_test.go index a88320deb..10ed59077 100644 --- a/pkg/cluster/k8sres_test.go +++ b/pkg/cluster/k8sres_test.go @@ -5,9 +5,8 @@ import ( "fmt" "reflect" "sort" - "time" - "testing" + "time" "github.com/stretchr/testify/assert" @@ -2979,6 +2978,49 @@ func TestGenerateResourceRequirements(t *testing.T) { ResourceLimits: acidv1.ResourceDescription{CPU: "1", Memory: "2Gi"}, }, }, + { + subTest: "test HugePages are passed through to the postgres container", + config: config.Config{ + Resources: configResources, + PodManagementPolicy: "ordered_ready", + }, + pgSpec: acidv1.Postgresql{ + ObjectMeta: metav1.ObjectMeta{ + Name: clusterName, + Namespace: namespace, + }, + Spec: acidv1.PostgresSpec{ + Resources: &acidv1.Resources{ + ResourceRequests: acidv1.ResourceDescription{ + HugePages2Mi: "128Mi", + HugePages1Gi: "1Gi", + }, + ResourceLimits: acidv1.ResourceDescription{ + HugePages2Mi: "256Mi", + HugePages1Gi: "2Gi", + }, + }, + TeamID: "acid", + Volume: acidv1.Volume{ + Size: "1G", + }, + }, + }, + expectedResources: acidv1.Resources{ + ResourceRequests: acidv1.ResourceDescription{ + CPU: "100m", + Memory: "100Mi", + HugePages2Mi: "128Mi", + HugePages1Gi: "1Gi", + }, + ResourceLimits: acidv1.ResourceDescription{ + CPU: "1", + Memory: "500Mi", + HugePages2Mi: "256Mi", + HugePages1Gi: "2Gi", + }, + }, + }, } for _, tt := range tests {