b971b8
From 4b9cb933c1501309a858130f5465313699840ec1 Mon Sep 17 00:00:00 2001
b971b8
Message-Id: <4b9cb933c1501309a858130f5465313699840ec1@dist-git>
b971b8
From: Peter Krempa <pkrempa@redhat.com>
b971b8
Date: Tue, 23 Jun 2020 12:23:55 +0200
b971b8
Subject: [PATCH] qemublocktest: Re-introduce testing of checkpoint deletion
b971b8
MIME-Version: 1.0
b971b8
Content-Type: text/plain; charset=UTF-8
b971b8
Content-Transfer-Encoding: 8bit
b971b8
b971b8
Exercise the now arguably simpler checkpoint deletion code on the
b971b8
'basic', 'snapshots', and 'synthetic' test data sets.
b971b8
b971b8
Signed-off-by: Peter Krempa <pkrempa@redhat.com>
b971b8
Reviewed-by: Eric Blake <eblake@redhat.com>
b971b8
(cherry picked from commit 2c5a3da365d9a33ff23458ca2a914f9d35ebd510)
b971b8
https://bugzilla.redhat.com/show_bug.cgi?id=1804593
b971b8
Message-Id: <8df51bc8ee6e86fe36d971670ecdc89ecc989532.1592906423.git.pkrempa@redhat.com>
b971b8
Reviewed-by: Ján Tomko <jtomko@redhat.com>
b971b8
---
b971b8
 tests/qemublocktest.c                         | 18 ++++++++
b971b8
 .../checkpointdelete/basic-current-out.json   |  9 ++++
b971b8
 .../basic-intermediate1-out.json              |  9 ++++
b971b8
 .../basic-intermediate2-out.json              |  9 ++++
b971b8
 .../basic-intermediate3-out.json              |  9 ++++
b971b8
 .../checkpointdelete/basic-noparent-out.json  |  9 ++++
b971b8
 .../snapshots-current-out.json                |  9 ++++
b971b8
 .../snapshots-intermediate1-out.json          | 26 ++++++++++++
b971b8
 .../snapshots-intermediate2-out.json          | 26 ++++++++++++
b971b8
 .../snapshots-intermediate3-out.json          | 18 ++++++++
b971b8
 .../snapshots-noparent-out.json               | 42 +++++++++++++++++++
b971b8
 .../synthetic-current-out.json                |  9 ++++
b971b8
 .../synthetic-intermediate1-out.json          | 11 +++++
b971b8
 .../synthetic-intermediate2-out.json          | 11 +++++
b971b8
 .../synthetic-intermediate3-out.json          | 19 +++++++++
b971b8
 .../synthetic-noparent-out.json               | 11 +++++
b971b8
 16 files changed, 245 insertions(+)
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/basic-current-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/basic-intermediate1-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/basic-intermediate2-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/basic-intermediate3-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/basic-noparent-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/snapshots-current-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/snapshots-intermediate1-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/snapshots-intermediate2-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/snapshots-intermediate3-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/snapshots-noparent-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/synthetic-current-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/synthetic-intermediate1-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/synthetic-intermediate2-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/synthetic-intermediate3-out.json
b971b8
 create mode 100644 tests/qemublocktestdata/checkpointdelete/synthetic-noparent-out.json
b971b8
b971b8
diff --git a/tests/qemublocktest.c b/tests/qemublocktest.c
b971b8
index 0ec964c2c6..7c8f67dd6b 100644
b971b8
--- a/tests/qemublocktest.c
b971b8
+++ b/tests/qemublocktest.c
b971b8
@@ -1329,6 +1329,24 @@ mymain(void)
b971b8
 
b971b8
     TEST_CHECKPOINT_DELETE("empty", "a", "empty");
b971b8
 
b971b8
+    TEST_CHECKPOINT_DELETE("basic-noparent", "a", "basic");
b971b8
+    TEST_CHECKPOINT_DELETE("basic-intermediate1", "b", "basic");
b971b8
+    TEST_CHECKPOINT_DELETE("basic-intermediate2", "c", "basic");
b971b8
+    TEST_CHECKPOINT_DELETE("basic-intermediate3", "d", "basic");
b971b8
+    TEST_CHECKPOINT_DELETE("basic-current", "current", "basic");
b971b8
+
b971b8
+    TEST_CHECKPOINT_DELETE("snapshots-noparent", "a", "snapshots");
b971b8
+    TEST_CHECKPOINT_DELETE("snapshots-intermediate1", "b", "snapshots");
b971b8
+    TEST_CHECKPOINT_DELETE("snapshots-intermediate2", "c", "snapshots");
b971b8
+    TEST_CHECKPOINT_DELETE("snapshots-intermediate3", "d", "snapshots");
b971b8
+    TEST_CHECKPOINT_DELETE("snapshots-current", "current", "snapshots");
b971b8
+
b971b8
+    TEST_CHECKPOINT_DELETE("synthetic-noparent", "a", "synthetic");
b971b8
+    TEST_CHECKPOINT_DELETE("synthetic-intermediate1", "b", "synthetic");
b971b8
+    TEST_CHECKPOINT_DELETE("synthetic-intermediate2", "c", "synthetic");
b971b8
+    TEST_CHECKPOINT_DELETE("synthetic-intermediate3", "d", "synthetic");
b971b8
+    TEST_CHECKPOINT_DELETE("synthetic-current", "current", "synthetic");
b971b8
+
b971b8
 #define TEST_BITMAP_VALIDATE(testname, bitmap, rc) \
b971b8
     do { \
b971b8
         blockbitmapvalidatedata.name = testname; \
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/basic-current-out.json b/tests/qemublocktestdata/checkpointdelete/basic-current-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..6ed1b63b66
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/basic-current-out.json
b971b8
@@ -0,0 +1,9 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "current"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/basic-intermediate1-out.json b/tests/qemublocktestdata/checkpointdelete/basic-intermediate1-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..e1dd4920cd
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/basic-intermediate1-out.json
b971b8
@@ -0,0 +1,9 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "b"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/basic-intermediate2-out.json b/tests/qemublocktestdata/checkpointdelete/basic-intermediate2-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..5638532d35
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/basic-intermediate2-out.json
b971b8
@@ -0,0 +1,9 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "c"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/basic-intermediate3-out.json b/tests/qemublocktestdata/checkpointdelete/basic-intermediate3-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..8c56b0e689
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/basic-intermediate3-out.json
b971b8
@@ -0,0 +1,9 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "d"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/basic-noparent-out.json b/tests/qemublocktestdata/checkpointdelete/basic-noparent-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..e87382fdb4
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/basic-noparent-out.json
b971b8
@@ -0,0 +1,9 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "a"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/snapshots-current-out.json b/tests/qemublocktestdata/checkpointdelete/snapshots-current-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..6ed1b63b66
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/snapshots-current-out.json
b971b8
@@ -0,0 +1,9 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "current"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate1-out.json b/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate1-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..1cde09131e
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate1-out.json
b971b8
@@ -0,0 +1,26 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "b"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-2-format",
b971b8
+      "name": "b"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-3-format",
b971b8
+      "name": "b"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-3-format
b971b8
+libvirt-2-format
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate2-out.json b/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate2-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..6fcd7ee0d1
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate2-out.json
b971b8
@@ -0,0 +1,26 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "c"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-2-format",
b971b8
+      "name": "c"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-3-format",
b971b8
+      "name": "c"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-3-format
b971b8
+libvirt-2-format
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate3-out.json b/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate3-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..2977dc3042
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/snapshots-intermediate3-out.json
b971b8
@@ -0,0 +1,18 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "d"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-2-format",
b971b8
+      "name": "d"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-2-format
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/snapshots-noparent-out.json b/tests/qemublocktestdata/checkpointdelete/snapshots-noparent-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..e4a6a413a3
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/snapshots-noparent-out.json
b971b8
@@ -0,0 +1,42 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "a"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-2-format",
b971b8
+      "name": "a"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-3-format",
b971b8
+      "name": "a"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-4-format",
b971b8
+      "name": "a"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-5-format",
b971b8
+      "name": "a"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-5-format
b971b8
+libvirt-4-format
b971b8
+libvirt-3-format
b971b8
+libvirt-2-format
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/synthetic-current-out.json b/tests/qemublocktestdata/checkpointdelete/synthetic-current-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..6ed1b63b66
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/synthetic-current-out.json
b971b8
@@ -0,0 +1,9 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-1-format",
b971b8
+      "name": "current"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate1-out.json b/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate1-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..6eb9716aaa
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate1-out.json
b971b8
@@ -0,0 +1,11 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-3-format",
b971b8
+      "name": "b"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-3-format
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate2-out.json b/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate2-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..92b849cfc9
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate2-out.json
b971b8
@@ -0,0 +1,11 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-3-format",
b971b8
+      "name": "c"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-3-format
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate3-out.json b/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate3-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..466df52007
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/synthetic-intermediate3-out.json
b971b8
@@ -0,0 +1,19 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-2-format",
b971b8
+      "name": "d"
b971b8
+    }
b971b8
+  },
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-3-format",
b971b8
+      "name": "d"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-3-format
b971b8
+libvirt-2-format
b971b8
diff --git a/tests/qemublocktestdata/checkpointdelete/synthetic-noparent-out.json b/tests/qemublocktestdata/checkpointdelete/synthetic-noparent-out.json
b971b8
new file mode 100644
b971b8
index 0000000000..0479815f36
b971b8
--- /dev/null
b971b8
+++ b/tests/qemublocktestdata/checkpointdelete/synthetic-noparent-out.json
b971b8
@@ -0,0 +1,11 @@
b971b8
+[
b971b8
+  {
b971b8
+    "type": "block-dirty-bitmap-remove",
b971b8
+    "data": {
b971b8
+      "node": "libvirt-5-format",
b971b8
+      "name": "a"
b971b8
+    }
b971b8
+  }
b971b8
+]
b971b8
+reopen nodes:
b971b8
+libvirt-5-format
b971b8
-- 
b971b8
2.27.0
b971b8