Skip to content

Commit 1bf4621

Browse files
committed
Implemented: FS#2131 - skip network reconfigure requests on mirror servers that are for a different server_id
1 parent 9db5672 commit 1bf4621

File tree

2 files changed

+17
-5
lines changed

2 files changed

+17
-5
lines changed

server/lib/classes/modules.inc.php

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -108,16 +108,24 @@ function processDatalog() {
108108
*/
109109

110110
$replication_error = false;
111+
$data['mirrored'] = false;
111112

112113
$this->current_datalog_id = $d['datalog_id'];
113114

114115
/*
115116
* If we are in a mirror setup, rewrite the server_id of records that originally
116117
* belonged to the mirrored server to the local server_id
117118
*/
119+
118120
if($conf['mirror_server_id'] > 0 && $d['dbtable'] != 'server') {
119-
if(isset($data['new']['server_id']) && $data['new']['server_id'] == $conf['mirror_server_id']) $data['new']['server_id'] = $conf['server_id'];
120-
if(isset($data['old']['server_id']) && $data['old']['server_id'] == $conf['mirror_server_id']) $data['old']['server_id'] = $conf['server_id'];
121+
if(isset($data['new']['server_id']) && $data['new']['server_id'] == $conf['mirror_server_id']) {
122+
$data['new']['server_id'] = $conf['server_id'];
123+
$data['mirrored'] = true;
124+
}
125+
if(isset($data['old']['server_id']) && $data['old']['server_id'] == $conf['mirror_server_id']) {
126+
$data['old']['server_id'] = $conf['server_id'];
127+
$data['mirrored'] = true;
128+
}
121129
}
122130

123131
if(count($data['new']) > 0) {
@@ -217,6 +225,9 @@ function processDatalog() {
217225
}
218226
*/
219227

228+
//* Data on a single server is never mirrored
229+
$data['mirrored'] = false;
230+
220231
$this->current_datalog_id = $d['datalog_id'];
221232
if(is_array($data['old']) || is_array($data['new'])) {
222233
$this->raiseTableHook($d['dbtable'],$d['action'],$data);

server/plugins-available/network_settings_plugin.inc.php

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,9 @@ function update($event_name,$data) {
7979
$app->uses('getconf');
8080
$server_config = $app->getconf->get_server_config($conf['server_id'], 'server');
8181

82-
// Configure the debian network card settings
83-
if($server_config['auto_network_configuration'] == 'y') {
82+
//* Configure the debian network card settings.
83+
//* Dont configure
84+
if($server_config['auto_network_configuration'] == 'y' && $data['mirrored'] == false) {
8485

8586
if (is_file('/etc/debian_version'))
8687
{
@@ -238,7 +239,7 @@ function update($event_name,$data) {
238239
}
239240

240241
} else {
241-
$app->log('Network configuration disabled in server settings.',LOGLEVEL_DEBUG);
242+
$app->log('Network configuration disabled in server settings or server is a mirror server.',LOGLEVEL_DEBUG);
242243
}
243244

244245
}

0 commit comments

Comments
 (0)