[Pkg-freeradius-maintainers] Bug#920345: Bug#920345: freeradius: running freeradius with debug(-X) option breaks systemd unit

[Mariusz Gronczewski]

Well, there is, adding -f option by default means it will always run
in foreground, regardless of whether -X is used or not

On Thu, 24 Jan 2019 at 15:58, Michael Stapelberg <stapelberg at debian.org> wrote:
>
> The -X option is special in that it changes the way freeradius starts up.
>
> It’s expected that if your actions break the contract with the init system (in this case by specifying -X), you’re responsible to rectify that.
>
> If there was a simple way to make the package work in any/all cases, it’d be up for it, but I’m not aware of one. Hence my question for what your suggestion is — I just don’t see a way out of this situation.
>
> My personal recommendation is to not use /etc/default, but rather systemctl edit to do any overrides, but that’s not Debian’s official line.
>
> On Thu, Jan 24, 2019 at 3:37 PM Mariusz Gronczewski <xani666 at gmail.com> wrote:
>>
>> In our case it was enough to add dropin changing execstart to include
>> -f and type to simple:
>>
>> ExecStart=/usr/sbin/freeradius -f $FREERADIUS_OPTIONS
>> Type=simple
>>
>> What do you mean by "it is expected" ? Currently enabling debug (by
>> setting FREERADIUS_OPTIONS="-X") cause it to enter restart loop,
>> surely that isn't an expected behaviour ?
>>
>> On Thu, 24 Jan 2019 at 13:52, Michael Stapelberg <stapelberg at debian.org> wrote:
>> >
>> > Yes, this is expected. What change are you suggesting?
>> >
>> > On Thu, Jan 24, 2019 at 1:45 PM Mariusz Gronczewski <xani666 at gmail.com> wrote:
>> >>
>> >> Package: freeradius
>> >> Version: 3.0.12+dfsg-5+deb9u1
>> >> Severity: normal
>> >>
>> >> Currently the type of systemd service is forking.
>> >>
>> >> Adding debug to cmdline causes freeradius to run in foreground (and dump debug
>> >> to stdout), which means systemd timeouts on starting service because it assumes
>> >> it will fork.
>> >>
>> >> Changing type to simple, and adding -f (run in foreground) option in unit file
>> >> fixes that
>> >>
>> >>
>> >>
>> >>
>> >> --
>> >> Mariusz Gronczewski (XANi) <xani666 at gmail.com>
>> >> GnuPG: 0xEA8ACE64
>> >>
>> >> _______________________________________________
>> >> Pkg-freeradius-maintainers mailing list
>> >> Pkg-freeradius-maintainers at alioth-lists.debian.net
>> >> https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/pkg-freeradius-maintainers
>> >
>> >
>> >
>> > --
>> > Best regards,
>> > Michael
>>
>>
>>
>> --
>> Mariusz Gronczewski (XANi) <xani666 at gmail.com>
>> GnuPG: 0xEA8ACE64
>
>
>
> --
> Best regards,
> Michael



-- 
Mariusz Gronczewski (XANi) <xani666 at gmail.com>
GnuPG: 0xEA8ACE64