Merge pull request #56525 from tianshapjq/testcase-helpers_linux.go
Automatic merge from submit-queue. If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. new testcase to helpers_linux.go new testcase to helpers_linux.go, PTAL. ```release-note NONE ```
This commit is contained in:
commit
4d6a6ced8c
@ -24,6 +24,7 @@ import (
|
|||||||
|
|
||||||
"k8s.io/api/core/v1"
|
"k8s.io/api/core/v1"
|
||||||
"k8s.io/apimachinery/pkg/api/resource"
|
"k8s.io/apimachinery/pkg/api/resource"
|
||||||
|
"strconv"
|
||||||
)
|
)
|
||||||
|
|
||||||
// getResourceList returns a ResourceList with the
|
// getResourceList returns a ResourceList with the
|
||||||
@ -230,3 +231,82 @@ func TestMilliCPUToQuota(t *testing.T) {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestHugePageLimits(t *testing.T) {
|
||||||
|
Mi := int64(1024 * 1024)
|
||||||
|
type inputStruct struct {
|
||||||
|
key string
|
||||||
|
input string
|
||||||
|
}
|
||||||
|
|
||||||
|
testCases := []struct {
|
||||||
|
name string
|
||||||
|
inputs []inputStruct
|
||||||
|
expected map[int64]int64
|
||||||
|
}{
|
||||||
|
{
|
||||||
|
name: "no valid hugepages",
|
||||||
|
inputs: []inputStruct{
|
||||||
|
{
|
||||||
|
key: "2Mi",
|
||||||
|
input: "128",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
expected: map[int64]int64{},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "2Mi only",
|
||||||
|
inputs: []inputStruct{
|
||||||
|
{
|
||||||
|
key: v1.ResourceHugePagesPrefix + "2Mi",
|
||||||
|
input: "128",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
expected: map[int64]int64{2 * Mi: 128},
|
||||||
|
},
|
||||||
|
{
|
||||||
|
name: "2Mi and 4Mi",
|
||||||
|
inputs: []inputStruct{
|
||||||
|
{
|
||||||
|
key: v1.ResourceHugePagesPrefix + "2Mi",
|
||||||
|
input: "128",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: v1.ResourceHugePagesPrefix + strconv.FormatInt(2*Mi, 10),
|
||||||
|
input: "256",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: v1.ResourceHugePagesPrefix + "4Mi",
|
||||||
|
input: "512",
|
||||||
|
},
|
||||||
|
{
|
||||||
|
key: "4Mi",
|
||||||
|
input: "1024",
|
||||||
|
},
|
||||||
|
},
|
||||||
|
expected: map[int64]int64{2 * Mi: 384, 4 * Mi: 512},
|
||||||
|
},
|
||||||
|
}
|
||||||
|
|
||||||
|
for _, testcase := range testCases {
|
||||||
|
t.Run(testcase.name, func(t *testing.T) {
|
||||||
|
resourceList := v1.ResourceList{}
|
||||||
|
|
||||||
|
for _, input := range testcase.inputs {
|
||||||
|
value, err := resource.ParseQuantity(input.input)
|
||||||
|
if err != nil {
|
||||||
|
t.Fatalf("error in parsing hugepages, value: %s", input.input)
|
||||||
|
} else {
|
||||||
|
resourceList[v1.ResourceName(input.key)] = value
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
resultValue := HugePageLimits(resourceList)
|
||||||
|
|
||||||
|
if !reflect.DeepEqual(testcase.expected, resultValue) {
|
||||||
|
t.Errorf("unexpected result, expected: %v, actual: %v", testcase.expected, resultValue)
|
||||||
|
}
|
||||||
|
})
|
||||||
|
|
||||||
|
}
|
||||||
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user