search

cherti / mailexporter

Export Prometheus-style metrics about mail server functionality
https://prometheus.io
GNU General Public License v3.0
45 stars 9 forks source link

randomized duration between the monitoring jobs #19

Closed el-if closed 7 years ago

cherti commented 7 years ago

Thinking about it, I think it would be nicer to actually move the delay to the beginning of the monitor-function itself. Can you just move the time.Sleep-call to the beginning of the monitor-function?

cherti commented 7 years ago

Additionally, as we now don't have any startupOffset anymore, that section has to go from the mailexporter.conf as well.

cherti commented 7 years ago

Ok, I think now that the startMonitoringJobs-function doesn't block anymore, because the monitor-functions do that on their own, it would be best if we got rid of that function altogether. Can you move the contents of the startMonitoringJobs-function back to the main-function and remove it?

cherti commented 7 years ago

Thanks!