Bug#32130078 Bug#30647187 FIXES AUTOTEST EVENT DROP FAILURES

VarunNagaraju · VarunNagaraju · commit 3c8cc811effe · 2021-05-28T01:13:52.000+05:30
The upgrade/downgrade tests create and drop events between data node
restarts. Since, these event create and drop do not use schema
transactions, data node failures are handled with request retries
which can result in operations being attempted more than once, and
failing on the second (and reported) attempt. So, until
WL#4101 is implemented, the failure during event handling in
testUpgrade.cpp is ignored.

Change-Id: I827f2c2a6b06b05153bfb1a7b36112c7b394084d
diff --git a/storage/ndb/test/ndbapi/testUpgrade.cpp b/storage/ndb/test/ndbapi/testUpgrade.cpp
@@ -657,8 +657,8 @@ int runUpgrade_NR1(NDBT_Context* ctx, NDBT_Step* step){
       if (restarter.waitNodesStarted(&nodeId, 1))
         return NDBT_FAILED;
       
-      if (createDropEvent(ctx, step))
-        return NDBT_FAILED;
+      // Return value is ignored until WL#4101 is implemented.
+      createDropEvent(ctx, step);
     }
   }
   
@@ -798,9 +798,10 @@ runUpgrade_Half(NDBT_Context* ctx, NDBT_Step* step)
 
     CHK_NDB_READY(GETNDB(step));
 
-    if (event && createDropEvent(ctx, step))
+    if (event)
     {
-      return NDBT_FAILED;
+      // Return value is ignored until WL#4101 is implemented.
+      createDropEvent(ctx, step);
     }
 
     ndbout << "Half started" << endl;
@@ -855,9 +856,10 @@ runUpgrade_Half(NDBT_Context* ctx, NDBT_Step* step)
 
     CHK_NDB_READY(GETNDB(step));
 
-    if (event && createDropEvent(ctx, step))
+    if (event)
     {
-      return NDBT_FAILED;
+      // Return value is ignored until WL#4101 is implemented.
+      createDropEvent(ctx, step);
     }
   }
 
@@ -1171,10 +1173,8 @@ runBasic(NDBT_Context* ctx, NDBT_Step* step)
         trans.clearTable(pNdb, records/2);
         break;
       case 3:
-        if (createDropEvent(ctx, step, false))
-        {
-          return NDBT_FAILED;
-        }
+        // Return value is ignored until WL#4101 is implemented.
+        createDropEvent(ctx, step, false);
         break;
       }
     }

Original file line number	Diff line number	Diff line change
`@@ -657,8 +657,8 @@ int runUpgrade_NR1(NDBT_Context* ctx, NDBT_Step* step){`
`657`	`657`	`if (restarter.waitNodesStarted(&nodeId, 1))`
`658`	`658`	`return NDBT_FAILED;`
`659`	`659`
`660`		`- if (createDropEvent(ctx, step))`
`661`		`- return NDBT_FAILED;`
	`660`	`+ // Return value is ignored until WL#4101 is implemented.`
	`661`	`+ createDropEvent(ctx, step);`
`662`	`662`	`}`
`663`	`663`	`}`
`664`	`664`
`@@ -798,9 +798,10 @@ runUpgrade_Half(NDBT_Context* ctx, NDBT_Step* step)`
`798`	`798`
`799`	`799`	`CHK_NDB_READY(GETNDB(step));`
`800`	`800`
`801`		`- if (event && createDropEvent(ctx, step))`
	`801`	`+ if (event)`
`802`	`802`	`{`
`803`		`- return NDBT_FAILED;`
	`803`	`+ // Return value is ignored until WL#4101 is implemented.`
	`804`	`+ createDropEvent(ctx, step);`
`804`	`805`	`}`
`805`	`806`
`806`	`807`	`ndbout << "Half started" << endl;`
`@@ -855,9 +856,10 @@ runUpgrade_Half(NDBT_Context* ctx, NDBT_Step* step)`
`855`	`856`
`856`	`857`	`CHK_NDB_READY(GETNDB(step));`
`857`	`858`
`858`		`- if (event && createDropEvent(ctx, step))`
	`859`	`+ if (event)`
`859`	`860`	`{`
`860`		`- return NDBT_FAILED;`
	`861`	`+ // Return value is ignored until WL#4101 is implemented.`
	`862`	`+ createDropEvent(ctx, step);`
`861`	`863`	`}`
`862`	`864`	`}`
`863`	`865`
`@@ -1171,10 +1173,8 @@ runBasic(NDBT_Context* ctx, NDBT_Step* step)`
`1171`	`1173`	`trans.clearTable(pNdb, records/2);`
`1172`	`1174`	`break;`
`1173`	`1175`	`case 3:`
`1174`		`- if (createDropEvent(ctx, step, false))`
`1175`		`- {`
`1176`		`- return NDBT_FAILED;`
`1177`		`- }`
	`1176`	`+ // Return value is ignored until WL#4101 is implemented.`
	`1177`	`+ createDropEvent(ctx, step, false);`
`1178`	`1178`	`break;`
`1179`	`1179`	`}`
`1180`	`1180`	`}`