Repository navigation
ci: remove broken host-MySQL step from integration-tests - #1115
HusainMain wants to merge 7 commits into
Conversation
The Start MYSQL Server step runs sudo /etc/init.d/mysql start, but the Actions runner image migrated to Ubuntu 26.04 which no longer ships /etc/init.d/mysql. Every PR since Oct 31 (including maintainers' own) fails at this step. The workflow already defines a services.mysql container (mariadb:12.3 on port 8888) that the tests connect to via 127.0.0.1:8888, so the legacy host-level step is redundant. Found while waiting for PR OWASP#1091 (JDBC driver fix for OWASP#1082) to pass CI.
HusainMain
changed the title
|
Removes the legacy WhyThe Actions runner image migrated to Ubuntu 26.04, which no longer ships The job's ImpactUnblocks integration-tests for all open PRs, including #1091 (JDBC driver fix for #1082). Testing
Found while waiting for PR #1091 to pass CI. Note: I couldn't create a separate GitHub issue for this (automated submissions are rate-limited), so I'm tracking it here as a standalone PR since it's infrastructure, not related to the JDBC fix itself. |
|
The Ubuntu 26 fix has since landed on dev through #1116, so this PR now only contains the Setup.java change, which duplicates #1091. #1091 is the dedicated fix for #1082 and already has review history, so I'd suggest closing this one and keeping that. To get #1091 running again, merge the latest dev into it. It also needs a regression test, or manual test steps in the description (delete database.properties, restart Tomcat, run setup, confirm there is no "No suitable driver"). |
No description provided.