Skip to content

fix(jobs): replaced deprecated command argument --no-mariadb-socket by the recommanded --mariadb-user-host-login-scope='%' (wildcard)#261

Open
fjudith wants to merge 1 commit intofrappe:mainfrom
alithya-oss:fix/create-site
Open

fix(jobs): replaced deprecated command argument --no-mariadb-socket by the recommanded --mariadb-user-host-login-scope='%' (wildcard)#261
fjudith wants to merge 1 commit intofrappe:mainfrom
alithya-oss:fix/create-site

Conversation

@fjudith
Copy link
Copy Markdown

@fjudith fjudith commented Dec 16, 2025

Please provide enough information so that others can review your pull request:

Explain the details for making this change. What existing problem does the pull request solve?

The create-site uses the deprecated --no-mariadb-socket command line argument.
Using the recommended --mariadb-user-host-login-scope='%' has been tested successfully

error message: --no-mariadb-socket is DEPRECATED; use --mariadb-user-host-login-scope='%' (wildcard) or --mariadb-user-host-login-scope=, instead. The name of this option was misleading: it had nothing to do with sockets.

… by the recommanded `--mariadb-user-host-login-scope='%'` (wildcard)
@ismxilxrif
Copy link
Copy Markdown
Contributor

if i'm not mistaken, this would break v14 and earlier deployments, if any, unless, we drop support for those versions, which idk if it's unlikely or not.

@revant
Copy link
Copy Markdown
Collaborator

revant commented Feb 4, 2026

Can you resolve conflict?

https://discuss.frappe.io/t/reminder-erpnext-v14-has-reached-end-of-life/160084

We can merge I guess

@ismxilxrif
Copy link
Copy Markdown
Contributor

Can we make it adjustable with a key that lets us choose which route this would take? That way we won’t break existing installations, although given the nature of it, this is really only used for new sites anyway. Still, I believe we should always maintain compatibility. just my 2 cents

@github-actions
Copy link
Copy Markdown
Contributor

This PR has been automatically marked as stale. You have a week to explain why you believe this is an error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants