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

Bug #494499 reported by Thierry Carrez on 2009-12-09
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
update-manager (Ubuntu)
Medium
Michael Vogt
Lucid
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: added: iso-testing
Changed in update-manager (Ubuntu Lucid):
status: New → Confirmed
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)
Starbetrayer (starbetrayer) wrote :

I have this one

Thierry Carrez (ttx) on 2009-12-18
Changed in update-manager (Ubuntu Lucid):
importance: Undecided → Medium
Michael Vogt (mvo) on 2010-01-08
Changed in update-manager (Ubuntu Lucid):
assignee: Chuck Short (zulcss) → Michael Vogt (mvo)
status: Confirmed → In Progress
Michael Vogt (mvo) on 2010-01-08
Changed in update-manager (Ubuntu Lucid):
status: In Progress → Fix Committed
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  Edit
Everyone can see this information.

Other bug subscribers