Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update logrotate in Ubuntu for mysql.err #540

Open
misilot opened this issue May 9, 2024 · 0 comments · May be fixed by #541
Open

Update logrotate in Ubuntu for mysql.err #540

misilot opened this issue May 9, 2024 · 0 comments · May be fixed by #541

Comments

@misilot
Copy link
Contributor

misilot commented May 9, 2024

The error messages in at least Ubuntu 20.04 for mysql get logged to syslog, so the mysql.err log is not handled by logrotate when this role is installed.

looking at /etc/logrotate.d/mysql-server is mentions

The error log is obsolete, messages go to syslog now.

# - I put everything in one block and added sharedscripts, so that mysql gets
#   flush-logs'd only once.
#   Else the binary logs would automatically increase by n times every day.
# - The error log is obsolete, messages go to syslog now.
/var/log/mysql.log /var/log/mysql/*log {
	daily
	rotate 7
	missingok
	create 640 mysql adm
	compress
	sharedscripts
	postrotate
		test -x /usr/bin/mysqladmin || exit 0
		# If this fails, check debian.conf!
		MYADMIN="/usr/bin/mysqladmin --defaults-file=/etc/mysql/debian.cnf"
		if [ -z "`$MYADMIN ping 2>/dev/null`" ]; then
		  # Really no mysqld or rather a missing debian-sys-maint user?
		  # If this occurs and is not a error please report a bug.
		  #if ps cax | grep -q mysqld; then
		  if killall -q -s0 -umysql mysqld; then
 		    exit 1
		  fi
		else
		  $MYADMIN flush-logs
		fi
	endscript
}
misilot added a commit to misilot/ansible-role-mysql that referenced this issue May 10, 2024
This allows it to be managed by the default logrotate configuration

Fixes geerlingguy#540
@misilot misilot linked a pull request May 10, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant