[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
Proposed debdiff All code inside the init script tests if status returns 0 or non-zero so it is not affected. If there is anything that relies on asterisk returning the wrong code, then it must be fixed... asterisk (1:1.4.21.2~dfsg-1ubuntu3) intrepid; urgency=low * debian/asterisk.init: Fix status action so that it returns the LSB-compliant return codes (LP: #248947) * debian/control: added lsb-base dependency for using status_of_proc -- Thierry Carrez [EMAIL PROTECTED] Mon, 29 Sep 2008 14:21:59 +0200 ** Attachment added: asterisk_1.4.21.2~dfsg-1ubuntu3.debdiff http://launchpadlibrarian.net/18034696/asterisk_1.4.21.2%7Edfsg-1ubuntu3.debdiff ** Changed in: asterisk (Ubuntu) Importance: Undecided = Low Status: New = Confirmed -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
Feature Freeze Exception requested for Thierry's attached patch. The change is an improvement of the asterisk init script status action, which currently is a pretty crufty ps | awk, and instead uses the LSB's status_of_proc() function. This fix is required for High Availability heartbeat pings, such as linux-ha. Diffstat says: asterisk.init | 11 +++ changelog |8 control |2 +- 3 files changed, 12 insertions(+), 9 deletions(-) This change builds successfully on all platforms, as it's only a change to shell code. It installs and upgrades successfully, in that it's changing a conffile managed by the package. :-Dustin -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
I have subscribed motu-release to this bug, in order to process the Feature Freeze Exception noted in my previous comment. Thanks, :-Dustin -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
Hi Dustin, up to FinalFreeze, you need a freeze exception only for (non-bugfix) new upstream versions, or if you really want to do an invasive change in an upload (e.g. like causing a transition). Unsubscribing motu-release again, no need for an exception ;) Cheers, Stefan. -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
This bug was fixed in the package asterisk - 1:1.4.21.2~dfsg-1ubuntu3 --- asterisk (1:1.4.21.2~dfsg-1ubuntu3) intrepid; urgency=low * debian/asterisk.init: Fix status action so that it returns the LSB-compliant return codes (LP: #248947) * debian/control: added lsb-base dependency for using status_of_proc -- Thierry Carrez [EMAIL PROTECTED] Mon, 29 Sep 2008 14:21:59 +0200 ** Changed in: asterisk (Ubuntu) Status: Confirmed = Fix Released -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
I think that it would not be a good idea to continue to use awk to detect if the daemon is running. It would in my opinion make more sense to make this code use the status_of_proc() function which is now included in the init functions. -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
I just noticed that you're also suggesting to change the return value from 1 to 3, and while this may be correct from an LSB perspective, I strongly suspect that there are other applications that rely on the return value being 1. It would be prudent to identify is there is any code that is shipped with Ubuntu that relies on the asterisk return value before actually changing this return value. -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 248947] Re: asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha)
See https://wiki.ubuntu.com/InitScriptStatusActions for suggestions on how to implement this. -- asterisk init script is not LSB compliant (for heartbeat, i.e. linux-ha) https://bugs.launchpad.net/bugs/248947 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
