Skip to content

Commit c47446a

Browse files
authored
This PR is to add comments to TestTransformResourcesForWorkerWithValue in pkg/ddc/alluxio/transform_resources_test.go. (#4869)
* Add comments/notation to TestTransformResourcesForWorkerWithValue in pkg/ddc/alluxio/transform_resources_test.go. Signed-off-by: YYYYimo <2623855900@qq.com> * Fix: Apply gofmt and update go.sum Signed-off-by: YYYYimo <2623855900@qq.com> --------- Signed-off-by: YYYYimo <2623855900@qq.com>
1 parent 6a59442 commit c47446a

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

pkg/ddc/alluxio/transform_resources_test.go

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -274,6 +274,15 @@ func TestTransformResourcesForWorkerWithTieredStore(t *testing.T) {
274274
}
275275
}
276276

277+
// TestTransformResourcesForWorkerWithValue tests the transformResourcesForWorker function.
278+
// It checks if the worker's memory resources (limits and requests) are correctly
279+
// transformed based on the AlluxioRuntime specification, particularly considering
280+
// the memory quota defined in the tiered storage configuration.
281+
// The test includes scenarios where:
282+
// 1. The specified memory limit is less than the memory tier quota, resulting in an error
283+
// and the original resource values being retained.
284+
// 2. The specified memory limit is sufficient, and the memory request is adjusted
285+
// to match the memory tier quota.
277286
func TestTransformResourcesForWorkerWithValue(t *testing.T) {
278287

279288
resources := corev1.ResourceRequirements{}

0 commit comments

Comments
 (0)