Merge pull request #52338 from PiotrProkop/fix-hugepages

Automatic merge from submit-queue (batch tested with PRs 51041, 52297, 52296, 52335, 52338)

Fix pagesize mount option name

**What this PR does / why we need it**:
Fixes #52337  .
This commit is contained in:
Kubernetes Submit Queue 2017-09-12 11:10:18 -07:00 committed by GitHub
commit b05d8ad1ec
2 changed files with 6 additions and 5 deletions

View File

@ -54,7 +54,8 @@ type emptyDirPlugin struct {
var _ volume.VolumePlugin = &emptyDirPlugin{}
const (
emptyDirPluginName = "kubernetes.io/empty-dir"
emptyDirPluginName = "kubernetes.io/empty-dir"
hugePagesPageSizeMountOption = "pagesize"
)
func getPath(uid types.UID, volName string, host volume.VolumeHost) string {
@ -321,7 +322,7 @@ func getPageSizeMountOptionFromPod(pod *v1.Pod) (string, error) {
return "", fmt.Errorf("hugePages storage requested, but there is no resource request for huge pages.")
}
return fmt.Sprintf("pageSize=%s", pageSize.String()), nil
return fmt.Sprintf("%s=%s", hugePagesPageSizeMountOption, pageSize.String()), nil
}

View File

@ -323,7 +323,7 @@ func TestGetHugePagesMountOptions(t *testing.T) {
},
},
shouldFail: false,
expectedResult: "pageSize=2Mi",
expectedResult: "pagesize=2Mi",
},
"testWithProperValuesAndDifferentPageSize": {
pod: &v1.Pod{
@ -347,7 +347,7 @@ func TestGetHugePagesMountOptions(t *testing.T) {
},
},
shouldFail: false,
expectedResult: "pageSize=1Gi",
expectedResult: "pagesize=1Gi",
},
"InitContainerAndContainerHasProperValues": {
pod: &v1.Pod{
@ -371,7 +371,7 @@ func TestGetHugePagesMountOptions(t *testing.T) {
},
},
shouldFail: false,
expectedResult: "pageSize=1Gi",
expectedResult: "pagesize=1Gi",
},
"InitContainerAndContainerHasDifferentPageSizes": {
pod: &v1.Pod{