diff options
author | Ed Santiago <santiago@redhat.com> | 2021-04-14 10:43:19 -0600 |
---|---|---|
committer | Ed Santiago <santiago@redhat.com> | 2021-04-14 12:32:20 -0600 |
commit | 1cf2b3eb28959a8e2c9ccd4b0d40186c6731104d (patch) | |
tree | e1cc3b667d1a50fe9d75fb4b55439da8b173f5d2 /vendor | |
parent | 4ee203dde7c9e8de2484f00c7f251c0f32ed0ad2 (diff) | |
download | podman-1cf2b3eb28959a8e2c9ccd4b0d40186c6731104d.tar.gz podman-1cf2b3eb28959a8e2c9ccd4b0d40186c6731104d.tar.bz2 podman-1cf2b3eb28959a8e2c9ccd4b0d40186c6731104d.zip |
compose test: ongoing efforts to diagnose flakes
Yay, we got a failure with the new code (#10017). It shows
one ECONNRESET followed by a lot of ECONNREFUSED over an 8-second
period (actually 15s because of the second curl retry).
My hunch: the container itself is dying. No amount of retrying
will get anything to work. So, instead of the curl retry, if
curl fails, run 'docker-compose logs', 'podman ps', and 'ss -tulpn'
and hope that one/more of those tells us something useful when
the test flakes again.
Also: DUH! Bitten by one of the most common bash pitfalls.
Checking exit status after 'local' will always be zero.
Split the declaration and the action into separate lines.
Also: if curl fails, return immediately. There's no point in
running the string output comparison.
Also: in _show_ok(), don't emit "actual/expect" messages
if both strings are empty.
Signed-off-by: Ed Santiago <santiago@redhat.com>
Diffstat (limited to 'vendor')
0 files changed, 0 insertions, 0 deletions