update-manager jaunty partial upgrade exits silently

Bug #363500 reported by amimou
8
Affects Status Importance Assigned to Milestone
nvidia-common (Ubuntu)
Fix Released
High
Unassigned
Jaunty
Fix Released
High
Unassigned
update-manager (Ubuntu)
Fix Released
Undecided
Michael Vogt
Jaunty
Fix Released
Undecided
Michael Vogt

Bug Description

Binary package hint: update-manager

TEST CASE:
- after inspecting the code again I verified that update-manager will use the installed
  nvidia-common code if it can import it, so the problem in u-m is fixed by the new
  nvidia-common
- the change was just a updated build-dependency, so just checking for regressions should
  be fine

release 9.04

package version: 1:0.111.6
the update-manager gui disapears may be after finishing the step 'preparing for upgrade'. when run from the command line, the same thing happens and there is no output trace.
before taking new updates (apport ,python fixes ...) the upgrade fails as well but there was a trace when run on the command line. (something related to python_hook module does not exist).

ProblemType: Bug
Architecture: i386
DistroRelease: Ubuntu 9.04
ExecutablePath: /usr/bin/update-manager
InterpreterPath: /usr/bin/python2.6
NonfreeKernelModules: nvidia
Package: update-manager 1:0.111.6
PackageArchitecture: all
ProcEnviron:
 PATH=(custom, no user)
 LANG=en_CA.UTF-8
 SHELL=/bin/bash
SourcePackage: update-manager
Uname: Linux 2.6.28-11-generic i686

Related branches

Revision history for this message
amimou (chelfim) wrote :
Revision history for this message
Michael Vogt (mvo) wrote :

I just debugged this and it turns out that nvidia-common is doing the following in
NvidiaDetection.__init__():
...
      if not os.path.isdir(datadir):
            print 'none'
            logging.debug("dir %s not found" % datadir)
            # Exit without an error exit status
            # so as not to compromise dist-upgrades
            # if the modaliases are not installed.
            exit(0)
...

This is not the right thing to do when a object is imported by other applications (like in this case where nvidia-detector is imported by update-manager).

Revision history for this message
Michael Vogt (mvo) wrote :
Changed in nvidia-common (Ubuntu Jaunty):
importance: Undecided → High
status: New → In Progress
Revision history for this message
Michael Vogt (mvo) wrote :
Revision history for this message
Michael Vogt (mvo) wrote :

I close the update-manager task, its catching exceptions from NvidiaDetector so it does not need modifications once the nvidia-common fix is uploaded.

Changed in update-manager (Ubuntu Jaunty):
status: New → Invalid
Revision history for this message
Michael Vogt (mvo) wrote :

It turns there is a rebuild required in update-manager as well, it embeds a copy of the nvidia-detector backend.

Changed in update-manager (Ubuntu Jaunty):
assignee: nobody → Michael Vogt (mvo)
status: Invalid → In Progress
Michael Vogt (mvo)
summary: - update-manager jaunty partial upgrade crashes silently
+ update-manager jaunty partial upgrade exists silently
Michael Vogt (mvo)
description: updated
Revision history for this message
Martin Pitt (pitti) wrote : Re: update-manager jaunty partial upgrade exists silently

As far as I remember, the nvidia-common part was already done in 0.2.11. Please reopen if I'm wrong.

Changed in nvidia-common (Ubuntu Jaunty):
status: In Progress → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Accepted update-manager into jaunty-proposed-proposed; please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

tags: added: verification-needed
Michael Vogt (mvo)
description: updated
Revision history for this message
Launchpad Janitor (janitor) wrote : Re: update-manager jaunty partial upgrade exists silently

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

---------------
update-manager (1:0.111.8) jaunty-proposed; urgency=low

  * DistUpgrade/DistUpgrade.cfg:
    - add "grub" to the list of packages to keep installed
      (LP: #363465)
    - ensure brasero is upgraded (thanks to Chris Jones for
      the report) (LP: #364136)
    - ensure guidance-power-manager is removed on upgrade
      (LP: #364620)
  * DistUpgrade/DistUpgradeCache.py:
    - support DistUpgradeCache.markUpgrade()
  * DistUpgrade/mirrors.cfg:
    - add "mirror.files.bigpond.com" (thanks to wgrant)
  * debian/control:
    - build-depend on latest nvidia-common (LP: #363500) to ensure
      the nvidia-common if is included in the internal copy of
      u-m
  * DistUpgrade/DistUpgradeQuirks.py:
    - when the upgrade starts, remove old hal.postinst to prevent
      the trigger from running that causes network-manager to shutdown
      all connections (LP: #327053)
  * UpdateManager/Core/MetaRelease.py, UpdateManager/MetaReleaseGObject.py:
    - fix "no longer supported" message (LP: #364583)

 -- Michael Vogt <email address hidden> Mon, 20 Apr 2009 13:53:01 +0200

Changed in update-manager (Ubuntu Jaunty):
status: In Progress → Fix Released
summary: - update-manager jaunty partial upgrade exists silently
+ update-manager jaunty partial upgrade exits silently
Revision history for this message
Brian Murray (brian-murray) wrote :

This was also fixed in Karmic.

Changed in update-manager (Ubuntu):
status: In Progress → 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.