rados/test.sh fails in the nightly run

Make test more robust by using my_snaps vector for snap IDs

Signed-off-by: David Zafman <david.zafman@inktank.com>

diff --git a/src/test/librados/snapshots.cc b/src/test/librados/snapshots.cc
index 1aec45456baba94e48e3366d19ba14e13a1538b5..b084aa1f82f5c132c452f020c43395e134757e9c 100644 (file)
--- a/src/test/librados/snapshots.cc
+++ b/src/test/librados/snapshots.cc
@@ -337,6 +337,7 @@ TEST(LibRadosSnapshots, SelfManagedSnapRollbackPP) {
   ASSERT_EQ(1u, ss.clones.size());
   ASSERT_EQ(head, ss.clones[0].cloneid);
   ASSERT_EQ(1u, ss.clones[0].snaps.size());    //this could go away in the future
+  ASSERT_EQ(my_snaps[0], ss.clones[0].snaps[0]);
   ASSERT_EQ(0u, ss.clones[0].overlap.size());
   ASSERT_EQ(384u, ss.clones[0].size);
 
@@ -358,8 +359,8 @@ TEST(LibRadosSnapshots, SelfManagedSnapRollbackPP) {
   ASSERT_EQ(2u, ss.clones.size());
   ASSERT_EQ(3u, ss.clones[0].cloneid);
   ASSERT_EQ(2u, ss.clones[0].snaps.size());
-  ASSERT_EQ(2u, ss.clones[0].snaps[0]);        //this could go away in the future
-  ASSERT_EQ(3u, ss.clones[0].snaps[1]);
+  ASSERT_EQ(my_snaps[0], ss.clones[0].snaps[0]);       //this could go away in the future
+  ASSERT_EQ(my_snaps[1], ss.clones[0].snaps[1]);
   ASSERT_EQ(2u, ss.clones[0].overlap.size());
   ASSERT_EQ(0u, ss.clones[0].overlap[0].first);
   ASSERT_EQ(128u, ss.clones[0].overlap[0].second);
@@ -419,7 +420,7 @@ TEST(LibRadosSnapshots, SelfManagedSnapOverlapPP) {
   ASSERT_EQ(1u, ss.clones.size());
   ASSERT_EQ(head, ss.clones[0].cloneid);
   ASSERT_EQ(1u, ss.clones[0].snaps.size());
-  ASSERT_EQ(2u, ss.clones[0].snaps[0]);        //this could go away in the future
+  ASSERT_EQ(my_snaps[0], ss.clones[0].snaps[0]);       //this could go away in the future
   ASSERT_EQ(0u, ss.clones[0].overlap.size());
   ASSERT_EQ(1152u, ss.clones[0].size);
 
@@ -442,8 +443,8 @@ TEST(LibRadosSnapshots, SelfManagedSnapOverlapPP) {
   ASSERT_EQ(2u, ss.clones.size());
   ASSERT_EQ(3u, ss.clones[0].cloneid);
   ASSERT_EQ(2u, ss.clones[0].snaps.size());
-  ASSERT_EQ(2u, ss.clones[0].snaps[0]);        //this could go away in the future
-  ASSERT_EQ(3u, ss.clones[0].snaps[1]);
+  ASSERT_EQ(my_snaps[0], ss.clones[0].snaps[0]);       //this could go away in the future
+  ASSERT_EQ(my_snaps[1], ss.clones[0].snaps[1]);
   ASSERT_EQ(5u, ss.clones[0].overlap.size());
   ASSERT_EQ(0u, ss.clones[0].overlap[0].first);
   ASSERT_EQ(128u, ss.clones[0].overlap[0].second);
@@ -480,8 +481,8 @@ TEST(LibRadosSnapshots, SelfManagedSnapOverlapPP) {
   ASSERT_EQ(3u, ss.clones.size());
   ASSERT_EQ(3u, ss.clones[0].cloneid);
   ASSERT_EQ(2u, ss.clones[0].snaps.size());
-  ASSERT_EQ(2u, ss.clones[0].snaps[0]);        //this could go away in the future
-  ASSERT_EQ(3u, ss.clones[0].snaps[1]);
+  ASSERT_EQ(my_snaps[0], ss.clones[0].snaps[0]);       //this could go away in the future
+  ASSERT_EQ(my_snaps[1], ss.clones[0].snaps[1]);
   ASSERT_EQ(5u, ss.clones[0].overlap.size());
   ASSERT_EQ(0u, ss.clones[0].overlap[0].first);
   ASSERT_EQ(128u, ss.clones[0].overlap[0].second);
@@ -497,7 +498,7 @@ TEST(LibRadosSnapshots, SelfManagedSnapOverlapPP) {
 
   ASSERT_EQ(4u, ss.clones[1].cloneid);
   ASSERT_EQ(1u, ss.clones[1].snaps.size());
-  ASSERT_EQ(4u, ss.clones[1].snaps[0]);
+  ASSERT_EQ(my_snaps[2], ss.clones[1].snaps[0]);
   ASSERT_EQ(4u, ss.clones[1].overlap.size());
   ASSERT_EQ(0u, ss.clones[1].overlap[0].first);
   ASSERT_EQ(128u, ss.clones[1].overlap[0].second);