MOTD shows "run-parts: /etc/update-motd.d/91-release-upgrade exited with return code 1"

Bug #494499 reported by Thierry Carrez
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
update-manager (Ubuntu)
Fix Released
Medium
Michael Vogt
Lucid
Fix Released
Medium
Michael Vogt

Bug Description

Binary package hint: update-manager

Lucid alpha1 server ISO testing. A superfluous and annoying message is displayed on login:

ubuntu login: ubuntu
Password:
run-parts: /etc/update-motd.d/91-release-upgrade exited with return code 1
Linux ubuntu...

This is because /etc/update-motd.d/91-release-upgrade returns 1 when there is nothing to upgrade to. If there is no side-effect, it should be fixed to return 0 in all "normal" cases. Other solution is for update-motd to catch run-parts error output.

Tags: iso-testing
tags: added: iso-testing
Changed in update-manager (Ubuntu Lucid):
status: New → Confirmed
Revision history for this message
Jos Boumans (jib) wrote :

Chuck, can you please propose a fix and bring it to the attention of the foundations team?

That way they can make sure it doesn't break anything else.

Thanks,

Changed in update-manager (Ubuntu Lucid):
assignee: nobody → Chuck Short (zulcss)
Revision history for this message
Starbetrayer (starbetrayer) wrote :

I have this one

Thierry Carrez (ttx)
Changed in update-manager (Ubuntu Lucid):
importance: Undecided → Medium
Michael Vogt (mvo)
Changed in update-manager (Ubuntu Lucid):
assignee: Chuck Short (zulcss) → Michael Vogt (mvo)
status: Confirmed → In Progress
Michael Vogt (mvo)
Changed in update-manager (Ubuntu Lucid):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package update-manager - 1:0.131.3

---------------
update-manager (1:0.131.3) lucid; urgency=low

  * DistUpgrade/DistUpgradeController.py:
    - add missing check for ubuntu-security when testing for
      mirrors (thanks to Stuart Langridge)
  * DistUpgrade/DistUpgrade.cfg:
    - add example SlideshowUrl
  * DistUpgrade/DistUpgradeController.py, DistUpgrade/DistUpgradeView.py,
    DistUpgrade/DistUpgradeViewGtk.py:
    - add slideshow support based on webkit
    - call percent() JS method on the webkit view
  * check-new-release:
    - use exit codes if run with --quiet otherwise only print
      (LP: #494499)
  * DistUpgrade/DistUpgradeConfigParser.py:
    - fix crash in _interpolate (LP: #500705)
  * UpdateManager/Core/MetaRelease.py:
    - do not crash on stat failure (LP: #496144)
  * UpdateManager/Core/MyCache.py:
    - do not crash if the lock can not be released (LP: #410574)
  * UpdateManager/SafeGConfClient.py
    - implement gconfclient that does not crash if gconf is not
      working (LP: #261471)
 -- Michael Vogt <email address hidden> Tue, 12 Jan 2010 18:15:52 +0100

Changed in update-manager (Ubuntu Lucid):
status: Fix Committed → 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.