nova-compute dies without proper error message

Bug #1031351 reported by Mate Lakat
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Undecided
Andrew Bogott
openstack-common
Fix Released
Undecided
Mate Lakat

Bug Description

Using xenapi driver (xenapi.XenAPIDriver) on top of an ubuntu 12.04 + xcp-xapi, starting devstack nova-compute dies without any information:

stack@DevStackOSDomU:~/devstack$ cd /opt/stack/nova && sg libvirtd /opt/stack/nova/bin/nova-compute
2012-07-31 13:03:32 INFO nova.compute.manager [-] Loading compute driver 'xenapi.XenAPIDriver'

Some better error handling is required around:

~/nova$ cat -n nova/compute/manager.py | grep -A 20 Loading
   284 LOG.info(_("Loading compute driver '%s'") % compute_driver)
   285 try:
   286 self.driver = utils.check_isinstance(
   287 importutils.import_object_ns('nova.virt', compute_driver),
   288 driver.ComputeDriver)
   289 except ImportError as e:
   290 LOG.error(_("Unable to load the virtualization driver: %s") % (e))
   291 sys.exit(1)
   292

The actual problem, is that an Exception is raised in the driver code, see #1025216

Tags: xcp-xapi
Mate Lakat (mate-lakat)
tags: added: xcp-xapi
Changed in nova:
assignee: nobody → Mate Lakat (mate-lakat)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/10603

Changed in nova:
status: New → In Progress
Mate Lakat (mate-lakat)
description: updated
description: updated
Revision history for this message
Mark McLoughlin (markmc) wrote :
Changed in openstack-common:
status: New → In Progress
assignee: nobody → Mate Lakat (mate-lakat)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-common (master)

Reviewed: https://review.openstack.org/10611
Committed: http://github.com/openstack/openstack-common/commit/58404a1a9461900367dbb7a8b3f2f98441e12959
Submitter: Jenkins
Branch: master

commit 58404a1a9461900367dbb7a8b3f2f98441e12959
Author: Mate Lakat <email address hidden>
Date: Tue Jul 31 17:53:57 2012 +0100

    Install a qualified except hook.

    Fixes bug 1031351.

    As an Exception is raised, it was not visible in the log, because
    it wasn't qualified. The fix sets up the exception hook to be
    qualified as the product_name.

    Change-Id: Ie8061d003b05ea8b00db5d8ed8fa445ccc822942

Changed in openstack-common:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to openstack-common (master)

Fix proposed to branch: master
Review: https://review.openstack.org/10878

Changed in openstack-common:
status: Fix Committed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to openstack-common (master)

Reviewed: https://review.openstack.org/10878
Committed: http://github.com/openstack/openstack-common/commit/babd6ca3324b3a684dc1e0beedadfc89981039c7
Submitter: Jenkins
Branch: master

commit babd6ca3324b3a684dc1e0beedadfc89981039c7
Author: Mate Lakat <email address hidden>
Date: Mon Aug 6 11:32:22 2012 +0100

    Fix meaningless test case

    Related to fixing bug 1031351

    The testcase was meaningless.

    Change-Id: I7b88b86a600fac0c30bcac354262ac00593b495c

Changed in openstack-common:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/10895

Changed in nova:
assignee: Mate Lakat (mate-lakat) → Andrew Bogott (andrewbogott)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/10898

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/10898
Committed: http://github.com/openstack/nova/commit/a7987aaa86c9cd8c6509118028a61ce7445ec125
Submitter: Jenkins
Branch: master

commit a7987aaa86c9cd8c6509118028a61ce7445ec125
Author: Andrew Bogott <email address hidden>
Date: Fri Aug 3 01:57:40 2012 -0500

    Migrate a logging change from common:

    commit 58404a1a9461900367dbb7a8b3f2f98441e12959
    Author: Mate Lakat <email address hidden>
    Date: Tue Jul 31 17:53:57 2012 +0100

        Install a qualified except hook.

        Fixes bug 1031351.

        As an Exception is raised, it was not visible in the log, because
        it wasn't qualified. The fix sets up the exception hook to be
        qualified as the product_name.

    Change-Id: I38c48d0dd2ac885fcb226310b6039be667af9e0d

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → folsom-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: folsom-3 → 2012.2
Mark McLoughlin (markmc)
Changed in openstack-common:
milestone: none → 2012.2
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.