Skip to content

Commit cfd5d43

Browse files
PS-11435 [9.7] Add auth_openid_connect suite to default MTR runs (#6141)
The suite now runs in default mtr invocations and in the Jenkins parallel MTR per-worker split. Also guard the shared set_idp_vars.inc include: on servers built with -DWITH_AUTH_OPENID_CONNECT=OFF, AUTH_OIDC is empty and idp.test and proxy.test would run INSTALL PLUGIN with an empty SONAME and fail the default MTR run instead of skipping.
1 parent f8e45d3 commit cfd5d43

3 files changed

Lines changed: 16 additions & 3 deletions

File tree

mysql-test/mysql-test-run.pl

100755100644
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -269,6 +269,10 @@ sub set_term_args {
269269
#
270270
# Suites run by default (i.e. when invoking ./mtr without parameters)
271271
#
272+
# @DEFAULT_SUITES is divided into three groups, separated by blank lines:
273+
# 1. Default upstream MTR suites
274+
# 2. Percona-added MTR suites
275+
# 3. Non-default upstream MTR suites
272276
our @DEFAULT_SUITES = qw(
273277
auth_sec
274278
binlog
@@ -311,6 +315,7 @@ sub set_term_args {
311315
component_connection_control
312316
jdv
313317
318+
auth_openid_connect
314319
component_audit_log_filter
315320
percona
316321
percona_binlog

mysql-test/suite/auth_openid_connect/inc/set_idp_vars.inc

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,11 @@
1+
# Skip when the server build does not provide the auth_openid_connect
2+
# plugin (built with -DWITH_AUTH_OPENID_CONNECT=OFF). $AUTH_OIDC is empty
3+
# in that case and INSTALL PLUGIN would run with an empty SONAME.
4+
if (!$AUTH_OIDC)
5+
{
6+
--skip Auth OpenID Connect plugin not available in this build
7+
}
8+
19
if ($MTR_OIDC_IDP_HOST)
210
{
311
--let $IDP_HOST = $MTR_OIDC_IDP_HOST

mysql-test/suites-groups.sh

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,7 @@ function set_suites() {
3434
WORKER_3_MTR_SUITES="main|big,binlog|nobig,percona_binlog|nobig,innodb_zip|nobig,engines/funcs|nobig,innodb_gis|big,parts|big,binlog_nogtid|nobig,collations,sysschema|nobig,rocksdb_sys_vars,connection_control|nobig,jdv"
3535
WORKER_4_MTR_SUITES="innodb|big,group_replication|nobig,rpl_gtid|nobig,percona_rpl_gtid|nobig,binlog|big,percona_binlog|big,innodb_fts|nobig,binlog_nogtid|big,funcs_1|nobig,innodb_gis|nobig,perfschema|big,federated|nobig,gis|big"
3636
WORKER_5_MTR_SUITES="main|nobig,rpl|big,percona_rpl|big,rpl_nogtid|big,component_encryption_udf|big,x|nobig,binlog_gtid|nobig,component_keyring_file|nobig,funcs_2|big,test_services,binlog_gtid|big,stress|nobig,information_schema,component_js_lang|nobig,component_connection_control|big"
37-
WORKER_6_MTR_SUITES="group_replication|big,percona_innodb|nobig,rocksdb|big,innodb_undo|nobig,stress|big,x|big,auth_sec|nobig,rocksdb_rpl|big,gcol|nobig,query_rewrite_plugins,secondary_engine,interactive_utilities,connection_control|big,percona|big"
37+
WORKER_6_MTR_SUITES="group_replication|big,percona_innodb|nobig,rocksdb|big,innodb_undo|nobig,stress|big,x|big,auth_sec|nobig,auth_openid_connect,rocksdb_rpl|big,gcol|nobig,query_rewrite_plugins,secondary_engine,interactive_utilities,connection_control|big,percona|big"
3838
WORKER_7_MTR_SUITES="rpl|nobig,percona_rpl|nobig,innodb_undo|big,perfschema|nobig,component_keyring_file|big,percona_innodb|big,sys_vars|nobig,auth_sec|big,opt_trace|nobig,json,engines/iuds|nobig,test_service_sql_api,service_status_var_registration,opt_trace|big"
3939
WORKER_8_MTR_SUITES="engines/funcs|big,clone|big,rocksdb|nobig,clone|nobig,parts|nobig,federated|big,engines/iuds|big,gis|nobig,sys_vars|big,gcol|big,funcs_2|nobig,service_udf_registration,percona-pam-for-mysql"
4040
elif [[ "$1" == "RelWithDebInfo" ]]; then
@@ -43,7 +43,7 @@ function set_suites() {
4343
WORKER_1_MTR_SUITES="main|nobig,main|big,parts,percona,component_audit_log_filter,engines/iuds,component_connection_control,opt_trace,information_schema"
4444
WORKER_2_MTR_SUITES="component_encryption_udf,innodb_zip,gis,json,component_js_lang,router"
4545
WORKER_3_MTR_SUITES="group_replication|big,binlog,percona_binlog,test_service_sql_api,component_masking_functions,connection_control,service_sys_var_registration,jdv"
46-
WORKER_4_MTR_SUITES="rocksdb|big,rpl_gtid,percona_rpl_gtid,auth_sec,engines/funcs,innodb_undo,interactive_utilities,query_rewrite_plugins,audit_null"
46+
WORKER_4_MTR_SUITES="rocksdb|big,rpl_gtid,percona_rpl_gtid,auth_sec,auth_openid_connect,engines/funcs,innodb_undo,interactive_utilities,query_rewrite_plugins,audit_null"
4747
WORKER_5_MTR_SUITES="rocksdb|nobig,percona_innodb,sys_vars,perfschema,innodb_fts,binlog_gtid,rpl_encryption,service_udf_registration,procfs"
4848
WORKER_6_MTR_SUITES="rpl|nobig,percona_rpl|nobig,rpl_nogtid|big,rocksdb_rpl,clone,innodb_gis,funcs_1,funcs_2,collations,jp,percona-pam-for-mysql"
4949
WORKER_7_MTR_SUITES="group_replication|nobig,rpl_nogtid|nobig,innodb|nobig,component_keyring_file,rocksdb_stress,gcol,stress,test_services,secondary_engine,service_status_var_registration"
@@ -58,6 +58,6 @@ function set_suites() {
5858
WORKER_5_MTR_SUITES="main|big,innodb_gis|big,rpl_gtid|big,percona_rpl_gtid|big,binlog|big,percona_binlog|big,funcs_1,auth_sec|big,binlog_gtid|nobig,opt_trace,secondary_engine,service_status_var_registration"
5959
WORKER_6_MTR_SUITES="main|nobig,group_replication|nobig,parts|big,innodb_undo|big,innodb_fts|nobig,perfschema|big,engines/iuds,innodb_zip|nobig,rocksdb_sys_vars,component_connection_control,service_udf_registration"
6060
WORKER_7_MTR_SUITES="innodb|big,rpl|nobig,percona_rpl|nobig,engines/funcs,sys_vars,x|nobig,component_keyring_file|nobig,stress,funcs_2,json,interactive_utilities,procfs"
61-
WORKER_8_MTR_SUITES="component_encryption_udf,innodb|nobig,innodb_fts|big,rpl_gtid|nobig,percona_rpl_gtid|nobig,gcol,gis,auth_sec|nobig,binlog_gtid|big,test_service_sql_api,connection_control,percona-pam-for-mysql"
61+
WORKER_8_MTR_SUITES="component_encryption_udf,innodb|nobig,innodb_fts|big,rpl_gtid|nobig,percona_rpl_gtid|nobig,gcol,gis,auth_sec|nobig,auth_openid_connect,binlog_gtid|big,test_service_sql_api,connection_control,percona-pam-for-mysql"
6262
fi
6363
}

0 commit comments

Comments
 (0)