kotovalexarian-likes-github/moby--moby
1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
Code Releases Activity

Add verify config to verifyContainerSettings

Browse source 
Signed-off-by: Lei Jitang <leijitang@huawei.com>
This commit is contained in:
Lei Jitang 2015-06-13 09:31:19 +08:00
parent 422fa7f0a6
commit b37832e353
4 changed files with 13 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
daemon/create.go
View file

@ -19,18 +19,11 @@ func (daemon *Daemon) ContainerCreate(name string, config *runconfig.Config, hos
return "", nil, fmt.Errorf("Config cannot be empty in order to create a container") return "", nil, fmt.Errorf("Config cannot be empty in order to create a container")
} }
warnings, err := daemon.verifyContainerSettings(hostConfig) warnings, err := daemon.verifyContainerSettings(hostConfig, config)
if err != nil { if err != nil {
return "", warnings, err return "", warnings, err
} }
// The check for a valid workdir path is made on the server rather than in the
// client. This is because we don't know the type of path (Linux or Windows)
// to validate on the client.
if config.WorkingDir != "" && !filepath.IsAbs(config.WorkingDir) {
return "", warnings, fmt.Errorf("The working directory '%s' is invalid. It needs to be an absolute path.", config.WorkingDir)
}
container, buildWarnings, err := daemon.Create(config, hostConfig, name) container, buildWarnings, err := daemon.Create(config, hostConfig, name)
if err != nil { if err != nil {
if daemon.Graph().IsNotExist(err, config.Image) { if daemon.Graph().IsNotExist(err, config.Image) {

11
daemon/daemon_unix.go
View file

@ -112,9 +112,18 @@ func checkKernel() error {
return nil return nil
} }
func (daemon *Daemon) verifyContainerSettings(hostConfig *runconfig.HostConfig) ([]string, error) { func (daemon *Daemon) verifyContainerSettings(hostConfig *runconfig.HostConfig, config *runconfig.Config) ([]string, error) {
var warnings []string var warnings []string
if config != nil {
// The check for a valid workdir path is made on the server rather than in the
// client. This is because we don't know the type of path (Linux or Windows)
// to validate on the client.
if config.WorkingDir != "" && !filepath.IsAbs(config.WorkingDir) {
return warnings, fmt.Errorf("The working directory '%s' is invalid. It needs to be an absolute path.", config.WorkingDir)
}
}
if hostConfig == nil { if hostConfig == nil {
return warnings, nil return warnings, nil
} }

2
daemon/daemon_windows.go
View file

@ -42,7 +42,7 @@ func checkKernel() error {
return nil return nil
} }
func (daemon *Daemon) verifyContainerSettings(hostConfig *runconfig.HostConfig) ([]string, error) { func (daemon *Daemon) verifyContainerSettings(hostConfig *runconfig.HostConfig, config *runconfig.HostConfig) ([]string, error) {
// TODO Windows. Verifications TBC // TODO Windows. Verifications TBC
return nil, nil return nil, nil
} }

2
daemon/start.go
View file

@ -20,7 +20,7 @@ func (daemon *Daemon) ContainerStart(name string, hostConfig *runconfig.HostConf
return fmt.Errorf("Container already started") return fmt.Errorf("Container already started")
} }
if _, err = daemon.verifyContainerSettings(hostConfig); err != nil { if _, err = daemon.verifyContainerSettings(hostConfig, nil); err != nil {
return err return err
} }