Nginx fails to start on 13.04

Bug #1174158 reported by shrinath_m2
32
This bug affects 6 people
Affects Status Importance Assigned to Milestone
nginx (Ubuntu)
Fix Released
Medium
Adam Conrad
Raring
Fix Released
Medium
Adam Conrad

Bug Description

[Impact]

 * This bug prevents nginx-extras from running correctly in Raring and later.

[Test Case]

 * `nginx -t` from the command line, as superuser if it says it needs it,
   will allow you to test and see whether this bug is fixed in your installation

[Regression Potential]

 * There is zero regression potential from this fix.

[Other Info]

 * No additional info.

------

Original Description:

Update - Problem is with nginx-extras package only. Nginx-full or any other package doesn't have this problem.

`nginx -t` gives the following error message -
`nginx: [alert] version 1.2.6 (Ubuntu) of nginx.pm is required, but 1.2.6 was found`

Fails to start, doesn't give any error message on `service nginx restart`.

Was working on 12.04, 12.10 -> broken when upgraded to 13.04.

ProblemType: Bug
DistroRelease: Ubuntu 13.04
Package: nginx-extras 1.2.6-1ubuntu3
ProcVersionSignature: Ubuntu 3.8.0-19.29-generic 3.8.8
Uname: Linux 3.8.0-19-generic x86_64
ApportVersion: 2.9.2-0ubuntu8
Architecture: amd64
Date: Mon Apr 29 10:43:24 2013
InstallationDate: Installed on 2012-09-27 (213 days ago)
InstallationMedia: Ubuntu 12.04 LTS "Precise Pangolin" - Release amd64 (20120425)
MarkForUpload: True
ProcEnviron:
 LANGUAGE=en_IN:en
 PATH=(custom, no user)
 XDG_RUNTIME_DIR=<set>
 LANG=en_IN
 SHELL=/bin/bash
SourcePackage: nginx
UpgradeStatus: Upgraded to raring on 2013-04-26 (2 days ago)

Related branches

Revision history for this message
shrinath_m2 (shrinath-m2) wrote :
Revision history for this message
David Bennett (davidbennett) wrote :

changing nxinx-extra to nginx-full seems so make nginx work again :)

This worked for me:
$ sudo apt-get install nginx-full

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in nginx (Ubuntu):
status: New → Confirmed
Revision history for this message
shrinath_m2 (shrinath-m2) wrote :

@davidbennett, nginx-full doesn't help with lua scripting. There are many other things that come with extras, but not with "full".

description: updated
Revision history for this message
Paulus (paulmd-ya) wrote :

The bug is within the version string, so it would run with the string fixed.

You may use the patch attached (place in debian/patches in the build tree, add it to the debian/patches/series) and recompile.

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

I'll take a look at this, and probably issue an SRU (Stable Release Update) for this within the next day or so.

Status being changed to "In Progress" because I'm working on getting this fixed. Importance set to "Medium" because it has a moderate impact on the software, and because there is no fix short of downloading the source yourself and recompiling the software.

Notes: This could be caused by the introduction of a patch to the software that adds Ubuntu branding to the version string. That patch will likely be dropped in the SRU.

Changed in nginx (Ubuntu):
assignee: nobody → Thomas Ward (teward)
importance: Undecided → Medium
status: Confirmed → In Progress
Revision history for this message
Adam Conrad (adconrad) wrote :

This is as a result of this SRU:

http://launchpadlibrarian.net/136741416/nginx_1.2.6-1ubuntu2_1.2.6-1ubuntu3.diff.gz

If that had been the following instead, it would be fine:

#define NGINX_VER "nginx/" NGINX_VERSION " (Ubuntu)"

(Needs testing, regardless, which the previous SRU seems to have lacked)

Revision history for this message
Adam Conrad (adconrad) wrote :

Or, rather, that wasn't an SRU, just a late cycle upload. Got my timing confused.

Thomas Ward (teward)
Changed in nginx (Ubuntu):
status: In Progress → Triaged
Thomas Ward (teward)
Changed in nginx (Ubuntu Raring):
assignee: nobody → Thomas Ward (teward)
importance: Undecided → Medium
status: New → Triaged
Revision history for this message
Adam Conrad (adconrad) wrote :

Here's a proper patch to apply the change described in my previous comment.

Thomas Ward (teward)
Changed in nginx (Ubuntu Raring):
assignee: Thomas Ward (teward) → nobody
Changed in nginx (Ubuntu):
assignee: Thomas Ward (teward) → nobody
Revision history for this message
Adam Conrad (adconrad) wrote :

My above patch was tested in a back-and-forth on IRC and confirmed working, uploaded to both saucy and raring.

Changed in nginx (Ubuntu Raring):
assignee: nobody → Adam Conrad (adconrad)
Changed in nginx (Ubuntu):
assignee: nobody → Adam Conrad (adconrad)
Thomas Ward (teward)
description: updated
Adam Conrad (adconrad)
Changed in nginx (Ubuntu):
status: Triaged → Fix Committed
Changed in nginx (Ubuntu Raring):
status: Triaged → In Progress
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello shrinath_m2, or anyone else affected,

Accepted nginx into raring-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/nginx/1.2.6-1ubuntu3.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in nginx (Ubuntu Raring):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nginx - 1.2.6-1ubuntu4

---------------
nginx (1.2.6-1ubuntu4) saucy; urgency=low

  * debian/patches/ubuntu-branding.patch: Move Ubuntu branding from
    the numerical version string to the long version (LP: #1174158)
 -- Adam Conrad <email address hidden> Thu, 09 May 2013 11:50:52 -0600

Changed in nginx (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
shrinath_m2 (shrinath-m2) wrote :

Verified with version '1.2.6-1ubuntu3.1' (Ubuntu:13.04/raring-proposed [amd64]) for 'nginx-extras'.

Doing sudo nginx -t did not report any issues. Proposed package is good to go live.

tags: removed: verification-needed
tags: added: verification-done
Revision history for this message
Scott Kitterman (kitterman) wrote :

Based on the severity of the issue and the obviousness of the fix I'll waive the normal 7 day bake in time for this update.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nginx - 1.2.6-1ubuntu3.1

---------------
nginx (1.2.6-1ubuntu3.1) raring; urgency=low

  * debian/patches/ubuntu-branding.patch: Move Ubuntu branding from
    the numerical version string to the long version (LP: #1174158)
 -- Adam Conrad <email address hidden> Thu, 09 May 2013 11:50:52 -0600

Changed in nginx (Ubuntu Raring):
status: Fix Committed → Fix Released
Revision history for this message
Scott Kitterman (kitterman) wrote : Update Released

The verification of this Stable Release Update has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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