proxysql

Commit Graph

Author	SHA1	Message	Date
Rene Cannao	8df688dd8a	test: register 5 new unit tests in groups.json for CI	2 weeks ago
Rene Cannao	a36834f968	test: add 6 unit test files with 676 assertions (Round 4) New test files (test/tap/tests/unit/): Core: - c_tokenizer_unit-t (59 tests): tokenizer/tokenize/free_tokenizer, c_split_2, mysql_query_digest_and_first_comment (literal replacement, comment stripping, NULL handling, grouping_limit, IN-clause), mysql_query_digest_first_stage, strip_comments - config_write_unit-t (111 tests): all 7 Write_*_to_configfile() functions with in-memory SQLite — MySQL users/servers/query rules, scheduler, restapi, global variables, proxysql_servers; tests field formatting, NULL handling, double-quote escaping, prefix grouping - listen_validator_unit-t (112 tests): trim_copy, split_listener_list, is_ipv4/ipv6_wildcard, parse_listener (IPv4/IPv6/Unix socket/ invalid/edge cases), sets_intersect, listeners_conflict, validate_module_listener_conflicts - ezoption_parser_unit-t (110 tests): standalone ezOptionParser — flags, string/int/float options, defaults, required/unknown, aliases, multi-value, usage layout, export/import, validators, vector getters - pgsql_txn_state_unit-t (+58 tests, 112 total): EXTENDED with PgSQL_ExplicitTxnStateMgr state machine tests — BEGIN/COMMIT/ ROLLBACK lifecycle, ROLLBACK AND CHAIN, savepoint add/release/ rollback_to, reset_state, fill_internal_session JSON, case- insensitive savepoints, END/ABORT synonyms. Uses minimal PgSQL_Session mock with variable snapshots. GenAI/MCP (PROXYSQLGENAI=1): - genai_discovery_schema_unit-t (172 tests): Discovery_Schema with in-memory SQLite — run/agent_run management, schema/object/column/ index/FK insertion, profile/LLM summary/domain/metrics/notes, list_objects pagination+ordering, get_object with columns/indexes, get_relationships FK mappings, FTS rebuild+search, fingerprint_mcp_args, compute_mcp_digest determinism, MCP_Query_Digest_Stats timing, MCP_Query_Processor_Output, MCP_Query_Rule defaults Running totals: 27 test files, 1944+ assertions, all passing.	2 weeks ago
Rene Cannao	a8c5432fc5	fix: remove stale proxysql.db before launching cluster sync replicas test_cluster_sync_mysql_servers-t spawns replica ProxySQL instances with a specific config file setting admin port to 96061/96062. If a proxysql.db from a previous run exists in the workdir, ProxySQL ignores the config file and loads from the DB instead, using the old admin port. The test then fails to connect on the expected port. Fix: remove proxysql.db and proxysql_stats.db before launching the replica, not just after. The post-launch cleanup is kept as well for normal operation.	2 weeks ago
Rene Cannao	0f76a18329	fix: resolve CI failures in charset_find and statistics unit tests charset_find_unit-t: - Fix plan count mismatch: plan(63) but only 62 ok() calls. Changed to plan(62). statistics_unit-t: - Root cause: TSDB auto-detect in Makefile searched for symbol 'ProxySQL_TSDB' which doesn't exist. Changed to 'init_tsdb_variables'. Without TSDB detection, struct layout mismatch caused statsdb_disk to appear as NULL (member offset shifted by #ifdef PROXYSQLTSDB). - Added statsdb_ok guard + skip() for test_init_creates_tables to prevent segfault if construction fails (graceful degradation). - Use TMPDIR/tmp directly instead of GloVars.datadir for statsdb path.	2 weeks ago
Rene Cannao	99dcb5b160	fix: namespace stats_utils helpers to avoid symbol collision ProxySQL_Admin_Tests2.cpp defines its own parse_int_or_zero() in an anonymous namespace. Our declaration in Stats_Tool_Handler.h (included transitively via cpp.h) created an ambiguous overload error. Fix: wrap the 4 helper functions in namespace stats_utils in both the header and implementation. A using-directive within the .cpp keeps existing call sites working without qualification.	2 weeks ago
Rene Cannao	99c86e5759	test: run check_groups.py at start of run-multi-group.bash Abort CI early if any compiled test binaries are missing from groups.json. This prevents silent test omissions — if a developer adds a test but forgets to register it, CI fails immediately with a clear message instead of silently skipping it.	2 weeks ago
Rene Cannao	093693ba06	test: add check_groups.py to verify groups.json completeness Python tool that checks consistency between groups.json and compiled TAP test binaries: - WARNING (exit 0): tests in groups.json that aren't compiled yet - ERROR (exit 1): compiled tests missing from groups.json Scans test/tap/tests/, test/tap/tests/unit/, and test/tap/tests_with_deps/ for ELF binaries matching *-t pattern. Usage: python3 test/tap/groups/check_groups.py	2 weeks ago
Rene Cannao	fa668d646a	test: register 21 new unit tests in groups.json for CI Add all new unit test binaries to the CI test group configuration: - 13 core tests → unit-tests-g1 - 8 GenAI tests → ai-g1 This ensures CI pipelines discover and run all new tests.	2 weeks ago
Rene Cannao	ece648fc16	test: add 6 more unit test files with 550 assertions (Tier 2+3) Expand coverage for firewall whitelist, statistics, config validation, logging, and GenAI thread variable management. New test files (test/tap/tests/unit/): Core: - query_processor_firewall_unit-t (44 tests): load_firewall() users/rules/sqli fingerprints, find_firewall_whitelist_user/rule, whitelisted_sqli_fingerprint, reload/clear, memory accounting, PgSQL parity, multiple digest bsearch - statistics_unit-t (20 tests, 95 with TSDB): timer functions (timetoget for MySQL threads, query cache, CPU, logger, digest), TSDB set/get/has variable with range validation, insert/query metrics, backend health, downsampling, retention - config_validation_unit-t (54 tests): addField() formatting and escaping, validate_backend_users() duplicate detection for MySQL/PgSQL, validate_backend_servers() with field aliases and default ports, validate_proxysql_servers() - log_utils_unit-t (42 tests): LogBuffer append/write/operator<< overloads, data() accessor, reset(), should_log() rate limiting, GetLogBufferThreadContext() lifecycle GenAI/MCP (PROXYSQLGENAI=1): - genai_mcp_thread_unit-t (197 tests): MCP_Threads_Handler constructor defaults (26 vars), boolean/port/timeout/string variable set/get with range validation, has_variable, get_variables_list, null safety, target auth map - genai_thread_unit-t (193 tests): GenAI_Threads_Handler constructor defaults (31 vars), integer range validation for 15+ variables, double/boolean/string round-trips, has_variable, get_variables_list, overwrite and rejection persistence Note: found potential bug — llm_cache_enabled has no get/set handler despite being listed in variables array Minimal production change: - include/proxysql_config.h: add friend class for test access to private validate_backend_users/servers/proxysql_servers methods Running totals: 21 test files, 1268 assertions, all passing.	2 weeks ago
Rene Cannao	1d2f16b506	test: add 4 more unit test files with 198 assertions (Tier 1) Expand coverage for charset lookups, GenAI Query/LLM/MCP components. New test files (test/tap/tests/unit/): Core: - charset_find_unit-t (62 tests): all 4 proxysql_find_charset functions — edge cases for NULL returns, utf8mb3→utf8 translation, case-insensitive matching, collation preference GenAI/MCP (PROXYSQLGENAI=1): - genai_query_handler_unit-t (65 tests): validate_sql_identifier, escape_string_literal, strip_leading_comments, json_string/ json_int/json_double safe extraction, is_pgsql_protocol, is_runtime_online_status, uppercase_or_unknown - genai_llm_clients_unit-t (35 tests): is_retryable_error HTTP status codes (408/429/5xx) and CURL error codes, plus LLM_WriteCallback data accumulation - genai_mcp_endpoint_unit-t (36 tests): validate_bearer_token extraction/comparison, create_jsonrpc_response/error JSON-RPC 2.0 formatting Minimal production changes for testability: - include/Query_Tool_Handler.h: friend class + extern declarations for 8 pure helper functions - lib/Query_Tool_Handler.cpp: remove static from 8 file-local pure functions (no behavioral change) - include/MCP_Endpoint.h: friend class + new static validate_bearer_token() extracted from authenticate_request() - lib/MCP_Endpoint.cpp: extract token validation into static method - lib/LLM_Clients.cpp: remove static from is_retryable_error() and rename WriteCallback to LLM_WriteCallback (avoid collisions) Running totals: 15 test files, 718 assertions, all passing.	2 weeks ago
Rene Cannao	accc68c9dc	test: add 11 new unit test files with 520 assertions for lib/ coverage Add targeted unit tests for pure functions and isolated components across lib/, increasing test coverage for both core utilities and GenAI/MCP components. New test files (test/tap/tests/unit/): Core utilities: - pgsql_variables_validator_unit-t (75 tests): bool, float, string, memory size, search_path, and client_encoding validators - gen_utils_unit-t (28 tests): mywildcmp wildcard matching, escape_string_single_quotes, Proxy_file_regular - mysql_encode_unit-t (23 tests): mysql_encode_length, CPY3/CPY8, proxy_my_crypt XOR, write_encoded_length - proxysql_utils_unit-t (23 tests): hex/unhex, sql_escape, split_str, replace_str, generate_multi_rows_query, checksums - proxy_protocol_unit-t (22 tests): PROXY v1 header parsing, subnet validation, IPv4/IPv6 network matching - pgsql_error_helper_unit-t (139 tests): SQLSTATE code mapping, all 38 error class branches, severity classification, fill/reset - pgsql_txn_state_unit-t (54 tests): TxnCmdParser for BEGIN, COMMIT, ROLLBACK, SAVEPOINT, RELEASE, AND CHAIN variants - sqlite3db_unit-t (21 tests): execute, return_one_int, check_table_structure, build_table, row ops, locking GenAI/MCP (PROXYSQLGENAI=1): - genai_stats_parsing_unit-t (79 tests): parse_server_filter, parse_digest_filter, parse_ll_or_zero, get_interval_config, calculate_percentile - genai_fts_string_unit-t (32 tests): MySQL_FTS sanitize_name, escape_identifier, escape_sql, table name construction - genai_anomaly_unit-t (24 tests): normalize_query SQL normalization, check_sql_injection pattern detection Minimal production changes for GenAI testability: - include/Anomaly_Detector.h: add friend class for test access - include/Stats_Tool_Handler.h: declare 4 parsing helpers, move 2 pure computation methods (get_interval_config, calculate_percentile) to public - lib/Stats_Tool_Handler.cpp: remove static from 4 free helper functions (no behavioral change, only linkage visibility)	2 weeks ago
Rene Cannao	1ebe4de9b7	Fix mysql84 pre-proxysql hook, basictests setup, and move flaky tests - mysql84 pre-proxysql.bash: use docker exec for all ProxySQL admin commands (was using mysql -h127.0.0.1 which can't reach the container) - basictests setup-infras.bash: use INSERT OR REPLACE when remapping hostgroups 1300/1301 -> 0/1 to avoid UNIQUE constraint violation when target hostgroups already have entries - proxysql-tester.py: skip per-stage coverage generation when MULTI_GROUP=1 (was spawning 32 gcov processes on 35k .gcno files) - Move test_cluster_sync-t and test_read_only_actions_offline_hard_servers-t to legacy-g5 pending investigation of cluster sync algorithm issues. Move reg_test_3765_ssl_pollout-t from legacy-g5 to legacy-g4.	2 weeks ago
Rene Cannao	39840b5393	Exclude /usr and deps from coverage reports fastcov was including system headers (/usr/include/c++/...) and vendored dependencies (deps/json/json.hpp) in coverage output. Use --exclude instead of --include since project source files have bare filenames in gcov output (no path prefix to match against), while system/deps paths are always absolute.	2 weeks ago
Rene Cannao	5a8b6e9a89	Fix gcov coverage collection for multi-group CI runs Three problems caused machine stalls and 0% coverage in multi-group runs: 1. All test-runner containers wrote TAP test .gcda files to the shared ${WORKSPACE}, causing concurrent corruption and memory bloat when multiple gcov processes fought over the same files. 2. Each container independently ran fastcov -j$(nproc) in its EXIT trap, spawning hundreds of concurrent gcov processes across all containers. 3. GCOV_PREFIX_STRIP=3 placed .gcda at proxysql/lib/obj/ but .gcno was copied to lib/obj/ — path mismatch meant gcov couldn't correlate them. Additionally, the --include filter expected full workspace paths but gcov records bare filenames, so all results were filtered out. Changes: - Set GCOV_PREFIX=/gcov/tap on test-runner containers so TAP .gcda files go to per-INFRA_ID /gcov/tap/ instead of shared workspace. - Pass MULTI_GROUP=1 from run-multi-group.bash to test-runners. When set, the EXIT trap copies .gcno adjacent to .gcda (handling path offset from GCOV_PREFIX_STRIP) but skips fastcov/genhtml. - Add centralized coverage collection in run-multi-group.bash: processes each group's /gcov directory sequentially with fastcov -j4 after all groups finish, then combines the .info files. - Remove --include filter from fastcov (source paths are bare filenames, not full paths, so the filter rejected everything). - Standalone mode (no MULTI_GROUP) is unchanged.	2 weeks ago
Rene Cannao	83c3e76506	Fix libparser.so loading for CI test binaries TAP test binaries link libcpp_dotenv.so which transitively needs libparser.so. The RUNPATH baked into libcpp_dotenv.so contains stale paths from the original build, so the dynamic linker can't find it. - Add LD_LIBRARY_PATH for the SKIP_PROXYSQL (unit test) host path - Add TAP_DEPS_PATH to env-isolated.bash so proxysql-tester.py constructs the correct LD_LIBRARY_PATH inside Docker containers	2 weeks ago
Rene Cannao	f8c64142cb	Fix cluster CI: remove duplicate primary INSERT and add sync_algorithm=3 - Remove redundant INSERT of primary into proxysql_servers on nodes. Commit `b02147b92` added the primary to the shared PROXYSQL_SERVERS_SQL but didn't remove the pre-existing per-node INSERT, causing a UNIQUE constraint violation that broke all multi-node CI groups. - Add cluster_mysql_servers_sync_algorithm=3 to test_cluster_sync config files and update hardcoded datadir paths to match current workspace.	2 weeks ago
Rene Cannao	c7647c7387	Fix coverage: generate separate reports and combine with fastcov -C Coverage data comes from two sources that may overlap: /gcov — ProxySQL daemon execution (via GCOV_PREFIX) WORKSPACE — TAP test execution (.gcda next to binaries) Both may cover the same code (e.g. libproxysql.a exercised by both ProxySQL runtime and TAP test harness). Previously we copied .gcda files between directories, which either overwrote (cp -f) or kept only the first version (cp -n) instead of combining counters. Now generates two separate .info files (proxysql.info + tap_tests.info) and combines them with fastcov -C, which correctly sums execution counters for overlapping source files.	2 weeks ago
Rene Cannao	8ca54a9def	Fix coverage: don't overwrite ProxySQL's gcda files with stale copies The coverage collection copied ALL .gcno and .gcda files from the workspace to /gcov using cp -f. But ProxySQL's .gcda files (with real execution data from the Docker container) were already in /gcov via GCOV_PREFIX. The cp -f overwrote them with the host's stale .gcda files (zero or outdated execution counts). This caused lib/MySQLFFTO.cpp and lib/PgSQLFFTO.cpp (and likely other ProxySQL library files) to show 0% coverage despite the FFTO tests running successfully and the .gcda files having real data. Fix: copy .gcno files with cp -f (always needed for fastcov), but copy .gcda files with cp -n (no-clobber) so ProxySQL's existing execution data in /gcov is preserved. TAP test .gcda files that don't exist in /gcov yet are still copied.	2 weeks ago
Rene Cannao	b02147b92b	Include primary in its own proxysql_servers table When a cluster node syncs proxysql_servers from the primary and runs LOAD PROXYSQL SERVERS TO RUNTIME, it rebuilds the node list from the synced data. If the primary isn't in its own proxysql_servers table, the synced list won't include it. The node then destroys the cluster monitor thread for the primary and never recreates it. Evidence: node 1 log shows "Destroyed Cluster Node Entry for host proxysql:6032" followed by "closing thread for peer proxysql:6032" after syncing proxysql_servers from the primary. After that, the node never detected any further checksum changes from the primary, causing test_cluster1-t and test_unshun_algorithm-t to fail. Fixes #5542.	2 weeks ago
Rene Cannao	f98a0e85ed	Remove Query_Processor.cpp debug filter that broke fast_routing test The debug filter for Query_Processor.cpp/process_query suppressed the "Searching thread-local 'rules_fast_routing' hashmap" messages that test_query_rules_fast_routing_algorithm-t relies on to verify fast routing behavior (expects 20 log entries, gets 0).	2 weeks ago
Rene Cannao	72a1c5dd4a	Set cluster_mysql_servers_sync_algorithm=3 in test_cluster_sync replica config The replica used the default algorithm (1) which syncs runtime mysql_servers — including monitor-shunned status. The test checks the config table (mysql_servers) for status='ONLINE', but the runtime sync overwrites the config table with SHUNNED status. With auto_select (3) and monitor enabled (-M flag), the replica uses mysql_servers_v2 sync which preserves the configured status.	2 weeks ago
Rene Cannao	0fa1b53709	Fix test_cluster_sync-t: connect to spawned replica via 127.0.0.1 The test spawns its own ProxySQL replica (port 16062) inside the test-runner container, but connected to it via cl.host (TAP_ADMINHOST) which resolves to the main ProxySQL container in the Docker setup. Since port 16062 doesn't exist on the main container, the connection hung indefinitely, causing the test to stall. Now uses R_HOST="127.0.0.1" for all replica connections (R_PORT), since the replica runs locally inside the same container as the test.	2 weeks ago
Rene Cannao	3c4ebf0da7	Set cluster_mysql_servers_sync_algorithm=3 (auto_select) on all nodes The default value (1) syncs both configured AND runtime mysql_servers, including changes made by the monitor (read_only checks moving servers between writer/reader hostgroups). This caused circular sync: a cluster node's monitor modified servers, creating a newer epoch, and the primary synced this back — overwriting its correct config and saving the corrupted version to disk. With auto_select (3), the algorithm chooses mysql_servers_v2 when the monitor is enabled, which only syncs user-configured changes and ignores monitor-triggered runtime modifications. Evidence: node 6042's monitor removed MariaDB servers from hostgroup 1701 ("Setting status OFFLINE HARD"), creating checksum 0x7C0A67D5 with epoch 1774288555. Primary (epoch 1774288549) synced from node 6042 and saved to disk, losing hostgroup 1300 permanently. Fixes #5540.	2 weeks ago
Rene Cannao	fb9ff4476e	Include TAP test coverage: copy .gcda files from source tree to /gcov TAP test binaries are compiled with gcov but their .gcda files are written to the source tree (next to the binaries), not to /gcov. The coverage collection only scanned /gcov, missing all TAP test coverage data. Now copies both .gcno and .gcda files from the workspace into /gcov before running fastcov, so TAP test coverage is included in reports.	2 weeks ago
Rene Cannao	0f12f5d6f1	Fix TAP_CLUSTER_NODES: include primary (6032) as first entry test_cluster1-t expects conns[0] to be the primary ProxySQL instance. TAP_CLUSTER_NODES only listed nodes 1-9 (ports 6042-6122), so the test loop operated on nodes 1-4 instead of primary + nodes 1-3. Node 4 (port 6072) is NOT in proxysql_servers (only core nodes 1-3 are registered), so changes on node 4 were invisible to the cluster — sync could never detect them, causing the 30-second timeout. Now TAP_CLUSTER_NODES starts with proxysql:6032 (primary), matching the old cluster_ports layout where 6032 was the first entry. Fixes #5535.	2 weeks ago
Rene Cannao	a4675096b5	Fix cluster admin_variables sync: set same admin vars on primary and nodes The primary had admin-debug=false and admin-restapi_enabled=false (defaults), while cluster nodes had them set to true. Since these variables are NOT in the CLUSTER_SYNC_INTERFACES_ADMIN exclusion list, they're included in the checksum. The checksum mismatch between primary and nodes caused admin_variables sync to never converge on satellite nodes. Now the primary sets the same admin-restapi_enabled=true and admin-debug=true as all nodes, ensuring matching checksums. Fixes #5535.	2 weeks ago
Rene Cannao	5f7fb254a9	Fix pgsql transaction test and auth_methods crash-on-skip - pgsql-transaction_variable_state_tracking-t: replace superuser-only variable allow_in_place_tablespaces with user-settable enable_mergejoin. Fixes #5536. - test_auth_methods-t: call stop_internal_noise_threads() before early return on MySQL 5.7, preventing 'terminate called without an active exception' crash from joinable std::thread destructors. Fixes #5537.	2 weeks ago
Rene Cannao	7739b86745	Move test_auth_methods-t from legacy-g2 to mysql84-g2 only Test requires MySQL 8.0+ (detects 5.7 and skips, but crashes with 'terminate called without an active exception' during noise thread cleanup). The crash-on-skip is a separate test bug.	2 weeks ago
Rene Cannao	5c9121ef40	Add hostgroup debug diagnostics and cache test tolerance - Add dump_hostgroup_debug() utility: dumps runtime_mysql_servers and stats_mysql_connection_pool for a given hostgroup via diag() output. Helps diagnose "hostgroup has no servers available" CI failures. - test_throttle_max_bytes_per_second_to_client-t: dump hostgroup 1300, 0, and 1 when create_table_test_sbtest1() fails. - test_warnings-t: dump target hostgroup state before running tests. - test_unsupported_queries-t: dump target hostgroup state before running tests. - test_query_cache_soft_ttl_pct-t: add tolerance of +1 to all three assertions. The soft TTL refresh race allows two threads to both see "needs refresh" simultaneously under load, producing one extra backend hit. This is a known property of the soft TTL mechanism.	2 weeks ago
Rene Cannao	42f4507e72	Fix test_tls_stats plan count and move caching_sha2 to mysql84 - test_tls_stats-t: plan(29) -> plan(27). The test executes 27 ok() calls on the happy path; plan was miscounted during authoring. - reg_test_4935-caching_sha2-t: remove from legacy-g4 (MySQL 5.7 doesn't support caching_sha2_password). Keep in mysql84-g4.	2 weeks ago
Rene Cannao	82c112c461	Fix coverage: restore GCOV_PREFIX for primary after unsetting for cluster nodes	2 weeks ago
René Cannaò	d1c86b141f	Merge pull request #5525 from sysown/v3.0-ci260322 Migrate CI-basictests from jenkins-build-scripts to in-repo test/infra	2 weeks ago
Rene Cannao	ba309e9c89	Fix groups.json: restore compact one-line-per-entry format	2 weeks ago
Rene Cannao	c84f4117ac	Fix pgsql admin port: set pgsql_ifaces in admin_variables, not pgsql_variables	2 weeks ago
Rene Cannao	98b56ec386	Move cluster node pgsql/sqlite ports to 7xxx range to avoid restapi collisions	2 weeks ago
Rene Cannao	48376427cf	Fix port conflicts: assign unique pgsql/sqlite ports to cluster nodes Cluster nodes only configured mysql admin and data ports, leaving pgsql (6133/6132) and sqlite (6030) at defaults — same as the primary. All nodes tried to bind the same ports, causing: 1. Bind failures for pgsql on cluster nodes 2. Cluster sync overwriting primary's pgsql config with empty tables (nodes had no working pgsql, so their pgsql tables were empty) Now each node gets unique ports for all interfaces: node N: mysql admin=6032+N10, mysql data=6033+N10, pgsql data=6133+N10, pgsql admin=6132+N10, sqlite=6030+N*10	2 weeks ago
Rene Cannao	e299de51cf	Add step-by-step diagnostics to reg_test__ssl_client_busy_wait-t Test now prints a description of what it does and numbered steps: 1. Connect to admin 2. Disable cluster scheduler 3. Measure idle CPU baseline 4. Trigger BUSY_LOOP (disconnect during query) 5. Measure CPU after BUSY_LOOP 6. Wait for cleanup 7. Verify CPU returned to idle 8. Trigger INF_LOOP (disconnect after response) 9. Measure CPU after INF_LOOP 10. Recover cluster scheduler	2 weeks ago
Rene Cannao	cc9a0414d3	Add connection details to noise thread failure logs All noise thread failure messages now include host, port, user, and the actual error message. Previously they only logged generic "connection failure" with no details, making it impossible to diagnose why a noise thread failed.	2 weeks ago
Rene Cannao	35a9bfb42e	Use 'proxysql' hostname in proxysql_servers instead of 127.0.0.1 TAP tests running in the test-runner container query proxysql_servers to find cluster nodes, then connect to them directly. With 127.0.0.1, the test-runner connects to itself instead of the ProxySQL container. Using 'proxysql' (the container hostname) works from both inside the container (resolves via /etc/hosts) and from the test-runner (resolves via Docker DNS on the shared network).	2 weeks ago
Rene Cannao	8705685f83	Address PR review comments: port collisions, gcov, loop bounds, fastcov - RESTAPI port offset changed from 6070+i to 7070+i to avoid collision with admin/mysql ports (6072 would collide with node4 admin) - Unset GCOV_PREFIX for background cluster nodes to prevent concurrent .gcda write corruption - Cap core-node and scheduler loops to min(NUM_NODES, 3) so smaller clusters don't reference non-existent nodes - TAP_CLUSTER_NODES now respects PROXYSQL_CLUSTER_NODES and SKIP_CLUSTER_START instead of hardcoding 9 - Fix fastcov flag: -i is invalid, use --include for source filtering	2 weeks ago
Rene Cannao	5a868d3366	Default AUTO_CLEANUP=1 in run-multi-group.bash	2 weeks ago
Rene Cannao	801a44399a	Split clickhouse tests into separate legacy-clickhouse group Move 3 clickhouse TAP tests (clickhouse_php_conn-t, test_clickhouse_server-t, test_clickhouse_server_libmysql-t) from legacy-g1/g3 to legacy-clickhouse-g1. Remove infra-clickhouse23 from legacy/infras.lst. This eliminates the clickhouse container from all legacy-g1..g4 runs, reducing container count per group.	2 weeks ago
Rene Cannao	6769ea4845	Make legacy/pre-proxysql.bash a no-op	2 weeks ago
Rene Cannao	359e6be734	Update debug filters in infra-mysql57 infra-config.sql	2 weeks ago
Rene Cannao	ce92a2a0e4	Redirect coverage generation output to log file in run-multi-group.bash fastcov, lcov, and genhtml produce verbose output that clutters the multi-group summary. Now redirected to coverage-generation.log inside the combined coverage report directory. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2 weeks ago
Rene Cannao	0590803889	Limit code coverage to proxysql source directories only Replace exclude-based filtering (-e /usr/include, deps/) with include-based filtering (-i) for include/, lib/, src/, test/. This avoids capturing coverage for system headers, vendored dependencies, and other non-proxysql code. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2 weeks ago
Rene Cannao	8c67073c15	Redirect coverage generation output to dedicated log file fastcov and genhtml produce verbose output that clutters the group test logs. Now redirected to coverage-generation.log inside the coverage report directory. Status messages still go to stdout. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2 weeks ago
Rene Cannao	6fcb8fd631	Run ProxySQL cluster nodes inside a single container Instead of spawning 1+9 Docker containers for the ProxySQL cluster, run all instances as background processes inside the primary container. Each node gets its own port pair (admin=6032+i10, mysql=6033+i10) and data directory. This eliminates 9 Docker containers per group, avoiding Docker networking/iptables contention that caused "no route to host" errors under parallel test execution. Changes: - start-proxysql-isolated.bash: starts cluster nodes as background processes inside the container, then initializes the cluster and installs the scheduler — all in one script - check_all_nodes.bash: polls 127.0.0.1 on different ports instead of container hostnames - env-isolated.bash: TAP_CLUSTER_NODES uses proxysql:PORT format - default/pre-proxysql.bash, legacy/pre-proxysql.bash: no longer call cluster_start.bash/cluster_init.bash (handled by start-proxysql) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2 weeks ago
Rene Cannao	879ebdc220	Fix startup order and add readiness check in test runner ensure-infras.bash: Move pre-proxysql hook (cluster setup) to run BEFORE backend infras start. Backends need ProxySQL and cluster to be fully ready because their docker-proxy-post.bash configures ProxySQL. Previous order was: ProxySQL → infras → cluster. New order: ProxySQL → cluster → infras (one by one) → setup hook. run-tests-isolated.bash: Add a readiness loop that waits up to 30s for ProxySQL admin port to be reachable from inside the test-runner container before running anything. Docker DNS resolution on newly created containers can take several seconds under load, causing "no route to host" (errno 113) errors on the first connection attempt. On failure, dumps DNS resolution diagnostics instead of silently proceeding. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2 weeks ago
Rene Cannao	d0872851a6	Enable coverage when either WITHGCOV or COVERAGE_MODE is set proxysql-tester.py only checked WITHGCOV to decide whether to run PROXYSQL GCOV DUMP. Since WITHGCOV now defaults to 0, passing COVERAGE=1 to run-multi-group.bash had no effect — the tester skipped coverage collection. Now checks both WITHGCOV and COVERAGE_MODE (which is passed into the container by run-tests-isolated.bash). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>	2 weeks ago

1 2 3 4 5 ...

2131 Commits (8df688dd8a5cb95a72bb765b7fbca680f856d117)