diff options
author | Ed Santiago <santiago@redhat.com> | 2022-07-06 14:38:48 -0600 |
---|---|---|
committer | Ed Santiago <santiago@redhat.com> | 2022-07-07 14:03:42 -0600 |
commit | 1161e8bd548d23ea9b8cf5ba12d89ded3f86a425 (patch) | |
tree | b16c7f871cbfb7695589706adf55277b2c3d8888 /cmd/podman/common | |
parent | 700f1faf6e5449e13970fc17311b09e4ca9ac2c3 (diff) | |
download | podman-1161e8bd548d23ea9b8cf5ba12d89ded3f86a425.tar.gz podman-1161e8bd548d23ea9b8cf5ba12d89ded3f86a425.tar.bz2 podman-1161e8bd548d23ea9b8cf5ba12d89ded3f86a425.zip |
manifest_test: safer registry setup and teardown
manifest_test:authenticated_push() is the final test left to
fix before merging #14397. The reason it's failing _seems_ to be
that podman is running with a mix of netavark and CNI, and
that _seems_ to be because this test invokes hack/podman-registry
which invokes plain podman without whatever options used in e2e.
Starting a registry directly from the test is insane: there is
no reusable code for doing that (see login_logout_test.go and
push_test.go. Yeesh.)
Solution: set $PODMAN, by inspecting the podmanTest object
which includes both a path and a list of options. podman-registry
will invoke that. (It will also override --root and --runroot.
This is the desired behavior).
Also: add cleanup. If auth-push test fails, stop the registry.
Also: add a sanity check to podman-registry script, have it
wait for the registry port to activate. Die if it doesn't.
That could've saved us a nice bit of debugging time.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'cmd/podman/common')
0 files changed, 0 insertions, 0 deletions