search

parcio / julea

A Flexible Storage Framework for HPC
GNU Lesser General Public License v3.0
32 stars 30 forks source link

Spack commit too old for MySQL #160

Closed kiraduwe closed 1 year ago

kiraduwe commented 1 year ago

When running MariaDB on the ants cluster with singularity, mysqlis not available which is required to connect to the database from outside of the container. When trying to install mysql via the JULEA spack, it turn out that unfortunately, the used spack commit in JULEA is too old; even the updated one from commit https://github.com/julea-io/julea/commit/36775dbc4da69598f5d4f19e00cb59e6b3e5b757. The following patch for MySQL is required to build it: https://github.com/mysql/mysql-server/commit/bd96c1b542b353d1dfdfd9e0df058f352fc7bc1a

Side note: It is also necessary to explicitely use cxxstd=17 as described in https://github.com/spack/spack/pull/31300

Current solution:

michaelkuhn commented 1 year ago

Which version of mysql are you trying to install? The Spack commit we use is from 2022-07-22 and there haven't been any changes to mysql since then. The patch you mentioned should be included in the newer versions (8.0.24+) already. At least mysql+client_only cxxstd=17 builds for me without the patch.

Moreover, JULEA itself should be using mariadb-c-client instead of mysql, right?

kiraduwe commented 1 year ago

Which version of mysql are you trying to install? The Spack commit we use is from 2022-07-22 and there haven't been any changes to mysql since then. The patch you mentioned should be included in the newer versions (8.0.24+) already. At least mysql+client_only cxxstd=17 builds for me without the patch.

Seems to be version 8.0.29. I will try that when I have time. Maybe specifiying cxxstd=17 is indeed sufficient.

Moreover, JULEA itself should be using mariadb-c-client instead of mysql, right?

That is right. This is only for my convenience. For example, to ensure an empty database I need to drop the tables in between runs from my script and wanted to avoid having to enter the singularity container for that. Also, connecting to the database interactively is otherwise only possible through singularity shell instead of mysql -u user....

kiraduwe commented 1 year ago

Which version of mysql are you trying to install? The Spack commit we use is from 2022-07-22 and there haven't been any changes to mysql since then. The patch you mentioned should be included in the newer versions (8.0.24+) already. At least mysql+client_only cxxstd=17 builds for me without the patch.

You're right :-) Great, then everything is fixed. Having multiple JULEA versions has led to some confusion and I must have forgotten to delete the dependencies folder after pulling the latest commit. That's why the errors where still there.