diff options
| author | Urvashi Mohnani <umohnani@redhat.com> | 2022-05-25 15:34:59 -0400 |
|---|---|---|
| committer | Urvashi Mohnani <umohnani@redhat.com> | 2022-07-05 10:32:49 -0400 |
| commit | 65d511c6d859f250b6e45107315436e73f125c3a (patch) | |
| tree | 346b5e2aff8ae3350b2457ad4f13fc203620d3a1 /pkg/domain/infra/abi/pods.go | |
| parent | 9539a89ee77682ed3f4d1d0be3b950523e2f2439 (diff) | |
| download | podman-65d511c6d859f250b6e45107315436e73f125c3a.tar.gz podman-65d511c6d859f250b6e45107315436e73f125c3a.tar.bz2 podman-65d511c6d859f250b6e45107315436e73f125c3a.zip | |
Fix podman pod unpaue TODO
Update the podman pod unpause to only show the paused
containers with autocomplete.
Fix a typo in the help command.
Update the unpause function to only attempt an unpause
on pasued pods instead of all the pods.
Update the tests accordingly.
Signed-off-by: Urvashi Mohnani <umohnani@redhat.com>
Diffstat (limited to 'pkg/domain/infra/abi/pods.go')
| -rw-r--r-- | pkg/domain/infra/abi/pods.go | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/pkg/domain/infra/abi/pods.go b/pkg/domain/infra/abi/pods.go index 1dca8c580..d287ebe08 100644 --- a/pkg/domain/infra/abi/pods.go +++ b/pkg/domain/infra/abi/pods.go @@ -158,6 +158,14 @@ func (ic *ContainerEngine) PodUnpause(ctx context.Context, namesOrIds []string, return nil, err } for _, p := range pods { + status, err := p.GetPodStatus() + if err != nil { + return nil, err + } + // If the pod is not paused or degraded, there is no need to attempt an unpause on it + if status != define.PodStatePaused && status != define.PodStateDegraded { + continue + } report := entities.PodUnpauseReport{Id: p.ID()} errs, err := p.Unpause(ctx) if err != nil && errors.Cause(err) != define.ErrPodPartialFail { |