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

Fix install-package-tests random build failures #160

Closed
danielbachhuber opened this issue Feb 7, 2023 · 6 comments
Closed

Fix install-package-tests random build failures #160

danielbachhuber opened this issue Feb 7, 2023 · 6 comments
Assignees
Labels
bug Something isn't working scope:testing

Comments

@danielbachhuber
Copy link
Member

Builds fail quite frequently with this error:

> install-package-tests
Warning: arning] Using a password on the command line interface can be insecure.
ERROR 2003 (HY000): Can't connect to MySQL server on '127.0.0.1:32768' (111)
/home/runner/work/extension-command/extension-command/vendor/wp-cli/wp-cli-tests/bin/install-package-tests: 81: [: Illegal number: 
Detected MySQL at version .
+ mysql -e CREATE DATABASE IF NOT EXISTS `wp_cli_test`; -h127.0.0.1 -P32768 --protocol=tcp -uroot -proot
Warning: arning] Using a password on the command line interface can be insecure.
ERROR 2003 (HY000): Can't connect to MySQL server on '127.0.0.1:32768' (111)
Script install-package-tests handling the prepare-tests event returned with error code 1
Error: Process completed with exit code 1.

image

Done is:

  • Builds no longer randomly fail with this error.
@danielbachhuber
Copy link
Member Author

@danielbachhuber
Copy link
Member Author

Opened a GitHub discussion https://github.com/orgs/community/discussions/52445

@danielbachhuber
Copy link
Member Author

Here's a job where mysql never became available: https://github.com/wp-cli/automated-tests/actions/runs/4683019581/jobs/8297593809

image

mysql said it was ready though:

image

LIES

@danielbachhuber
Copy link
Member Author

@wp-cli/committers Automated tests seem to be passing more reliably now with #167 and #168

image

@wojsmol
Copy link
Contributor

wojsmol commented Apr 17, 2023

@danielbachhuber Please re-enable testing workflow in automated-testing repository.

@danielbachhuber
Copy link
Member Author

@wojsmol Ah, the warning message at the top that I just looked past? 😁 I clicked the button.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working scope:testing
Projects
None yet
Development

No branches or pull requests

2 participants