From e5bed175741461edaa1de5dede0486d9f6afd328 Mon Sep 17 00:00:00 2001
From: allencloud <allen.sun@daocloud.io>
Date: Mon, 11 Jul 2016 18:29:17 +0800
Subject: [PATCH] update comparison log in container config
Signed-off-by: allencloud <allen.sun@daocloud.io>
---
daemon/daemon_unix.go | 36 +++++++++----------
daemon/daemon_windows.go | 24 ++++++-------
integration-cli/docker_api_containers_test.go | 2 +-
integration-cli/docker_cli_run_unix_test.go | 2 +-
4 files changed, 32 insertions(+), 32 deletions(-)
diff --git a/daemon/daemon_unix.go b/daemon/daemon_unix.go
index 15015b8e7d..6bac914a2b 100644
--- a/daemon/daemon_unix.go
+++ b/daemon/daemon_unix.go
@@ -288,8 +288,8 @@ func verifyContainerResources(resources *containertypes.Resources, sysInfo *sysi
return warnings, fmt.Errorf("You should always set the Memory limit when using Memoryswap limit, see usage")
}
if resources.MemorySwappiness != nil && *resources.MemorySwappiness != -1 && !sysInfo.MemorySwappiness {
- warnings = append(warnings, "Your kernel does not support memory swappiness capabilities, memory swappiness discarded.")
- logrus.Warn("Your kernel does not support memory swappiness capabilities, memory swappiness discarded.")
+ warnings = append(warnings, "Your kernel does not support memory swappiness capabilities. Memory swappiness discarded.")
+ logrus.Warn("Your kernel does not support memory swappiness capabilities. Memory swappiness discarded.")
resources.MemorySwappiness = nil
}
if resources.MemorySwappiness != nil {
@@ -307,7 +307,7 @@ func verifyContainerResources(resources *containertypes.Resources, sysInfo *sysi
return warnings, fmt.Errorf("Minimum memory reservation allowed is 4MB")
}
if resources.Memory > 0 && resources.MemoryReservation > 0 && resources.Memory < resources.MemoryReservation {
- return warnings, fmt.Errorf("Minimum memory limit should be larger than memory reservation limit, see usage")
+ return warnings, fmt.Errorf("Minimum memory limit can not be less than memory reservation limit, see usage")
}
if resources.KernelMemory > 0 && !sysInfo.KernelMemory {
warnings = append(warnings, "Your kernel does not support kernel memory limit capabilities. Limitation discarded.")
@@ -325,15 +325,15 @@ func verifyContainerResources(resources *containertypes.Resources, sysInfo *sysi
// only produce warnings if the setting wasn't to *disable* the OOM Kill; no point
// warning the caller if they already wanted the feature to be off
if *resources.OomKillDisable {
- warnings = append(warnings, "Your kernel does not support OomKillDisable, OomKillDisable discarded.")
- logrus.Warn("Your kernel does not support OomKillDisable, OomKillDisable discarded.")
+ warnings = append(warnings, "Your kernel does not support OomKillDisable. OomKillDisable discarded.")
+ logrus.Warn("Your kernel does not support OomKillDisable. OomKillDisable discarded.")
}
resources.OomKillDisable = nil
}
if resources.PidsLimit != 0 && !sysInfo.PidsLimit {
- warnings = append(warnings, "Your kernel does not support pids limit capabilities, pids limit discarded.")
- logrus.Warn("Your kernel does not support pids limit capabilities, pids limit discarded.")
+ warnings = append(warnings, "Your kernel does not support pids limit capabilities. Pids limit discarded.")
+ logrus.Warn("Your kernel does not support pids limit capabilities. Pids limit discarded.")
resources.PidsLimit = 0
}
@@ -400,28 +400,28 @@ func verifyContainerResources(resources *containertypes.Resources, sysInfo *sysi
return warnings, fmt.Errorf("Invalid QoS settings: %s does not support Maximum IO Bandwidth or Maximum IO IOps", runtime.GOOS)
}
if len(resources.BlkioWeightDevice) > 0 && !sysInfo.BlkioWeightDevice {
- warnings = append(warnings, "Your kernel does not support Block I/O weight_device.")
- logrus.Warn("Your kernel does not support Block I/O weight_device. Weight-device discarded.")
+ warnings = append(warnings, "Your kernel does not support Block I/O weight-device. Weight-device discarded.")
+ logrus.Warn("Your kernel does not support Block I/O weight-device. Weight-device discarded.")
resources.BlkioWeightDevice = []*pblkiodev.WeightDevice{}
}
if len(resources.BlkioDeviceReadBps) > 0 && !sysInfo.BlkioReadBpsDevice {
- warnings = append(warnings, "Your kernel does not support Block read limit in bytes per second.")
- logrus.Warn("Your kernel does not support Block I/O read limit in bytes per second. --device-read-bps discarded.")
+ warnings = append(warnings, "Your kernel does not support Block read limit in bytes per second. Device read bps discarded.")
+ logrus.Warn("Your kernel does not support Block I/O read limit in bytes per second. Device read bps discarded.")
resources.BlkioDeviceReadBps = []*pblkiodev.ThrottleDevice{}
}
if len(resources.BlkioDeviceWriteBps) > 0 && !sysInfo.BlkioWriteBpsDevice {
- warnings = append(warnings, "Your kernel does not support Block write limit in bytes per second.")
- logrus.Warn("Your kernel does not support Block I/O write limit in bytes per second. --device-write-bps discarded.")
+ warnings = append(warnings, "Your kernel does not support Block write limit in bytes per second. Device write bps discarded.")
+ logrus.Warn("Your kernel does not support Block I/O write limit in bytes per second. Device write bps discarded.")
resources.BlkioDeviceWriteBps = []*pblkiodev.ThrottleDevice{}
}
if len(resources.BlkioDeviceReadIOps) > 0 && !sysInfo.BlkioReadIOpsDevice {
- warnings = append(warnings, "Your kernel does not support Block read limit in IO per second.")
- logrus.Warn("Your kernel does not support Block I/O read limit in IO per second. -device-read-iops discarded.")
+ warnings = append(warnings, "Your kernel does not support Block read limit in IO per second. Device read iops discarded.")
+ logrus.Warn("Your kernel does not support Block I/O read limit in IO per second. Device read iops discarded.")
resources.BlkioDeviceReadIOps = []*pblkiodev.ThrottleDevice{}
}
if len(resources.BlkioDeviceWriteIOps) > 0 && !sysInfo.BlkioWriteIOpsDevice {
- warnings = append(warnings, "Your kernel does not support Block write limit in IO per second.")
- logrus.Warn("Your kernel does not support Block I/O write limit in IO per second. --device-write-iops discarded.")
+ warnings = append(warnings, "Your kernel does not support Block write limit in IO per second. Device write iops discarded.")
+ logrus.Warn("Your kernel does not support Block I/O write limit in IO per second. Device write iops discarded.")
resources.BlkioDeviceWriteIOps = []*pblkiodev.ThrottleDevice{}
}
@@ -481,7 +481,7 @@ func verifyPlatformContainerSettings(daemon *Daemon, hostConfig *containertypes.
warnings = append(warnings, w...)
if hostConfig.ShmSize < 0 {
- return warnings, fmt.Errorf("SHM size must be greater than 0")
+ return warnings, fmt.Errorf("SHM size can not be less than 0")
}
if hostConfig.OomScoreAdj < -1000 || hostConfig.OomScoreAdj > 1000 {
diff --git a/daemon/daemon_windows.go b/daemon/daemon_windows.go
index f6efe54784..4222c960b8 100644
--- a/daemon/daemon_windows.go
+++ b/daemon/daemon_windows.go
@@ -101,33 +101,33 @@ func verifyContainerResources(resources *containertypes.Resources, sysInfo *sysi
// TODO Windows: Add more validation of resource settings not supported on Windows
if resources.BlkioWeight > 0 {
- warnings = append(warnings, "Windows does not support Block I/O weight. Weight discarded.")
- logrus.Warn("Windows does not support Block I/O weight. --blkio-weight discarded.")
+ warnings = append(warnings, "Windows does not support Block I/O weight. Block I/O weight discarded.")
+ logrus.Warn("Windows does not support Block I/O weight. Block I/O weight discarded.")
resources.BlkioWeight = 0
}
if len(resources.BlkioWeightDevice) > 0 {
- warnings = append(warnings, "Windows does not support Block I/O weight_device.")
- logrus.Warn("Windows does not support Block I/O weight_device. --blkio-weight-device discarded.")
+ warnings = append(warnings, "Windows does not support Block I/O weight-device. Weight-device discarded.")
+ logrus.Warn("Windows does not support Block I/O weight-device. Weight-device discarded.")
resources.BlkioWeightDevice = []*pblkiodev.WeightDevice{}
}
if len(resources.BlkioDeviceReadBps) > 0 {
- warnings = append(warnings, "Windows does not support Block read limit in bytes per second.")
- logrus.Warn("Windows does not support Block I/O read limit in bytes per second. --device-read-bps discarded.")
+ warnings = append(warnings, "Windows does not support Block read limit in bytes per second. Device read bps discarded.")
+ logrus.Warn("Windows does not support Block I/O read limit in bytes per second. Device read bps discarded.")
resources.BlkioDeviceReadBps = []*pblkiodev.ThrottleDevice{}
}
if len(resources.BlkioDeviceWriteBps) > 0 {
- warnings = append(warnings, "Windows does not support Block write limit in bytes per second.")
- logrus.Warn("Windows does not support Block I/O write limit in bytes per second. --device-write-bps discarded.")
+ warnings = append(warnings, "Windows does not support Block write limit in bytes per second. Device write bps discarded.")
+ logrus.Warn("Windows does not support Block I/O write limit in bytes per second. Device write bps discarded.")
resources.BlkioDeviceWriteBps = []*pblkiodev.ThrottleDevice{}
}
if len(resources.BlkioDeviceReadIOps) > 0 {
- warnings = append(warnings, "Windows does not support Block read limit in IO per second.")
- logrus.Warn("Windows does not support Block I/O read limit in IO per second. -device-read-iops discarded.")
+ warnings = append(warnings, "Windows does not support Block read limit in IO per second. Device read iops discarded.")
+ logrus.Warn("Windows does not support Block I/O read limit in IO per second. Device read iops discarded.")
resources.BlkioDeviceReadIOps = []*pblkiodev.ThrottleDevice{}
}
if len(resources.BlkioDeviceWriteIOps) > 0 {
- warnings = append(warnings, "Windows does not support Block write limit in IO per second.")
- logrus.Warn("Windows does not support Block I/O write limit in IO per second. --device-write-iops discarded.")
+ warnings = append(warnings, "Windows does not support Block write limit in IO per second. Device write iops discarded.")
+ logrus.Warn("Windows does not support Block I/O write limit in IO per second. Device write iops discarded.")
resources.BlkioDeviceWriteIOps = []*pblkiodev.ThrottleDevice{}
}
return warnings, nil
diff --git a/integration-cli/docker_api_containers_test.go b/integration-cli/docker_api_containers_test.go
index ea1358cae0..99ea700955 100644
--- a/integration-cli/docker_api_containers_test.go
+++ b/integration-cli/docker_api_containers_test.go
@@ -1267,7 +1267,7 @@ func (s *DockerSuite) TestPostContainersCreateShmSizeNegative(c *check.C) {
status, body, err := sockRequest("POST", "/containers/create", config)
c.Assert(err, check.IsNil)
c.Assert(status, check.Equals, http.StatusInternalServerError)
- c.Assert(getErrorMessage(c, body), checker.Contains, "SHM size must be greater than 0")
+ c.Assert(getErrorMessage(c, body), checker.Contains, "SHM size can not be less than 0")
}
func (s *DockerSuite) TestPostContainersCreateShmSizeHostConfigOmitted(c *check.C) {
diff --git a/integration-cli/docker_cli_run_unix_test.go b/integration-cli/docker_cli_run_unix_test.go
index c1995b7ce1..9e379ab285 100644
--- a/integration-cli/docker_cli_run_unix_test.go
+++ b/integration-cli/docker_cli_run_unix_test.go
@@ -687,7 +687,7 @@ func (s *DockerSuite) TestRunWithMemoryReservationInvalid(c *check.C) {
testRequires(c, memoryReservationSupport)
out, _, err := dockerCmdWithError("run", "-m", "500M", "--memory-reservation", "800M", "busybox", "true")
c.Assert(err, check.NotNil)
- expected := "Minimum memory limit should be larger than memory reservation limit"
+ expected := "Minimum memory limit can not be less than memory reservation limit"
c.Assert(strings.TrimSpace(out), checker.Contains, expected, check.Commentf("run container should fail with invalid memory reservation"))
out, _, err = dockerCmdWithError("run", "--memory-reservation", "1k", "busybox", "true")