1
0
mirror of https://github.com/mariadb-corporation/mariadb-columnstore-engine.git synced 2025-11-21 09:20:51 +03:00

Allow both mysql and mariadb as owners of the data dir

This commit is contained in:
Alexander Presnyakov
2025-10-28 17:13:46 +00:00
parent d96ed68824
commit 7ff93ba16f
4 changed files with 13 additions and 9 deletions

View File

@@ -453,7 +453,7 @@ class ConfigController:
if diag: if diag:
raise_422_error( raise_422_error(
module_logger, func_name, module_logger, func_name,
f'Invariant checks failed. Details: {diag.strip()}', f'Invariant checks failed. Details:\n{diag.strip()}',
exc_info=False exc_info=False
) )

View File

@@ -2,8 +2,8 @@ import logging
import os import os
from typing import Optional, Tuple from typing import Optional, Tuple
from mr_kot import Runner, Status, check, check_all, fact, parametrize from mr_kot import Runner, Status, any_of, check, check_all, fact, parametrize
from mr_kot_fs_validators import GroupIs, HasMode, IsDir, OwnerIs from mr_kot_fs_validators import Exists, GroupIs, HasMode, IsDir, OwnerIs
from cmapi_server import helpers from cmapi_server import helpers
from cmapi_server.constants import MCS_DATA_PATH from cmapi_server.constants import MCS_DATA_PATH
@@ -76,6 +76,7 @@ REQUIRED_LOCAL_DIRS = [
def required_dirs_perms(dir: str) -> Tuple[Status, str]: def required_dirs_perms(dir: str) -> Tuple[Status, str]:
status, ev = check_all( status, ev = check_all(
dir, dir,
Exists(),
IsDir(), IsDir(),
HasMode('1755'), HasMode('1755'),
) )
@@ -87,7 +88,10 @@ def required_dirs_ownership(dir: str) -> Tuple[Status, str]:
# Check ownership only when not in containers # Check ownership only when not in containers
status, ev = check_all( status, ev = check_all(
dir, dir,
OwnerIs('mysql'), Exists(),
GroupIs('mysql'), # The correct owner is mysql, but i've seen mariadb as owner of the mountpoint,
# so we allow both
any_of(OwnerIs('mysql'), OwnerIs('mariadb')),
any_of(GroupIs('mysql'), GroupIs('mariadb')),
) )
return (status, ev) return (status, ev)

View File

@@ -19,5 +19,5 @@ typer==0.15.2
pydantic==2.11.7 pydantic==2.11.7
sentry-sdk==2.34.1 sentry-sdk==2.34.1
# Invariant checks # Invariant checks
mr_kot==0.8.5 mr_kot==0.9.2
mr_kot_fs_validators==0.1.0 mr_kot_fs_validators==0.2.0

View File

@@ -144,9 +144,9 @@ more-itertools==10.7.0
# cherrypy # cherrypy
# jaraco-functools # jaraco-functools
# jaraco-text # jaraco-text
mr-kot==0.8.5 mr-kot==0.9.2
# via -r requirements.in # via -r requirements.in
mr-kot-fs-validators==0.1.0 mr-kot-fs-validators==0.2.0
# via -r requirements.in # via -r requirements.in
multidict==6.6.4 multidict==6.6.4
# via # via