Skip to content

Conversation

@RincewindsHat
Copy link
Member

Fixes #2178

ping @waja

@RincewindsHat
Copy link
Member Author

@waja you need a backport for that?

@waja
Copy link
Member

waja commented Nov 27, 2025

@waja you need a backport for that?

That would be great, if you have the spare time. I'm unfortunately loaded with tons of other things for the next time.

@RincewindsHat
Copy link
Member Author

RincewindsHat commented Nov 27, 2025

303,328c303,316
< 				if (use_deprecated_slave_status) {
< 					if (strcmp(fields[i].name, "Slave_IO_Running") == 0) {
< 						slave_io_field = i;
< 						continue;
< 					}
< 					if (strcmp(fields[i].name, "Slave_SQL_Running") == 0) {
< 						slave_sql_field = i;
< 						continue;
< 					}
< 					if (strcmp(fields[i].name, "Seconds_Behind_Master") == 0) {
< 						seconds_behind_field = i;
< 						continue;
< 					}
< 				} else {
< 					if (strcmp(fields[i].name, "Replica_IO_Running") == 0) {
< 						slave_io_field = i;
< 						continue;
< 					}
< 					if (strcmp(fields[i].name, "Replica_SQL_Running") == 0) {
< 						slave_sql_field = i;
< 						continue;
< 					}
< 					if (strcmp(fields[i].name, "Seconds_Behind_Source") == 0) {
< 						seconds_behind_field = i;
< 						continue;
< 					}
---
> 				if ((strcmp(fields[i].name, "Slave_IO_Running") == 0) ||
> 				    (strcmp(fields[i].name, "Replica_IO_Running") == 0)) {
> 					slave_io_field = i;
> 					continue;
> 				}
> 				if ((strcmp(fields[i].name, "Slave_SQL_Running") == 0) ||
> 				    (strcmp(fields[i].name, "Replica_SQL_Running") == 0)) {
> 					slave_sql_field = i;
> 					continue;
> 				}
> 				if (strcmp(fields[i].name, "Seconds_Behind_Master") == 0) ||
> 					(strcmp(fields[i].name, "Seconds_Behind_Source") == 0)) {
> 					seconds_behind_field = i;
> 					continue;

that is a diff against that debian patch 26.

@RincewindsHat RincewindsHat merged commit 7827b55 into monitoring-plugins:master Nov 27, 2025
7 checks passed
@RincewindsHat RincewindsHat deleted the fix/check_mysql_replica_again branch November 27, 2025 23:30
@waja
Copy link
Member

waja commented Nov 28, 2025

@RincewindsHat

There was a hyphen missing, but I managed to fix on my own. :)

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 this pull request may close these issues.

check_mysql: replication broken on MariaDB 10.11.14-0

2 participants