diff options
| author | Daniel J Walsh <dwalsh@redhat.com> | 2018-03-17 20:08:27 -0400 |
|---|---|---|
| committer | Atomic Bot <atomic-devel@projectatomic.io> | 2018-03-29 01:55:20 +0000 |
| commit | c54816dfc39b00824b9d4902ed2e533b3c6c07a7 (patch) | |
| tree | b810d6625d3f095a7d7e8561bde651062ab9f44d /libpod/runtime_ctr.go | |
| parent | f936b745b66de3cbbdf924a26bb35766afe5acba (diff) | |
| download | podman-c54816dfc39b00824b9d4902ed2e533b3c6c07a7.tar.gz podman-c54816dfc39b00824b9d4902ed2e533b3c6c07a7.tar.bz2 podman-c54816dfc39b00824b9d4902ed2e533b3c6c07a7.zip | |
Check for duplicate names when generating new container and pod names.
This fixes the situation where we fail to create a container when a name already exists.
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
Closes: #517
Approved by: baude
Diffstat (limited to 'libpod/runtime_ctr.go')
| -rw-r--r-- | libpod/runtime_ctr.go | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/libpod/runtime_ctr.go b/libpod/runtime_ctr.go index 51c2001d0..3b4710d96 100644 --- a/libpod/runtime_ctr.go +++ b/libpod/runtime_ctr.go @@ -50,6 +50,15 @@ func (r *Runtime) NewContainer(rSpec *spec.Spec, options ...CtrCreateOption) (c ctr.state.State = ContainerStateConfigured ctr.runtime = r + if ctr.config.Name == "" { + name, err := r.generateName() + if err != nil { + return nil, err + } + + ctr.config.Name = name + } + // Set up storage for the container if err := ctr.setupStorage(); err != nil { return nil, errors.Wrapf(err, "error configuring storage for container") |