summaryrefslogtreecommitdiff
path: root/libpod/container_api.go
diff options
context:
space:
mode:
authorMatthew Heon <matthew.heon@gmail.com>2018-03-07 11:15:00 -0500
committerAtomic Bot <atomic-devel@projectatomic.io>2018-03-08 16:40:21 +0000
commitd23b9fd4ed96cbf03c5da3af6bc5837d9998a75a (patch)
tree6871bb77a274e8a17e847080e6cf11e03fb97be8 /libpod/container_api.go
parent04d56c9fe30235c9f99cf816fc6078fa5b2f1512 (diff)
downloadpodman-d23b9fd4ed96cbf03c5da3af6bc5837d9998a75a.tar.gz
podman-d23b9fd4ed96cbf03c5da3af6bc5837d9998a75a.tar.bz2
podman-d23b9fd4ed96cbf03c5da3af6bc5837d9998a75a.zip
Refactor saving OCI spec to disk into separate function
It will be needed for restarting containers Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #462 Approved by: baude
Diffstat (limited to 'libpod/container_api.go')
-rw-r--r--libpod/container_api.go31
1 files changed, 2 insertions, 29 deletions
diff --git a/libpod/container_api.go b/libpod/container_api.go
index 253428256..9020a2dde 100644
--- a/libpod/container_api.go
+++ b/libpod/container_api.go
@@ -1,11 +1,9 @@
package libpod
import (
- "encoding/json"
"io/ioutil"
"os"
gosignal "os/signal"
- "path/filepath"
"strconv"
"time"
@@ -63,22 +61,6 @@ func (c *Container) Init() (err error) {
}
}()
- // If the OCI spec already exists, we need to replace it
- // Cannot guarantee some things, e.g. network namespaces, have the same
- // paths
- jsonPath := filepath.Join(c.bundlePath(), "config.json")
- if _, err := os.Stat(jsonPath); err != nil {
- if !os.IsNotExist(err) {
- return errors.Wrapf(err, "error doing stat on container %s spec", c.ID())
- }
- // The spec does not exist, we're fine
- } else {
- // The spec exists, need to remove it
- if err := os.Remove(jsonPath); err != nil {
- return errors.Wrapf(err, "error replacing runtime spec for container %s", c.ID())
- }
- }
-
// Copy /etc/resolv.conf to the container's rundir
runDirResolv, err := c.generateResolvConf()
if err != nil {
@@ -101,21 +83,12 @@ func (c *Container) Init() (err error) {
if err != nil {
return err
}
- c.runningSpec = spec
// Save the OCI spec to disk
- fileJSON, err := json.Marshal(c.runningSpec)
- if err != nil {
- return errors.Wrapf(err, "error exporting runtime spec for container %s to JSON", c.ID())
- }
- if err := ioutil.WriteFile(jsonPath, fileJSON, 0644); err != nil {
- return errors.Wrapf(err, "error writing runtime spec JSON to file for container %s", c.ID())
+ if err := c.saveSpec(spec); err != nil {
+ return err
}
- logrus.Debugf("Created OCI spec for container %s at %s", c.ID(), jsonPath)
-
- c.state.ConfigPath = jsonPath
-
// With the spec complete, do an OCI create
if err := c.runtime.ociRuntime.createContainer(c, c.config.CgroupParent); err != nil {
return err