hcq/MaxScale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix mysqlmon_rejoin_manual

Browse Source 
Added some extra connection refreshing and monitor waiting. Unclear why this
is needed now but it seems to work.
This commit is contained in:
Esa Korhonen
2019-06-11 12:22:00 +00:00
parent 239c4a8615
commit a5898f83f9
1 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
maxscale-system-test/mysqlmon_rejoin_manual.cpp
View File

@ -60,14 +60,14 @@ int main(int argc, char** argv)
print_gtids(test); print_gtids(test);
cout << "Bringing old master back online..." << endl; cout << "Bringing old master back online..." << endl;
test.repl->start_node(master_index, (char*) ""); test.repl->start_node(master_index, (char*) "");
test.maxscales->wait_for_monitor(); test.maxscales->wait_for_monitor(2);
test.repl->connect();
get_output(test); get_output(test);
test.tprintf("and manually rejoining it to cluster."); test.tprintf("and manually rejoining it to cluster.");
const char REJOIN_CMD[] = "maxadmin call command mariadbmon rejoin MySQL-Monitor server1"; const char REJOIN_CMD[] = "maxadmin call command mariadbmon rejoin MySQL-Monitor server1";
test.maxscales->ssh_node_output(0, REJOIN_CMD, true, &ec); test.maxscales->ssh_node_output(0, REJOIN_CMD, true, &ec);
test.maxscales->wait_for_monitor(); test.maxscales->wait_for_monitor(2);
get_output(test); get_output(test);
test.repl->connect();
string gtid_old_master; string gtid_old_master;
test.repl->connect(); test.repl->connect();
@ -88,10 +88,9 @@ int main(int argc, char** argv)
"Old master did not successfully rejoin the cluster (%s != %s).", "Old master did not successfully rejoin the cluster (%s != %s).",
gtid_final.c_str(), gtid_old_master.c_str()); gtid_final.c_str(), gtid_old_master.c_str());
// Switch master back to server1 so last check is faster // Switch master back to server1 so last check is faster
int ec; test.maxscales->ssh_node_output(0,
test.maxscales->ssh_node_output(0, "maxadmin call command mysqlmon switchover " "maxadmin call command mysqlmon switchover MySQL-Monitor server1 server2", true, &ec);
"MySQL-Monitor server1 server2", true, &ec); test.maxscales->wait_for_monitor(2);
test.maxscales->wait_for_monitor();
get_output(test); get_output(test);
master_id = get_master_server_id(test); master_id = get_master_server_id(test);
test.expect(master_id == old_master_id, "Switchover back to server1 failed."); test.expect(master_id == old_master_id, "Switchover back to server1 failed.");
@ -115,10 +114,11 @@ int main(int argc, char** argv)
test.expect(row.empty() || row[0].empty(), test.expect(row.empty() || row[0].empty(),
"server3 gtid is not empty as it should (%s).", row[0].c_str()); "server3 gtid is not empty as it should (%s).", row[0].c_str());
cout << "Rejoining server3.\n"; cout << "Rejoining server3.\n";
test.maxscales->ssh_node_output(0, "maxadmin call command mysqlmon rejoin " test.maxscales->ssh_node_output(0,
"MySQL-Monitor server3", true, &ec); "maxadmin call command mysqlmon rejoin MySQL-Monitor server3", true, &ec);
test.maxscales->wait_for_monitor(); test.maxscales->wait_for_monitor(2);
get_output(test); get_output(test);
test.repl->connect();
char result[100]; char result[100];
test.repl->connect(); test.repl->connect();
if (find_field(conn, sstatus_query.c_str(), "Master_Host", result) == 0) if (find_field(conn, sstatus_query.c_str(), "Master_Host", result) == 0)