<div dir="ltr">Hi,<div><br></div><div>I have a bug[1] on xenial where the user installed freeradius, freeradius-ldap and then disabled the service:</div><div><br></div><div>systemctl stop freeradius.service</div><div>systemctl disable freeradius.service</div><div><br></div><div>When the freeradius packages are upgraded, freeradius-ldap's postinst fails to force-reload the service, because it's not running:</div><div><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">freeradius.service is not active, cannot reload.</span><br style="color:rgb(51,51,51);font-family:monospace;font-size:12px"><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">invoke-rc.d: initscript freeradius, action "force-reload" failed.</span><br style="color:rgb(51,51,51);font-family:monospace;font-size:12px"><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">dpkg: error processing package freeradius-ldap (--configure):</span><br></div><div><br></div><div>In fact, even freeradius' postinst fails to start the service, but since its start invocation ends with "|| true", the error is swallowed. But not in the force-reload case:</div><div><br></div><div>freeradius.postinst:</div><div><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">      </span><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">   invoke-</span><span style="color:rgb(51,51,51);font-family:monospace;font-size:12px">rc.d freeradius $action || true</span><br></div><div><p style="margin:0px 0px 0.8em;padding:0px;width:auto;max-width:45em;color:rgb(51,51,51);font-family:monospace;font-size:12px"><br></p><p style="margin:0px 0px 0.8em;padding:0px;width:auto;max-width:45em">freeradius-ldap.postinst:<br><font color="#333333" face="monospace"><span style="font-size:12px">       </span></font><font color="#333333" face="monospace"><span style="font-size:12px">   invoke-</span></font><font color="#333333" face="monospace"><span style="font-size:12px">rc.d freeradius force-reload</span></font></p></div><div><br></div><div>A trivial workaround would be to add "|| true" to the force-reload line, of course. But even the one around "start" feels bad, because it could mask real errors.</div><div><br></div><div>What should be done here, from a policy perspective? Is there a better pattern out there for this scenario already in some package, or even a debhelper helper?</div><div><br></div><div>Thanks</div><div><br></div><div><br></div><div><br></div><div>1. <a href="https://bugs.launchpad.net/ubuntu/+source/freeradius/+bug/1712817" target="_blank">https://bugs.launchpad.net/<wbr>ubuntu/+source/freeradius/+<wbr>bug/1712817</a></div></div>