proxysql

Commit Graph

Author	SHA1	Message	Date
Rene Cannao	1c5e03f354	fix: use dynamic MySQL host/port in test_cluster_sync_mysql_servers The test hardcoded 127.0.0.1:13306-13309 as MySQL backend addresses. In the Unified CI, MySQL backends run in separate containers reachable by hostname (e.g. mysql1.infra-mysql57:3306), not on localhost ports. Use cl.mysql_host and cl.mysql_port from environment variables instead of hardcoded addresses. Non-ONLINE servers use offset ports as dummy entries since they're only sync test data and never connected to. Results: 44/44 tests pass (was 4/44 with immediate replica shutdown).	3 months ago
Rene Cannao	1232948fcb	fix: resolve cross-container networking for cluster sync tests The cluster sync tests spawn ProxySQL replicas inside the test-runner container. In the Unified CI, the primary ProxySQL runs in a separate container. The primary could not reach replicas registered as 127.0.0.1 because that resolves to the primary's own loopback. Fix: - Add --hostname test-runner to test-runner container so it has a stable DNS name on the Docker network - Add get_cluster_visible_host() to both cluster sync tests that returns the container hostname (resolvable by other containers) - Use this hostname for replica connection opts so proxysql_servers registrations and stats_proxysql_servers_checksums lookups work cross-container Results: - test_cluster_sync-t: 397/399 pass (was 21/399) - test_cluster_sync_mysql_servers-t: replicas now start and connect correctly, but further investigation needed for cluster node interference causing premature SHUTDOWN	3 months ago
Rene Cannao	69437751fd	fix: prevent cluster sync tests from hanging on replica shutdown failure Both test_cluster_sync-t and test_cluster_sync_mysql_servers-t launch ProxySQL replicas via system() in threads. If PROXYSQL SHUTDOWN fails or the admin connection is NULL, the process never exits and thread.join() hangs forever. Fix: replace system() with fork()/exec() to track the child PID. Before join(), wait up to 5 seconds for the process to exit, then SIGKILL it as a fallback. This guarantees the test always terminates.	3 months 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.	3 months ago
sysown1	b354cb6639	fix(tests): Replace hardcoded 127.0.0.1 with env-driven host/port CI migration to Docker containers requires tests to use environment variables for host/port configuration instead of hardcoded localhost addresses. Changes: - test_cluster_sync-t.cpp: Use cl.host and cl.admin_port for DELETE queries - test_cluster_sync_mysql_servers-t.cpp: Use cl.host and cl.admin_port - test_read_only_actions_offline_hard_servers-t.cpp: Use cl.host and cl.admin_port - test_simple_embedded_HTTP_server-t.cpp: Use cl.host for HTTP requests The CommandLine class reads from TAP_ADMINHOST and TAP_ADMINPORT environment variables set by CI infrastructure, enabling tests to work in containerized environments where ProxySQL runs on a different hostname.	3 months ago
Rene Cannao	582c22333f	test: Migration to new testing infrastructure and cleanup - Update CommandLine to support TAP_CLUSTER_NODES, TAP_WORKDIR, and other env vars. - Update default admin credentials to 'radmin' in CommandLine to match CI standards. - Enforce connections via ProxySQL in fast-forward and binlog tests. - Replace hardcoded credentials and hosts with CommandLine variables in cluster tests. - Improve portability of tests by using dynamic host/port and TEST_DEPS. - Ensure strict equality checks for query counts in binlog reader tests.	4 months ago
Javier Jaramago Fernández	cca2a88fa4	Refactor several TAP tests utilities - Simplified and unified condition waiting checks. - Added new helper function for fetching cluster core nodes.	2 years ago
Miro Stauder	f24e25f61a	use local includes - proxysql	2 years ago
Miro Stauder	08221a42e3	use local includes - mysql	2 years ago
Miro Stauder	6807d3a481	use local includes - libconfig	2 years ago
Javier Jaramago Fernández	9b00eb33a5	Make 'test_cluster_sync_mysql_servers' exit gracefully in clusterless env Current GitHub actions testing environment lacks of a ProxySQL cluster. This change allows the test to gracefully exit when the env conditions for it are not met.	2 years ago
Rahim Kanji	1decc6a993	Added delay to avoid race condition in tap test	3 years ago
Rahim Kanji	082a59d4c9	Fixed test case.	3 years ago
Rahim Kanji	f35003802e	* The cluster template file now includes "cluster_sync_interfaces=false." * In the event of any failure, it's crucial to call the cleanup function to prevent any orphaned replica instances from remaining active.	3 years ago
Rahim Kanji	5114069018	* Fixed test case	3 years ago
Rahim Kanji	e8b24ff290	* Added more test cases. * Cleared duplicate code. * Added comments	3 years ago
Rahim Kanji	625675ddbd	Created TAP test to ensure the cluster synchronization of 'runtime_mysql_servers' and 'mysql_servers_v2' across all nodes. The test involves creating two replica nodes, one with a monitor enabled and the other with a monitor disabled. The test verifies that the correct records are synced among these nodes based on the value of 'admin-cluster_mysql_servers_sync_algorithm'.	3 years ago

17 Commits (feature/pgsql-native-backend-protocol)