message: test: Force --nosandbox when --valgrind
diff: @@ -101,7 +101,7 @@ class BitcoinTestFramework(metaclass=BitcoinTestMetaClass):
  self.supports_cli = True
  self.bind_to_localhost_only = True
  self.parse_args()
  - self.disable_syscall_sandbox = self.options.nosandbox
  + self.disable_syscall_sandbox = self.options.nosandbox or self.options.valgrind
  self.default_wallet_name = "default_wallet" if self.options.descriptors else ""
  self.wallet_data_filename = "wallet.dat"
  # Optional list of wallet names that can be set in set_test_params to
  @@ -188,7 +188,7 @@ class BitcoinTestFramework(metaclass=BitcoinTestMetaClass):
  parser.add_argument("--perf", dest="perf", default=False, action="store_true",
  help="profile running nodes with perf for the duration of the test")
  parser.add_argument("--valgrind", dest="valgrind", default=False, action="store_true",
  - help="run nodes under the valgrind memory error detector: expect at least a ~10x slowdown, valgrind 3.14 or later required")
  + help="run nodes under the valgrind memory error detector: expect at least a ~10x slowdown. valgrind 3.14 or later required. Forces --nosandbox.")
  parser.add_argument("--randomseed", type=int,
  help="set a random seed for deterministically reproducing a previous test run")
  parser.add_argument('--timeout-factor', dest="timeout_factor", type=float, default=1.0, help='adjust test timeouts by a factor. Setting it to 0 disables all timeouts')
  