diff options
author | Matthew Heon <mheon@redhat.com> | 2021-07-08 14:22:33 -0400 |
---|---|---|
committer | Matthew Heon <mheon@redhat.com> | 2021-07-08 14:22:33 -0400 |
commit | e5fcffc551b42936c635d13658e7d0a9e95d3a16 (patch) | |
tree | 613f87f31de469eea788e40f2e1b22a90aa3a8e8 /pkg/systemd | |
parent | 31c3b952e5da7eb11a32464a0cd74856265efec5 (diff) | |
download | podman-e5fcffc551b42936c635d13658e7d0a9e95d3a16.tar.gz podman-e5fcffc551b42936c635d13658e7d0a9e95d3a16.tar.bz2 podman-e5fcffc551b42936c635d13658e7d0a9e95d3a16.zip |
Remove GetStore function from Libpod
We should not be exposing the store outside of Libpod. We want to
encapsulate it as an internal implementation detail - there's no
reason functions outside of Libpod should directly be
manipulating container storage. Convert the last use to invoke a
method on Libpod instead, and remove the function.
[NO TESTS NEEDED] as this is just a refactor.
Signed-off-by: Matthew Heon <mheon@redhat.com>
Diffstat (limited to 'pkg/systemd')
-rw-r--r-- | pkg/systemd/generate/containers.go | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/pkg/systemd/generate/containers.go b/pkg/systemd/generate/containers.go index 0e6e1b4df..083520316 100644 --- a/pkg/systemd/generate/containers.go +++ b/pkg/systemd/generate/containers.go @@ -160,16 +160,11 @@ func generateContainerInfo(ctr *libpod.Container, options entities.GenerateSyste nameOrID, serviceName := containerServiceName(ctr, options) - store := ctr.Runtime().GetStore() - if store == nil { - return nil, errors.Errorf("could not determine storage store for container") - } - var runRoot string if options.New { runRoot = "%t/containers" } else { - runRoot = store.RunRoot() + runRoot = ctr.Runtime().RunRoot() if runRoot == "" { return nil, errors.Errorf("could not lookup container's runroot: got empty string") } |