Skip to content

Conversation

stephanme
Copy link
Member

  • A short explanation of the proposed change:

Take out mysql 5.7 from the unit test matrix as it is explicitly not supported for cf-deployment.

  • An explanation of the use cases your change solves

Allows to use mysql features and rely on bug fixes that are not available in mysql 5.7.
Example: Dropping of duplicated unique constraints in DB migrations.

  • Links to any other associated PRs

Fixes #3949 and unblocks #3952

  • I have reviewed the contributing guide

  • I have viewed, signed, and submitted the Contributor License Agreement

  • I have made this pull request to the main branch

  • I have run all the unit tests using bundle exec rake

  • I have run CF Acceptance Tests

@stephanme stephanme requested a review from sethboyles October 11, 2024 09:11
@sethboyles
Copy link
Member

Our preference would be to hold off on this until May, as we support foundations using MySQL 5.7 until then.

@stephanme
Copy link
Member Author

OK, we need then to adapt #3949 to work with mysql 5.7.
Labeled it as blocked until May 2025.

@Samze
Copy link
Contributor

Samze commented May 27, 2025

Our support date for MySQL 5.7 is actually 2025-07-31 so perhaps we pause this until then?

@johha
Copy link
Contributor

johha commented Aug 20, 2025

@sethboyles @Samze Can we proceed on this? 2025-07-31 is over 😉

@sethboyles
Copy link
Member

Yes, please go ahead.

@sethboyles
Copy link
Member

(and thank you for your patience)

@johha johha removed the blocked label Aug 21, 2025
@johha johha merged commit 87dc55c into main Aug 21, 2025
12 checks passed
ari-wg-gitbot added a commit to cloudfoundry/capi-release that referenced this pull request Aug 21, 2025
Changes in cloud_controller_ng:

- Remove mysql 5.7 from unit tests
    PR: cloudfoundry/cloud_controller_ng#4018
    Author: Stephan Merker <[email protected]>
@moleske moleske deleted the remove-mysql-5-7 branch August 21, 2025 15:47
Gerg pushed a commit that referenced this pull request Aug 27, 2025
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.

Remove Mysql5.7 support / remove Mysql5.7 from PR validation
4 participants