Page MenuHomeSolus

Update mariadb to 10.4.6
ClosedPublic

Authored by livingsilver94 on Jul 1 2019, 9:12 AM.
Referenced Files
F11013843: D6640.id15920.diff
Fri, Aug 4, 5:46 PM
F11012056: D6640.diff
Thu, Aug 3, 11:59 PM
F11006980: D6640.diff
Tue, Aug 1, 6:01 PM
F10970344: D6640.id16010.diff
Fri, Jul 21, 2:14 AM
F10926631: D6640.id16575.diff
Mon, Jul 10, 1:38 PM
F10922932: D6640.id16153.diff
Sun, Jul 9, 8:47 PM
F10922926: D6640.id16570.diff
Sun, Jul 9, 8:45 PM
F10879733: D6640.id16575.diff
Jun 22 2023, 5:40 AM

Details

Summary

Changelog:
Release notes available here starting from version 10.3.14.

Package changes:

  • Move mariadb_first_install into mariadb-server package
  • Make MariaDB stateless
  • Minor fixes to package descriptions
  • Remove mariadb-test child package

Fixes T4337. Fixes T4053.

Test Plan

mariadb_first_install works if /var/db/mysql does not exist.

sudo systemctl start mariadb
nmap -sT localhost
# Port 3306/tcp is open
# Create /etc/mysql/my.cnf.d/test.cnf with
# [client-server]
# port = 4567
sudo systemctl restart mariadb
nmap -sT localhost
# Port 4567/tcp is open, 3306/tcp is not

mysqld --print-defaults correctly shows flags manually set in /usr/share/defaults/mysql/my.cnf.d/server.cnf.

Diff Detail

Repository
R2019 mariadb
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

livingsilver94 edited the test plan for this revision. (Show Details)

No AppArmor profiles for now. And they were in the wrong directory anyway.

livingsilver94 retitled this revision from Make mariadb stateless and move mariadb_first_install into mariadb-server to Make mariadb stateless; move mariadb_first_install into mariadb-server.
livingsilver94 edited the summary of this revision. (Show Details)
JoshStrobl added a subscriber: JoshStrobl.
JoshStrobl added inline comments.
package.yml
5

We should probably get this updated to 10.4.6-GA while we're at it.

94–121

I get the whole idea with why you aligned them, but honestly 4 spaces probably would be sufficient.

This revision now requires changes to proceed.Jul 2 2019, 5:29 PM

Update to 10.4.6 (I hate you Josh >.>)

livingsilver94 retitled this revision from Make mariadb stateless; move mariadb_first_install into mariadb-server to Update mariadb to 10.4.6.Jul 2 2019, 10:05 PM
livingsilver94 edited the summary of this revision. (Show Details)
livingsilver94 marked an inline comment as done.

Remove useless whitespace edit for the patch file

Make sure to read from /usr/share/defaults/mysql/my.cnf.d; restore old patch

I think we should remove the service file for multiple instances of mariadb (which is also not patched to run mariadb_first_install). What's your opinion?

I think we should remove the service file for multiple instances of mariadb (which is also not patched to run mariadb_first_install). What's your opinion?

Which service files are you talking about specifically? systemd has a ExecStartPre key for systemd services that we should preferably use for the services to have it run before ExecStart

pspec_x86_64.xml
861–862

@JoshStrobl this one. This service is not patched and it's meant to support multiple mariadb instances simultaneously.

pspec_x86_64.xml
861–862

Ah. Yea I'd nuke it.

JoshStrobl added inline comments.
package.yml
81–82

Honest question, is anybody actually going to ever use mysqltest? It's used for running test cases, seems to be more targeted for those that actually develop MariaDB than anything else. I'd almost say we could nuke this entirely and if possible, disable the compilation of it.

149

Should be 00644

This revision now requires changes to proceed.Aug 8 2019, 12:14 PM
package.yml
81–82

IDK, I just kept the things as they were, but yeah according to this article the -test suite is for MariaDB development. I'll delete it.

149

Ah OK. Many dirs inside /etc are with 755, I copied from them. I'll fix it.

sunnyflunk added inline comments.
package.yml
149

You are correct, dirs need to be 755 as they cannot be opened without the executable bit

package.yml
36

Have you tested to ensure building perl-dbd-mysql against all the mariadb bits in a local doesn't cause a circular dependency issue? Not sure if it'll want a specific release or release >= one at build time otherwise.

@JoshStrobl I gave a brief look at perl-dbd-mysql source code and it doesn't want a specific version of mariadb. I think there's no possibility for a circular dep to exist. Also, mytop calls perl-dbd-mysql at runtime, it's not linked.

Well, only one way to find out.

This revision is now accepted and ready to land.Sep 3 2019, 4:03 PM
This revision was automatically updated to reflect the committed changes.