spiegel_podman/test/system
Ed Santiago f5b3dc976c Tests: Fix common flakes, and improve apiv2 test log
- apiv2 - the 'ten /info requests' test is flaking often,
  taking ~8 seconds (our limit is 7, up from 5 a few weeks
  ago). Brent suggested that the first /info call might be
  expensive, because it needs to access storage. So, let's
  prime it by running one /info outside the timing loop.
  And, because even that continues to fail, bump it up
  to 10 seconds and file #8076 to track the slowdown.

- toolbox test - WaitForReady() has timed out, even on one
  occasion causing a run failure because it failed 3 times.
  Solution: bump up timeout from 2s to 5s. Not really great,
  but CI systems are underpowered, and it's not unreasonable
  that 2s might be too low.

- sdnotify test - add a 'podman wait' between stop & rm.
  This may prevent a "cannot rm container as it is running"
  race condition.

While working on this, Brent and I noticed a few ways that
test-apiv2 logging can be improved:

- test name: when request is POST, display the jsonified
  parameters, not the original input ones. This should
  make it much easier to reproduce failures.

- use curl's "--write-out" option to capture http code,
  content type, and request time. We were getting the
  first two via grep from logged headers; this is cleaner.
  And there was no other way to get timing. We now include
  the timing as X-Response-Time in the log file.

- abort on *any* curl error, not just 7 (cannot connect).
  Any error at all from curl is bad news.

Signed-off-by: Ed Santiago <santiago@redhat.com>
2020-10-20 11:32:49 -06:00
..
000-TEMPLATE BATS: new tests, and improvements to existing ones 2019-03-18 15:21:52 -06:00
001-basic.bats podman-remote does not support most of the global flags 2020-10-02 06:32:17 -04:00
005-info.bats system tests: helpers: safer parse_table 2020-09-23 16:38:05 -06:00
010-images.bats system tests: cleanup, and add more tests 2020-10-07 10:43:02 -06:00
015-help.bats Usability: prevent "-l" with arguments 2020-09-14 15:49:28 -06:00
020-tag.bats system test: untag all test 2020-10-01 18:34:08 +02:00
030-run.bats Add a system test to verify --runtime is preserved 2020-10-20 09:24:32 -04:00
035-logs.bats Fix Podman logs reading journald 2020-10-02 10:05:19 -04:00
040-ps.bats Implement review feedback 2019-03-07 14:09:00 -07:00
050-stop.bats Reenable remote system tests 2020-08-03 09:36:36 -06:00
055-rm.bats System tests: remove some misleading 'run's 2020-10-19 08:43:56 -06:00
060-mount.bats Merge pull request #7949 from edsantiago/bats 2020-10-12 06:13:14 -04:00
065-cp.bats refactor info 2020-04-06 12:45:42 -05:00
070-build.bats System test additions 2020-10-14 15:32:02 -06:00
075-exec.bats capabilities: always set ambient and inheritable 2020-09-30 15:14:07 +02:00
080-pause.bats System tests: reenable some skipped tests 2020-09-28 13:59:42 -06:00
110-history.bats Migrate away from docker.io 2020-09-08 06:06:06 -06:00
120-load.bats podman save: fix redirect of multi-images 2020-09-22 17:04:38 +02:00
130-kill.bats system tests: cleanup, and add more tests 2020-10-07 10:43:02 -06:00
140-diff.bats System tests: remove some misleading 'run's 2020-10-19 08:43:56 -06:00
150-login.bats Migrate away from docker.io 2020-09-08 06:06:06 -06:00
160-volumes.bats System test additions 2020-10-14 15:32:02 -06:00
200-pod.bats System test additions 2020-10-14 15:32:02 -06:00
220-healthcheck.bats Reenable remote system tests 2020-08-03 09:36:36 -06:00
250-systemd.bats run/create: record raw image 2020-09-15 13:33:22 +02:00
260-sdnotify.bats Tests: Fix common flakes, and improve apiv2 test log 2020-10-20 11:32:49 -06:00
300-cli-parsing.bats Handle podman-remote run --rm 2020-08-04 20:26:34 -04:00
320-system-df.bats fix podman system df format error 2020-07-31 01:40:42 +08:00
400-unprivileged-access.bats Use bash binary from env instead of /bin/bash for scripts 2020-08-17 10:42:23 +02:00
410-selinux.bats system tests: cleanup, and add more tests 2020-10-07 10:43:02 -06:00
420-cgroups.bats System test additions 2020-10-14 15:32:02 -06:00
500-networking.bats System test additions 2020-10-14 15:32:02 -06:00
build-testimage System tests: add podman run --tz 2020-09-29 12:23:58 -06:00
helpers.bash Add a system test to verify --runtime is preserved 2020-10-20 09:24:32 -04:00
helpers.t system tests: helpers: safer parse_table 2020-09-23 16:38:05 -06:00
README.md implement 'make remotesystem' 2019-08-08 11:44:55 -06:00
TODO.md Switch all references to github.com/containers/libpod -> podman 2020-07-28 08:23:45 -04:00

