evolix
/
ansible-roles
22
2
Fork 2
Code Issues 61 Pull Requests 18 Releases 32 Wiki Activity

packweb-multi-php: Allow php to contact local mysql with localhost #116

Merged
lpoujol merged 9 commits from mtrossevin/ansible-roles:packweb-multi-php2 into unstable 2020-10-26 09:29:20 +01:00
Conversation 2 Commits 9 Files Changed 7 +36 -4
mtrossevin commented 2020-10-19 17:46:03 +02:00
Owner
Copy Link

PHP translate localhost to a socket path when connecting to a mysql/mariadb database. This break when using packweb-multi-php as the socket isn't present in the containers.

This PR create a bind mount of the host /run/mysqld (=/var/run/mysqld) where this socket is present by default for the mariadb package in debian. It also change the default configuration of mysql so that it is also where the socket is put instead of /tmp/mysql.sock (the default with Oracle MySQL).

This PR was tested with an evolixised VM running Debian 10 (It worked !).

PHP translate localhost to a socket path when connecting to a mysql/mariadb database. This break when using packweb-multi-php as the socket isn't present in the containers. This PR create a bind mount of the host /run/mysqld (=/var/run/mysqld) where this socket is present by default for the mariadb package in debian. It also change the default configuration of mysql so that it is also where the socket is put instead of /tmp/mysql.sock (the default with Oracle MySQL). This PR was tested with an evolixised VM running Debian 10 (It worked !).
mtrossevin added the
enhancement
 label 2020-10-19 17:46:05 +02:00
lpoujol commented 2020-10-23 11:58:03 +02:00
Owner
Copy Link

LGTM

Just don't forget to also update the config for mysql client if you change the path of the UNIX socket in the role mysql-oracle

Gif tamponné

LGTM Just don't forget to also update the config for mysql client if you change the path of the UNIX socket in the role mysql-oracle ![Gif tamponné](https://media1.tenor.com/images/2cfad842ff3c7b9cca31d98d64a6c820/tenor.gif?itemid=17182961)
mtrossevin commented 2020-10-23 13:07:44 +02:00
Author
Owner
Copy Link

Just don’t forget to also update the config for mysql client if you change the path of the UNIX socket in the role mysql-oracle

Done

Cc: @lpoujol

> Just don’t forget to also update the config for mysql client if you change the path of the UNIX socket in the role mysql-oracle Done Cc: @lpoujol
lpoujol merged commit 82d9abca3d into unstable 2020-10-26 09:29:18 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something is not working

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
help wanted

Need some help

  
invalid

Something is wrong

  
question

More information is needed

  
security

Concern a security issue

  
wontfix

This won't be fixed

No Label
bug
duplicate
enhancement
help wanted
invalid
question
security
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: evolix/ansible-roles#116
No description provided.
Delete Branch "mtrossevin/ansible-roles:packweb-multi-php2"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?