Bug#35327006 Stabilize the weekly runs [noclose]

Prevent rpl.rpl_deprecations from running with transaction_write_set_extraction
OFF or replica_parallel_type=LOGICAL_CLOCK as this generates extra
deprecation warnings that make the test fail.

Introduce a skip file that tests if one or more options are specified
on the MTR command line and use this to replace the sequence of
different have_*.inc invocations in the test.

Change-Id: I7a5fff8d0b80d4fcf531cf6d25996b2a1ac564c6

Author: dahlerlend

mysql-test/include/excludenoskip.list

@@ -142,6 +142,10 @@ have_statement_timeout.inc
 # --bootstrap=--init-file=std_data/explicit_undo_32.sql
 have_innodb_default_undo_tablespaces.inc
 
+# 4.9 General utility to skip if a specific command line option is used.
+# The usage is described in the .inc file.
+skip_by_command_line_option.inc
+
 # 5.0 Different protocols
 # Reason for inclusion: Tests should run only with supported protocols and
 # skip on others.

mysql-test/include/skip_by_command_line_option.inc

@@ -0,0 +1,24 @@
+# ==== Purpose ====
+#
+# Skip a test if a given option was passed on the command line.
+#
+# ==== Usage ====
+#
+# --let $variable_names = NAME1[, NAME2 ...]
+# --source include/skip_by_command_line.inc
+#
+# Parameters:
+#
+# $variable_names
+#   Comma-separated list of single-quoted variable names. If any of
+#   those global variables were initialized from the command line, the
+#   test will be skipped.
+#
+
+if ($variable_names == '') {
+  --die !!! ERROR IN TEST: set $variable_names
+}
+if (`SELECT COUNT(*) > 0 FROM performance_schema.variables_info WHERE VARIABLE_NAME IN ($variable_names) AND VARIABLE_SOURCE IN ('COMMAND_LINE')`) {
+  --let $failing_variables = `SELECT GROUP_CONCAT(VARIABLE_NAME) FROM performance_schema.variables_info WHERE VARIABLE_NAME IN ($variable_names) AND VARIABLE_SOURCE IN ('COMMAND_LINE')`
+  --skip Skipped because the following are set on the command line: $failing_variables
+}

mysql-test/suite/rpl/t/rpl_deprecations.test

@@ -1,13 +1,17 @@
 --source include/master-slave.inc
 
-# no need to run multiple times
+# Each deprecated option specified on the command line, produces an
+# extra warning during server restart. So when the test executes with
+# extra options, it causes a failure of the the test assertion that
+# checks warnings after a server restart.  mtr can't help passing
+# binlog_format, so allow that (but run only in row format to avoid
+# unnecessary repetition). The other options are not passed by
+# default, so skip the test if they are set on the command line while
+# invoking mtr.
 --source include/have_binlog_format_row.inc
---source include/have_slave_repository_type_table.inc
---source include/only_mts_replica_parallel_workers.inc
---source include/only_replica_preserve_commit_order.inc
---source include/only_mts_replica_parallel_type_logical_clock.inc
---source include/have_default_replica_transaction_retries.inc
---source include/have_default_slave_rows_search_algorithms.inc
+let $variable_names = 'master_info_repository', 'relay_log_info_repository', 'replica_parallel_workers', 'replica_preserve_commit_order',
+'replica_parallel_type', 'replica_transaction_retries', 'slave_rows_search_algorithms','transaction_write_set_extraction';
+--source include/skip_by_command_line_option.inc
 
 #####################################################################
 # WL#12926: Deprecate --log_bin_use_v1_row_events                   #