diff options
| author | Matthew Heon <mheon@redhat.com> | 2021-12-22 09:35:00 -0500 |
|---|---|---|
| committer | Matthew Heon <mheon@redhat.com> | 2022-01-17 13:56:07 -0500 |
| commit | 141de86862898a4b9e35c15f51031952c63c7114 (patch) | |
| tree | c5ad04e0c9ec12d98d09ecfc51b54e6b4b30f494 /test/upgrade | |
| parent | 8514ebd1827b12bae8b5d53d8f0e36244d1b3c3a (diff) | |
| download | podman-141de86862898a4b9e35c15f51031952c63c7114.tar.gz podman-141de86862898a4b9e35c15f51031952c63c7114.tar.bz2 podman-141de86862898a4b9e35c15f51031952c63c7114.zip | |
Revamp Libpod state strings for Docker compat
Improve our compatibility with Docker by better handling the
state strings that we print in `podman ps`. Docker capitalizes
all states in `ps` (we do not) - fix this in our PS code. Also,
stop normalizing ContainerStateConfigured to the "Created" state,
and instead make it always be Created, with the existing Created
state becoming Initialized.
I didn't rename the actual states because I'm somewhat reticent
to make such a large change a day before we leave for break. It's
somewhat confusing that ContainerStateConfigured now returns
Created, but internally and externally we're still consistent.
[NO NEW TESTS NEEDED] existing tests should catch anything that
broke.
I also consider this a breaking change. I will flag appropriately
on Github.
Fixes RHBZ#2010432 and RHBZ#2032561
Signed-off-by: Matthew Heon <mheon@redhat.com>
Diffstat (limited to 'test/upgrade')
| -rw-r--r-- | test/upgrade/test-upgrade.bats | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/test/upgrade/test-upgrade.bats b/test/upgrade/test-upgrade.bats index 5d2435ab9..58e4fe0a3 100644 --- a/test/upgrade/test-upgrade.bats +++ b/test/upgrade/test-upgrade.bats @@ -223,7 +223,7 @@ EOF @test "inspect - all container status" { tests=" running | running | 0 -created | configured | 0 +created | created | 0 done | exited | 0 failed | exited | 17 " |