Quick overview of podman system tests. The idea is to use BATS, but with a framework for making it easy to add new tests and to debug failures.

Quick Start

Look at 030-run.bats for a simple but packed example. This introduces the basic set of helper functions:

  • setup (implicit) - resets container storage so there's one and only one (standard) image, and no running containers.

  • parse_table - you can define tables of inputs and expected results, then read those in a while loop. This makes it easy to add new tests. Because bash is not a programming language, the caller of parse_table sometimes needs to massage the returned values; 015-run.bats offers examples of how to deal with the more typical such issues.

  • run_podman - runs command defined in $PODMAN (default: 'podman' but could also be './bin/podman' or 'podman-remote'), with a timeout. Checks its exit status.

  • is - compare actual vs expected output. Emits a useful diagnostic on failure.

  • die - output a properly-formatted message to stderr, and fail test

  • skip_if_rootless - if rootless, skip this test with a helpful message.

  • skip_if_remote - like the above, but skip if testing podman-remote

  • random_string - returns a pseudorandom alphanumeric string

Test files are of the form NNN-name.bats where NNN is a three-digit number. Please preserve this convention, it simplifies viewing the directory and understanding test order. In particular, 00x tests should be reserved for a first-pass fail-fast subset of tests:

bats test/system/00*.bats || exit 1
bats test/system

...the goal being to provide quick feedback on catastrophic failures without having to wait for the entire test suite.

Running tests

To run the tests locally in your sandbox, you can use one of these methods:

  • make;PODMAN=./bin/podman bats ./test/system/070-build.bats # runs just the specified test
  • make;PODMAN=./bin/podman bats ./test/system # runs all

To test as root:

  • $ PODMAN=./bin/podman sudo --preserve-env=PODMAN bats test/system

Analyzing test failures

The top priority for this scheme is to make it easy to diagnose what went wrong. To that end, podman_run always logs all invoked commands, their output and exit codes. In a normal run you will never see this, but BATS will display it on failure. The goal here is to give you everything you need to diagnose without having to rerun tests.

The is comparison function is designed to emit useful diagnostics, in particular, the actual and expected strings. Please do not use the horrible BATS standard of [ x = y ]; that's nearly useless for tracking down failures.

If the above are not enough to help you track down a failure:

Debugging tests

Some functions have dprint statements. To see the output of these, set PODMAN_TEST_DEBUG="funcname" where funcname is the name of the function or perhaps just a substring.

Requirements

The jq tool is needed for parsing JSON output.

Further Details

TBD. For now, look in helpers.bash; each helper function has (what are intended to be) helpful header comments. For even more examples, see and/or run helpers.t; that's a regression test and provides a thorough set of examples of how the helpers work.