diff options
author | Matthew Heon <matthew.heon@gmail.com> | 2017-11-21 13:51:51 -0500 |
---|---|---|
committer | Atomic Bot <atomic-devel@projectatomic.io> | 2017-11-21 20:09:09 +0000 |
commit | 152882bf19e1c94110187264ed37d25976bf1e73 (patch) | |
tree | 05714e76fa0e96c24c0b8b36f6effd271bc260f4 | |
parent | 8e76ebcf6e8925d5fa6a8c9ab517d665b44c7b63 (diff) | |
download | podman-152882bf19e1c94110187264ed37d25976bf1e73.tar.gz podman-152882bf19e1c94110187264ed37d25976bf1e73.tar.bz2 podman-152882bf19e1c94110187264ed37d25976bf1e73.zip |
Add ability to retrieve container's log path to API
Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
Closes: #56
Approved by: rhatdan
-rw-r--r-- | libpod/container.go | 13 | ||||
-rw-r--r-- | libpod/oci.go | 2 |
2 files changed, 14 insertions, 1 deletions
diff --git a/libpod/container.go b/libpod/container.go index 710ae866b..f167adc35 100644 --- a/libpod/container.go +++ b/libpod/container.go @@ -150,6 +150,14 @@ func (c *Container) Labels() map[string]string { return labels } +// LogPath returns the path to the container's log file +// This file will only be present after Init() is called to create the container +// in runc +func (c *Container) LogPath() string { + // TODO store this in state and allow overriding + return c.logPath() +} + // State returns the current state of the container func (c *Container) State() (ContainerState, error) { c.lock.Lock() @@ -181,6 +189,11 @@ func (c *Container) bundlePath() string { return c.state.RunDir } +// The path to the container's logs file +func (c *Container) logPath() string { + return filepath.Join(c.config.StaticDir, "ctr.log") +} + // Retrieves the path of the container's attach socket func (c *Container) attachSocketPath() string { return filepath.Join(c.runtime.ociRuntime.socketsDir, c.ID(), "attach") diff --git a/libpod/oci.go b/libpod/oci.go index 3ad3fdd8e..ddbd2f606 100644 --- a/libpod/oci.go +++ b/libpod/oci.go @@ -139,7 +139,7 @@ func (r *OCIRuntime) createContainer(ctr *Container, cgroupParent string) error args = append(args, "-p", filepath.Join(ctr.state.RunDir, "pidfile")) // TODO container log location should be configurable // The default also likely shouldn't be this - args = append(args, "-l", filepath.Join(ctr.config.StaticDir, "ctr.log")) + args = append(args, "-l", ctr.logPath()) args = append(args, "--exit-dir", r.exitsDir) args = append(args, "--socket-dir-path", r.socketsDir) if ctr.config.Spec.Process.Terminal { |