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

Use upstream ProxySQL 2.6.2 #58145

Merged
merged 2 commits into from
May 17, 2024
Merged

Use upstream ProxySQL 2.6.2 #58145

merged 2 commits into from
May 17, 2024

Conversation

snickell
Copy link
Contributor

@snickell snickell commented Apr 21, 2024

Switch back to using upstream ProxySQL and upgrade from version 2.0.x to version 2.6.2

We previously used a code.org fork of ProxySQL in order to get this patch: sysown/proxysql#3234

That patch was applied to upstream, but we never switched back to upstream. As a result, our version of ProxySQL has been frozen at 2.0, which potentially isn't great for maintaining compatibility with the most recent aurora upgrades.

We broke the ability to provision "full stack" adhocs when we moved our fork of ProxySQL to our GitHub Organization because we haven't published a binary build, so this is a good opportunity to fix our provisioning logic. Being on mainline ProxySQL also lowers the risks with our upcoming Aurora 3 / MySQL 8 upgrade.

This is still a high risk change and should be deployed on a Friday afternoon in a limited 2nd production release so we can quickly revert, if needed.

@snickell snickell requested a review from a team as a code owner April 21, 2024 20:02
@sureshc
Copy link
Contributor

sureshc commented Apr 26, 2024

Canary test was successful today.

Let's merge this after the AP Computer Science Principles "create task" deadline has passed next week.

@snickell
Copy link
Contributor Author

snickell commented May 9, 2024

@sureshc we're clear of AP CS P, do you still want this merged in?

@sureshc
Copy link
Contributor

sureshc commented May 17, 2024

I plan to merge and deploy this tonight or at some point this weekend. This change is not blocking the Aurora 3 / MySQL 8 upgrade, but it reduces risk if there are any ProxySQL / Aurora compatibility issues with the upgrade because we'll be on the latest release and on the mainline.

@snickell
Copy link
Contributor Author

I like your thinking

@sureshc sureshc merged commit 857b254 into staging May 17, 2024
1 of 2 checks passed
@sureshc sureshc deleted the proxysql-2.6 branch May 17, 2024 22:50
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.

None yet

2 participants