search

geerlingguy / ansible-role-mysql

Ansible Role - MySQL
https://galaxy.ansible.com/geerlingguy/mysql/
MIT License
1.05k stars 862 forks source link

Fixes geerlingguy#440: move syslog config from [mysqld] to [mysqld_safe] #441

Closed kraxor closed 2 years ago

kraxor commented 3 years ago

Fixes #440 where setting mysql_log: syslog causes mysql to fail to start

stale[bot] commented 3 years ago

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

bmillemathias commented 3 years ago

Any news for this fix ? setting mysql_log: syslog as of now breaks mysql service because the instruction is wrong.

stale[bot] commented 3 years ago

This issue is no longer marked for closure.

kraxor commented 3 years ago

@bmillemathias

In your requirements.yml you can replace

roles:
  - geerlingguy.mysql

with

roles:
  - src: https://github.com/kraxor/ansible-role-mysql
    version: master
    name: geerlingguy.mysql

for a drop-in replacement (i.e. no other change is necessary) until this pull request is approved.

(TBH I don't think it will be ever approved, as Jeff Geerling told the world he doesn't have the time to maintain his repos, nor is he willing to involve other people in the maintenance, because these repos contribute to his income. Not kidding.)

stale[bot] commented 2 years ago

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

kraxor commented 2 years ago

You're stale.

stale[bot] commented 2 years ago

This issue is no longer marked for closure.

stale[bot] commented 2 years ago

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

stale[bot] commented 2 years ago

This pull request has been closed due to inactivity. If you feel this is in error, please reopen the pull request or file a new PR with the relevant details.