Hi,
I've upgraded my logstash deployments multiple times over the years (starting way back in the 1.x days), and I had set Logstash to check once a minute for config changes (auto-reload) in the logstash.yml file.
I recently noticed that some of my logstash (v8.12) systems were using a full CPU core's worth of processing while they were doing nothing. Digging into the issue, I discovered that they were constantly (multiple times a second) checking for new configs.
After searching online a while, I see that the semantics of config.reload.interval have subtly changed: now the interval is required to have an "s" after the number to indicate seconds. My config just had the bare number.
From the 8.12 Logstash documentation https://www.elastic.co/guide/en/logstash/current/reloading-config.html, it implies that the "s" unit qualifier is required. Shouldn't Logstash configs lacking this qualifier either output a warning message or error out entirely (that's what is implied by the documentation at the link)?
Thanks,
- Daniel