-
-
Notifications
You must be signed in to change notification settings - Fork 404
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
Provide additional admin email notifications #4385
Conversation
cactid.php
Outdated
@@ -41,6 +41,7 @@ function sig_handler($signo) { | |||
case SIGTERM: | |||
case SIGINT: | |||
cacti_log('WARNING: Cacti Daemon PID[' . getmypid() . '] Terminated on Device[' . gethostname() . ']', true, 'CACTID'); | |||
admin_email(__('Cacti System Warning'), __('WARNING: Cacti Daemon PID[' . getmypid() . '] Terminated on Device[' . gethostname() . ']', true, 'CACTID')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have used spaces instead of tabs.
CHANGELOG
Outdated
@@ -8,6 +8,7 @@ Cacti CHANGELOG | |||
-issue#4381: Fatal Errors can occur if you are not in a transaction and you attempt to commit one | |||
-issue#4383: Fixes error callbacks for php 8.0+ | |||
-feature#4344: Add a 15 minute poller interval to global_arrays.php | |||
-feature#4385 Add additional admin email notifications |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing colon :)
@@ -797,6 +797,7 @@ function sig_handler($signo) { | |||
} | |||
} else { | |||
cacti_log('WARNING: Cacti Polling Cycle Exceeded Poller Interval by ' . ($loop_end-$loop_start-$poller_interval) . ' seconds', true, 'POLLER', $level); | |||
admin_email(__('Cacti System Warning'), __('WARNING: Cacti Polling Cycle Exceeded Poller Interval by ' . ($loop_end-$loop_start-$poller_interval) . ' seconds', true, 'POLLER', $level)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have used spaces instead of tabs.
poller_automation.php
Outdated
@@ -50,9 +50,12 @@ | |||
db_force_remote_cnn(); | |||
} else { | |||
cacti_log('WARNING: Main Cacti database offline or in recovery. Can not run automation', false, 'AUTOM8'); | |||
admin_email(__('Cacti System Warning'), __('WARNING: Main Cacti database offline or in recovery')); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have used spaces instead of tabs.
poller_automation.php
Outdated
exit(1); | ||
} | ||
} | ||
#} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a reason for this comment character ?
poller_boost.php
Outdated
@@ -307,6 +307,8 @@ function boost_prepare_process_table() { | |||
/* if the runtime was exceeded, allow the next process to run */ | |||
if ($previous_start_time + $max_run_duration < $start_time) { | |||
cacti_log('WARNING: Detected Poller Boost Overrun, Possible Boost Poller Crash', false, 'BOOST SVR'); | |||
admin_email(__('Cacti System Warning'), __('WARNING: Detected Poller Boost Overrun, Possible Boost Poller Crash', 'BOOST SVR' )); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You have used spaces instead of tabs.
@netniV all fixed |
Add additional admin email alerts