Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V2.6.x testing global multiplexing disabled #4545

Open
wants to merge 4 commits into
base: v2.x
Choose a base branch
from

Conversation

mirostauder
Copy link
Collaborator

this adds test group for running all tap tests with
mysql-multiplexing='false'

tests are linked from the 'main' test/tap/tests folder
to folder test/test/test_multiplexing_false

to reconfigure proxysql, adds pre-test-run script
test/test/test_multiplexing_false/pre-test_multiplexing_false.bash

#
# configure 'tests_multiplexing_false'
#
# this script runs before running tests in this folder
#

# reconfigure proxysql
mysql -h ${TAP_ADMINHOST} -P ${TAP_ADMINPORT} -uadmin -padmin -e " \
SET mysql-multiplexing='false';
LOAD MYSQL VARIABLES TO RUNTIME;
SAVE MYSQL VARIABLES TO DISK;
"

@mirostauder
Copy link
Collaborator Author

retest this please

@mirostauder mirostauder force-pushed the v2.6.x-testing-global-multiplexing-disabled branch 4 times, most recently from 8dbb955 to bd34c4b Compare May 16, 2024 06:03
@mirostauder mirostauder force-pushed the v2.6.x-testing-global-multiplexing-disabled branch from bd34c4b to 1588b4a Compare May 16, 2024 07:38
@mirostauder
Copy link
Collaborator Author

consistently failing tests have been removed from test runs:

  • max_connections_ff-t
  • multiple_prepared_statements-t
  • mysql-init_connect-1-t
  • mysql-init_connect-2-t
  • reg_test_4264-commit_rollback-t
  • savepoint-3749-t
  • savepoint-948-t
  • test_auto_increment_delay_multiplex-t
  • test_backend_conn_ping-t

@JavierJF
Copy link
Collaborator

Retest this please.

1 similar comment
@JavierJF
Copy link
Collaborator

Retest this please.

@mirostauder
Copy link
Collaborator Author

Retest this please

@mirostauder
Copy link
Collaborator Author

Can one of the admins verify this patch?

@mirostauder
Copy link
Collaborator Author

retest this please

5 similar comments
@mirostauder
Copy link
Collaborator Author

retest this please

@mirostauder
Copy link
Collaborator Author

retest this please

@mirostauder
Copy link
Collaborator Author

retest this please

@mirostauder
Copy link
Collaborator Author

retest this please

@mirostauder
Copy link
Collaborator Author

retest this please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants