FS#31309 - [icecast] systemd service file is broken

Attached to Project: Community Packages
Opened by Jonathan (Da_Coynul) - Monday, 27 August 2012, 23:06 GMT
Last edited by Lukas Fleischer (lfleischer) - Friday, 08 February 2013, 17:32 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Lukas Fleischer (lfleischer)
Architecture All
Severity Medium
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 5
Private No

Details

Description:

icecast.service should include Type=forking in the [Service] section. Without this, the service will die immediately after starting.

Steps to reproduce:

#systemctl start icecast.service

#systemctl status icecast.service
icecast.service - Icecast Network Audio Streaming Server
Loaded: loaded (/usr/lib/systemd/system/icecast.service; enabled)
Active: inactive (dead) since Mon, 27 Aug 2012 18:17:08 -0400; 15s ago
Process: 18733 ExecStart=/usr/bin/icecast -b -c /etc/icecast.xml (code=exited, status=0/SUCCESS)
CGroup: name=systemd:/system/icecast.service
This task depends upon

Closed by  Lukas Fleischer (lfleischer)
Friday, 08 February 2013, 17:32 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in 2.3.3-3.
Comment by ianux (ianux) - Saturday, 20 October 2012, 19:18 GMT
Speaking of that, is there a reason to use -b parameter and use Type=forking ?
Systemd is supposed to fork daemons by itself, assign a PID to it, etc.
I use a custom service file with no -b parameter and everything is going fine.
Comment by Jonathan (Da_Coynul) - Tuesday, 18 December 2012, 16:58 GMT
Today I realized this is broken again despite the Type=forking modification. Now the service starts and ultimately times out even though the daemon is running properly. After some trial and error I came up with a new, working icecast.service file and have attached it for anyone who is interested. BTW @ianux, this would not work for me without the -b parameter.
   icecast.service (0.2 KiB)
Comment by Mantas Mikulėnas (grawity) - Thursday, 20 December 2012, 12:05 GMT
Works fine for me with "Type=forking" added, with the default Icecast configuration. Specifying PIDFile should be unnecessary.
Comment by Green (The_Green_Arrow) - Thursday, 20 December 2012, 21:34 GMT
Works fine for me too ;)
Comment by Jonathan (Da_Coynul) - Friday, 21 December 2012, 01:54 GMT
After a third removal and reinstall of icecast it is working again for me.
Comment by Fabian Stegemann (zetaron) - Tuesday, 05 February 2013, 21:41 GMT
Worked for me after removing the -b parameter from the service file.
Comment by Filip (mezcal) - Thursday, 07 February 2013, 08:02 GMT
Deleting -b works like a charm. There is also duplicate bug report https://bugs.archlinux.org/task/33729?project=5&cat%5B0%5D=33&string=icecast .
I think the package should be fixed.

Loading...