11 Commits (v3.0-CodeCov0325)

Author	SHA1	Message	Date
Rene Cannao	0691b8a858	build: address reviewer feedback and finalize dynamic linking architecture ... Implemented comprehensive fixes based on CodeRabbit reviews and user feedback: - Restored dynamic linking for 'libtap.so' using shared 'libpq' and 'libre2' from deps. - Configured absolute 'rpath' in all Makefiles to ensure reliable runtime discovery. - Refined '.gitignore' with directory-scoped PEM patterns and removed broad globs. - Hardened noise routines: added NULL checks for MySQL handles, sanitized reconnect intervals, and wrapped 'std::stol' in try-catch blocks. - Fixed 'internal_noise_rest_prometheus_poller' to use proper HTTP authentication instead of embedding credentials in the URL. - Corrected test plans and logic in 'mysql-set_transaction-t.cpp' and 'test_admin_stats-t.cpp'. - Updated 'NOISE_TESTING.md' with correct heading hierarchy and error mechanism details.	3 months ago
Rene Cannao	43d535465e	test: enhance noise framework with specialized parameters and REST poller ... - Refactored noise routines to handle their own parameters and provide synchronized final reports via a global mutex and 'noise_log' helper. - Implemented a 5-second grace period during shutdown to allow routines to finish reporting. - Corrected 'internal_noise_prometheus_poller' to use the proper 'SHOW PROMETHEUS METRICS' syntax and removed unnecessary PgSQL logic. - Added 'internal_noise_rest_prometheus_poller' to fetch metrics via the REST API (defaulting to http://admin:admin@localhost:6070/metrics). - Updated 'test_admin_stats-t' to utilize the new REST poller and adjusted its test plan accordingly.	3 months ago
Rene Cannao	03f5f9b317	test: fix test_admin_stats-t failure by clearing persistent history ... The test 'test_admin_stats-t' was failing in persistent CI environments because 'history_mysql_status_variables' contained data from previous runs. Since some metrics (like Monitor DNS or MyHGM pool stats) may be added to the history table later than the initial set, the row count per variable_id became inconsistent, violating the test's assumption. This commit adds an explicit DELETE FROM history_mysql_status_variables at the start of the test to ensure a clean state and consistent row counts for all variables during validation.	3 months ago
Rene Cannao	496f67c7a2	Enhance Noise Injection robustness and error handling ... - Implement robust connection health checks and retry logic in noise routines. - Introduce global 'noise_failure_detected' flag to propagate fatal noise errors to the main TAP test. - Update exit_status() to report failure if background noise encountered critical issues. - Integrated background noise tools into the TAP test plan (updated plan() counts). - Redirect noise tool stderr to test stderr for transparent error logging. - Refactor NoiseOptions as a std::map for dynamic, routine-specific parameterization. - Fix libtap variants build to ensure noise_utils matches the correct protocol flags. - Link libtap.so and tests against libpq to resolve PostgreSQL dependency errors. - Correct clean_utils target in Makefile to prevent accidental deletion of source headers.	3 months ago
Rene Cannao	ddc06660a2	Integrate background noise into major TAP tests ... - test_admin_stats-t: Added Admin and Stats noise - pgsql-basic_tests-t: Added MySQL traffic and Prometheus noise - test_cluster_sync-t: Added Admin, Prometheus, and Stats noise - test_auth_methods-t: Added Admin and MySQL traffic noise	3 months ago
Miro Stauder	803510fa5e	fix mysql includes	3 years ago
Miro Stauder	c78819bda9	use inludes from our deps intead of system	3 years ago
René Cannaò	aeeaaba845	Minor test fixes ... Same fixes are in commit 1bc48ad in PR #3899	4 years ago
Javier Jaramago Fernández	bb3ee8adba	Fix 'text_admin_stats-t' timing issue	4 years ago
Jared Jetsel	d114dbe3a3	Const strings where appropriate and improved test for statistics ... Changed the get_variable_id_for_name, and knows_variable_name to use const string &. Changed immutable strings to to be const as well. Added details comment to test_admin_stats-t.cpp test. In the test code, improved documentation, set a valid interval for metrics insertion, and added the commands to set the interval. Changed the sleep time to be the inteval plus a second should be long enough to let new metrics data be added. Put additional sleep before test 2. Fixed incorrect column index usage in test 4 result set. Reformulated test 5, to check the number of records inserted after new interval is correct.	5 years ago
Jared Jetsel	e041be515e	Added the local_testing_datadir/ directory to gitignore, which is made during jenkins build script setup. Modified a tap Makefile to include curl path to resolve compilation error when compiling tap tests. Added tap test file 'test_admin_stats-t.cpp' for testing the ProxySQL_Statistics code changes, table definitions, and expected results.	5 years ago