Temporarily disable file ingestion if LockWAL is tested (facebook#12642)

Summary:
As titled. A proper fix should probably be failing file ingestion if the DB is in a lock wal state as it promises to "Freezes the logical state of the DB".

Pull Request resolved: facebook#12642

Reviewed By: pdillinger

Differential Revision: D57235869

Pulled By: jowlyzhang

fbshipit-source-id: c70031463842220f865621eb6f53424df27d29e9

db/external_sst_file_basic_test.cc

@@ -674,6 +674,10 @@ TEST_F(ExternalSSTFileBasicTest, NoCopy) {
   ASSERT_EQ(file3_info.smallest_key, Key(110));
   ASSERT_EQ(file3_info.largest_key, Key(124));
 
+  ASSERT_OK(dbfull()->LockWAL());
+  // TODO(FIXME): should not allow file ingestion.
+  // With below line, ingestion will block, without it, ingestion can go
+  // through. ASSERT_OK(dbfull()->Put(WriteOptions(), "vo", "vo"));
   s = DeprecatedAddFile({file1}, true /* move file */);
   ASSERT_OK(s) << s.ToString();
   ASSERT_EQ(Status::NotFound(), env_->FileExists(file1));

tools/db_crashtest.py

@@ -864,9 +864,14 @@ def finalize_and_sanitize(src_params):
     elif (dest_params.get("use_put_entity_one_in") > 1 and
         dest_params.get("use_timed_put_one_in") == 1):
         dest_params["use_timed_put_one_in"] = 3
+    # TODO: re-enable this combination.
+    if dest_params.get("lock_wal_one_in") != 0 and dest_params["ingest_external_file_one_in"] != 0:
+        if random.choice([0, 1]) == 0:
+            dest_params["ingest_external_file_one_in"] = 0
+        else:
+            dest_params["lock_wal_one_in"] = 0
     return dest_params
 
-
 def gen_cmd_params(args):
     params = {}