Skip to content

Commit

Permalink
Add privacy test suite into default suite list
Browse files Browse the repository at this point in the history
Summary:
Previously, both the nightly runs and the diff sandcastle runs do not include the privacy test suite by default. I found that's caused due to this default test suite.

Adding privacy here will make the SC run like [mysql-diff-release-parallelbig-withperfschema](https://fburl.com/sandcastle/3xcrabur) to include privacy suite.

Also, remove `show variables like` to prevent test breaking whenever a new sys variable is introduced.

Reviewed By: aditya-jalan

Differential Revision: D29154929

fbshipit-source-id: 8fb63adc11d
  • Loading branch information
Chi-I Huang authored and facebook-github-bot committed Jun 17, 2021
1 parent 8dfbf79 commit 1721ff6
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 23 deletions.
2 changes: 1 addition & 1 deletion mysql-test/mysql-test-run.pl
Original file line number Diff line number Diff line change
Expand Up @@ -238,7 +238,7 @@
# Suites run by default (i.e. when invoking ./mtr without parameters)
#
our $DEFAULT_SUITES =
"rpl_mts,auth_sec,binlog_gtid,binlog_nogtid,clone,collations,column_statistics,connection_control,encryption,federated,funcs_2,gcol,grant,sysschema,gis,information_schema,innodb,innodb_fts,innodb_gis,innodb_undo,innodb_zip,json,main,opt_trace,parts,perfschema,query_rewrite_plugins,rpl,rpl_gtid,rpl_nogtid,rpl_recovery,secondary_engine,service_status_var_registration,service_sys_var_registration,service_udf_registration,sys_vars,binlog,test_service_sql_api,test_services,x";
"rpl_mts,auth_sec,binlog_gtid,binlog_nogtid,clone,collations,column_statistics,connection_control,encryption,federated,funcs_2,gcol,grant,sysschema,gis,information_schema,innodb,innodb_fts,innodb_gis,innodb_undo,innodb_zip,json,main,opt_trace,parts,perfschema,query_rewrite_plugins,rpl,rpl_gtid,rpl_nogtid,rpl_recovery,secondary_engine,service_status_var_registration,service_sys_var_registration,service_udf_registration,sys_vars,binlog,test_service_sql_api,test_services,x,privacy";

# End of list of default suites

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,15 +6,6 @@ PRIMARY KEY (`id`)
INSERT INTO privacy_blood_donation_table VALUES(1, "User1");
set @original_privacy_plugin_purpose_policy_check=@@session.privacy_plugin_purpose_policy_check;
set session privacy_plugin_purpose_policy_check=fail_open;
show variables like 'privacy_plugin_%';
Variable_name Value
privacy_plugin_purpose_policy_cache_control OFF
privacy_plugin_purpose_policy_cache_retention 0
privacy_plugin_purpose_policy_check FAIL_OPEN
privacy_plugin_purpose_policy_check_always_false OFF
privacy_plugin_purpose_policy_check_bypass_evaluator OFF
privacy_plugin_purpose_policy_check_debug_dump OFF
privacy_plugin_response_attrs_contain_policy_status OFF
select * from privacy_blood_donation_table;
id name
1 User1
Expand Down
9 changes: 0 additions & 9 deletions mysql-test/suite/privacy/r/privacy_plugin_cache.result
Original file line number Diff line number Diff line change
Expand Up @@ -9,15 +9,6 @@ set global privacy_plugin_purpose_policy_cache_control=on;
set global privacy_plugin_purpose_policy_cache_retention=15;
set global performance_schema_esms_by_all=on;
set global sql_findings_control=on;
show variables like 'privacy_plugin%';
Variable_name Value
privacy_plugin_purpose_policy_cache_control ON
privacy_plugin_purpose_policy_cache_retention 15
privacy_plugin_purpose_policy_check FAIL_OPEN
privacy_plugin_purpose_policy_check_always_false OFF
privacy_plugin_purpose_policy_check_bypass_evaluator OFF
privacy_plugin_purpose_policy_check_debug_dump OFF
privacy_plugin_response_attrs_contain_policy_status OFF
CREATE TABLE privacy_research (
id int NOT NULL,
name VARCHAR(30) NOT NULL DEFAULT '',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ INSERT INTO privacy_blood_donation_table VALUES(1, "User1");

set @original_privacy_plugin_purpose_policy_check=@@session.privacy_plugin_purpose_policy_check;
set session privacy_plugin_purpose_policy_check=fail_open;
show variables like 'privacy_plugin_%';
query_attrs_add shard xdb.dbclient_test.1;

select * from privacy_blood_donation_table;
Expand Down
3 changes: 0 additions & 3 deletions mysql-test/suite/privacy/t/privacy_plugin_cache.test
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,6 @@ set global sql_findings_control=on;
query_attrs_add shard xdb.dbclient_test.1;
query_attrs_add query_policy fbig::research;

### check system/session setup
show variables like 'privacy_plugin%';

### setup test (table, ...)
CREATE TABLE privacy_research (
id int NOT NULL,
Expand Down

0 comments on commit 1721ff6

Please sign in to comment.