[NO TESTS NEEDED] Ignore removed containers

[jwhonce]

Ignore condition when containers are removed while listing them for
ps output.

No tests added at this time as they would create a race condition for
CI.

See #11810 for reproducer.

Fixed up some godoc errors.

Fixes #11810

Signed-off-by: Jhon Honce jhonce@redhat.com

What this PR does / why we need it:

How to verify it

Which issue(s) this PR fixes:

Special notes for your reviewer:

[Luap99]

This should be handled somewhere in the backend. This fix could loop forever and does not cover api users.

[jwhonce]

This should be handled somewhere in the backend. This fix could loop forever and does not cover api users.

Should break out once database state is consistent, but you are right it does not cover the API just the CLI. I'll take a look to see if we have a single path to fix it all.

[Luap99]

This should be handled somewhere in the backend. This fix could loop forever and does not cover api users.

Should break out once database state is consistent, but you are right it does not cover the API just the CLI. I'll take a look to see if we have a single path to fix it all.

I think here would be good:

podman/pkg/ps/ps.go

Line 65 in c6a896b

listCon, err := ListContainerBatch(runtime, con, options)

[rhatdan]

LGTM
@containers/podman-maintainers PTAL

[Luap99]

LGTM

[Luap99]

/lgtm

[openshift-ci]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jwhonce, Luap99

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [Luap99,jwhonce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment