On Thu, Aug 29, 2019 at 07:02:44PM +0100, Daniel P. Berrangé wrote:
The OOM handling requires special build time options which we never
enable in our CI. Even once enabled the tests are incredibly slow and
typically require manual inspection of the results to weed out false
positives.
Since there was previous agreement to switch to abort on OOM in libvirt
code, there's no point continuing to keep the unused OOM testing code.
Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com>
---
configure.ac | 17 ---
docs/docs.html.in | 3 -
docs/internals/oomtesting.html.in | 213 ------------------------------
src/libvirt_private.syms | 4 -
src/util/viralloc.c | 111 ----------------
src/util/viralloc.h | 5 -
tests/Makefile.am | 1 -
tests/oomtrace.pl | 41 ------
tests/qemuxml2argvtest.c | 12 +-
tests/testutils.c | 189 +-------------------------
tests/testutils.h | 2 -
tests/virfirewalltest.c | 12 --
12 files changed, 6 insertions(+), 604 deletions(-)
delete mode 100644 docs/internals/oomtesting.html.in
delete mode 100755 tests/oomtrace.pl
Reviewed-by: Ján Tomko <jtomko(a)redhat.com>
Jano