initscript doesn't preserve return code

Bug #1464770 reported by Simon Déziel
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
nginx (Debian)
Fix Released
Unknown
nginx (Ubuntu)
Fix Released
Low
Unassigned
Wily
Fix Released
Low
Unassigned

Bug Description

Copied from the Debian bug (https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=788573):

Many actions offered by the initscript do not preserve or give the
proper return code. One notable example is the "configtest" that always
returns 0 even when problems are detected:

 echo "invalid" >> /etc/nging/nginx.conf
 service nginx configtest
 echo $?

Should return 1 but returns 0 instead.

Tags: patch
Revision history for this message
Simon Déziel (sdeziel) wrote :
no longer affects: nginx
Thomas Ward (teward)
Changed in nginx (Ubuntu):
importance: Undecided → Low
status: New → Triaged
Changed in nginx (Debian):
status: Unknown → New
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "Same patch as the one attached to the Debian bug" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Simon Déziel (sdeziel)
Changed in nginx (Ubuntu Wily):
status: Triaged → Fix Committed
Revision history for this message
Simon Déziel (sdeziel) wrote : Re: [Bug 1464770] Re: initscript doesn't preserve return code

On 06/16/2015 11:57 AM, Simon Déziel wrote:
> ** Changed in: nginx (Ubuntu Wily)
> Status: Triaged => Fix Committed

Oops, I might have been wrong as the fix was commit in Debian but have
yet to make it to Wily. Sorry about that :(

Revision history for this message
Thomas Ward (teward) wrote :

Simon: Yeah, the system will autoupdate the Debian bug status, however currently the Ubuntu one should stay "Triaged" - we have a couple other things as the Server Team to make some discussions and decisions first.

Changed in nginx (Ubuntu Wily):
status: Fix Committed → Triaged
Changed in nginx (Debian):
status: New → Fix Committed
Changed in nginx (Debian):
status: Fix Committed → Fix Released
Revision history for this message
Simon Déziel (sdeziel) wrote :

Thomas, now that the fix made it to Wily, would there be any chance to get this to Trusty via an SRU? Thanks in advance

Changed in nginx (Ubuntu Wily):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.