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

Add testing for new MySQL "Innovation" releases #5602

Closed
wants to merge 5 commits into from

Conversation

desrosj
Copy link
Contributor

@desrosj desrosj commented Oct 31, 2023

Trac ticket: https://core.trac.wordpress.org/ticket/59779


This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.

@desrosj desrosj changed the title Add MySQL 8.1 testing Add testing for new MySQL "Innovation" releases Oct 31, 2023
@desrosj desrosj marked this pull request as ready for review November 1, 2023 19:42
@johnbillion
Copy link
Member

We're at 500 GHA jobs with this configuration. Is that the limit?

@desrosj
Copy link
Contributor Author

desrosj commented Nov 22, 2023

Concurrently, yes. But this includes the Upgrade and Installation tests, which do not run on PRs or commits unless the corresponding workflow file is modified (or run manually). The installation test workflow accounts for 72 jobs, and the upgrade testing one has 384 in this run (some are not counted because the matrix was too large).

So for normal pushes and PRs, we're sitting around around 150-200.

It looks like the matrices in the upgrade testing workflow puts several of the jobs over the maximum allowed number of 255. I think I'll proceed with testing against these innovation releases in the PHPUnit workflow only for now.

@desrosj
Copy link
Contributor Author

desrosj commented Nov 22, 2023

Merged in https://core.trac.wordpress.org/changeset/57132. We can revisit the upgrade/install testing at a later time.

@desrosj desrosj closed this Nov 22, 2023
@desrosj desrosj deleted the try/adding-mysql-8.1-testing branch November 22, 2023 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants