linkedin / Burrow

Kafka Consumer Lag Checking
Apache License 2.0
3.73k stars 797 forks source link

Can't insert PID-file full path in the config file #572

Open cyprix opened 4 years ago

cyprix commented 4 years ago

Burrow version: 1.2.2 Linux 64-bit

I'm trying to pack burrow in RPM. In the config-file I've changed the pidfile path:

[general]
pidfile="/var/run/burrow.pid"
stdout-logfile="/var/log/burrow/burrow.out"
access-control-allow-origin="mysite.example.com"

But service failed with next error:

Cannot interpret contents of PID file: strconv.Atoi: parsing "": invalid syntaxBurrow failed at October 15, 2019 at 11:29am (UTC)

Same error, when I'm trying to run burrow manually:

 /opt/burrow/burrow --config-dir /etc/burrow/
Reading configuration from /etc/burrow/
Cannot interpret contents of PID file: strconv.Atoi: parsing "": invalid syntaxBurrow failed at October 15, 2019 at 11:34am (UTC)
Apsche commented 3 years ago

This issue seems to be abandoned, but for anyone who is getting this error, do not manually create the pid file. This issue happened to me when I manually created an empty file called burrow.pid, and the above error was thrown when attempting to parse an empty pid file. When starting burrow, it will create the pid file automatically and populate it with the process id.