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

Deprecate MySQL < 5.6, MariaDB < 5.5 #15262

Merged
merged 2 commits into from Jan 21, 2019

Conversation

yol
Copy link
Member

@yol yol commented Jan 15, 2019

We do not have the resources to maintain compatibility with all sorts of
ancient MySQL servers in the future (concerning both development
manpower and testing infrastructure). Therefore, deprecate very old
MySQL and MariaDB server versions so that we can remove support for them
when problems come up or we would have to go out of our way to keep them
happy.

@yol
Copy link
Member Author

yol commented Jan 15, 2019

@MartijnKaijser what do you think?

@yol yol self-assigned this Jan 15, 2019
@yol yol added Component: Database Type: Cleanup non-breaking change which removes non-working or unmaintained functionality labels Jan 15, 2019
@MartijnKaijser
Copy link
Member

Does it still work or should we bail out gracefully (just not start and throw an error)?

@yol
Copy link
Member Author

yol commented Jan 15, 2019

At the moment, we should be fine (since we already went through a few iterations and fixed stuff for <5.7 specifically). I just want to get this in so we don't have to do that again in the future and people are aware that they are running ancient stuff. If we do break compatibility (by accident or on purpose) at some point, I think we should bail out.

@MartijnKaijser
Copy link
Member

We might even need to go further than log only and show a GUI pop-up imo.

@yol
Copy link
Member Author

yol commented Jan 15, 2019

IMO that might be a bit too noisy (on every start?) and... I just have no idea how to even do that from that part of the code 🙈 MySQL DB is already an advanced feature without GUI setup so I kind of assume that people running such a setup are more "prone" to scroll through the logs :-)

@MartijnKaijser
Copy link
Member

Let's merge this as a start

@MartijnKaijser MartijnKaijser added this to the Leia 18.0-Final milestone Jan 15, 2019
@yol
Copy link
Member Author

yol commented Jan 15, 2019

updated

@thezoggy
Copy link

https://www.fromdual.com/support-for-mysql-from-oracle
mysql 5.5 came out on Dec 2010 and its extended support ended on Dec 2018. so probably good time to depreciate support for it.

@yol
Copy link
Member Author

yol commented Jan 16, 2019

jenkins build this please

@yol yol changed the title Deprecate MySQL < 5.7 and MariaDB < 10.2 Deprecate MySQL < 5.5 Jan 16, 2019
@yol
Copy link
Member Author

yol commented Jan 16, 2019

@thezoggy I agree, but team opinion was not with me. Strongest argument in my opinion: Debian stable still ships mysql-server-5.5 and ARM users cannot use the newer mysql.com packages since they only provide x86/x64.

@Rechi
Copy link
Member

Rechi commented Jan 16, 2019

Debian stable still ships mysql-server-5.5

That's only partly correct. Debian stable ships mariadb 10.1.37 as default mysql server, https://packages.debian.org/stable/mysql-server.

Edit: Not even partly correct. Debian stable (stretch) only ships mariadb, no mysql package. The packages having mysql in it's name are only pseudo packages depending on mariadb packages.

@yol
Copy link
Member Author

yol commented Jan 16, 2019

Edit: Not even partly correct. Debian stable (stretch) only ships mariadb, no mysql package. The packages having mysql in it's name are only pseudo packages depending on mariadb packages.

You are right. @popcornmix?

@yol
Copy link
Member Author

yol commented Jan 16, 2019

For context: The bottom line of the team discussion was that there is no need to deprecate versions that are still in widespread use (which 5.5 arguably is) as long as there are still people willing to do the required testing (@MilhouseVH) and fixing (@DaveTBlake).

@MartijnKaijser
Copy link
Member

Regardless there's no harm in putting a notice in the log to ask users to upgrade to something newer if available.

@yol
Copy link
Member Author

yol commented Jan 16, 2019

I think that warning for MySQL <5.6 (because 5.5 is EOL) and MariaDB <5.5 (RHEL is on 5.5 so no 10.1 unfortunately, and for MariaDB, 5.5 is still supported)is a fair compromise, esp. given that we know now that Debian stable fulfils this. @MilhouseVH @DaveTBlake @popcornmix ?

@popcornmix
Copy link
Member

I'm okay with that.

@MilhouseVH
Copy link
Contributor

Sure, a < 5.6 warning is fine.

@DaveTBlake
Copy link
Member

Sure, warn away.

MySQL 5.5 is EOL, MariaDB 5.5 still supported but older versions are
not recommended
@yol
Copy link
Member Author

yol commented Jan 17, 2019

updated

if someone gives his +1 I think we're good for merge - thanks all for the discussion

@yol
Copy link
Member Author

yol commented Jan 17, 2019

jenkins build this please

@yol yol changed the title Deprecate MySQL < 5.5 Deprecate MySQL < 5.6, MariaDB < 5.5 Jan 17, 2019
@MartijnKaijser MartijnKaijser merged commit f1038dd into xbmc:master Jan 21, 2019
@yol yol deleted the deprecate-old-mysql branch January 21, 2019 08:34
MartijnKaijser added a commit to MartijnKaijser/xbmc that referenced this pull request Jan 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Database Type: Cleanup non-breaking change which removes non-working or unmaintained functionality v18 Leia Wiki: Done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants