[SRU] 16.1.0 test failure UEFI is not supported

Bug #1758060 reported by Corey Bryant
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Invalid
Undecided
Unassigned
nova (Ubuntu)
Invalid
Undecided
Unassigned
Artful
Fix Released
Medium
Corey Bryant

Bug Description

[Impact]

The following test is failing in Ubuntu package builds for the stable 16.1.0 point release:

==============================
Failed 1 tests - output below:
==============================

nova.tests.unit.virt.libvirt.test_driver.LibvirtConnTestCase.test_get_guest_config_with_type_kvm_on_aarch64
-----------------------------------------------------------------------------------------------------------

Captured traceback:
~~~~~~~~~~~~~~~~~~~
    Traceback (most recent call last):
      File "/usr/lib/python2.7/dist-packages/mock/mock.py", line 1305, in patched
        return func(*args, **keywargs)
      File "nova/tests/unit/virt/libvirt/test_driver.py", line 4284, in test_get_guest_config_with_type_kvm_on_aarch64
        cfg = self._get_guest_config_via_fake_api(instance_ref)
      File "nova/tests/unit/virt/libvirt/test_driver.py", line 4315, in _get_guest_config_via_fake_api
        image_meta, disk_info)
      File "nova/virt/libvirt/driver.py", line 4942, in _get_guest_config
        root_device_name)
      File "nova/virt/libvirt/driver.py", line 4603, in _configure_guest_by_virt_type
        raise exception.UEFINotSupported()
    nova.exception.UEFINotSupported: UEFI is not supported

Captured pythonlogging:
~~~~~~~~~~~~~~~~~~~~~~~
    2018-03-22 13:11:56,498 WARNING [os_brick.initiator.connectors.remotefs] Connection details not present. RemoteFsClient may not initialize properly.
    2018-03-22 13:11:56,505 INFO [nova.virt.libvirt.driver] CPU mode "host-passthrough" was chosen. Live migration can break unless all compute nodes have identical cpus. AArch64 does not support other modes.

[Test Case]
Building the package will run unit tests.

[Regression Potential]
None. This just updates a test case.

Revision history for this message
Corey Bryant (corey.bryant) wrote :

Triaging to low for the Ubuntu package as it hasn't been uploaded yet. Holding off on upload for now.

Changed in nova (Ubuntu):
status: New → Triaged
importance: Undecided → Low
assignee: nobody → Corey Bryant (corey.bryant)
Matt Riedemann (mriedem)
tags: added: aarch64 libvirt
Revision history for this message
Matt Riedemann (mriedem) wrote :

Umm, I don't see that test in the nova repo upstream:

https://github.com/openstack/nova/blob/16.1.0/nova/tests/unit/virt/libvirt/test_driver.py

Do you have some code patched in?

Changed in nova:
status: New → Invalid
Revision history for this message
Corey Bryant (corey.bryant) wrote :

Apologies, we're carrying this test in the package.

summary: - 16.1.0 test failure UEFI is not supported
+ [SRU] 16.1.0 test failure UEFI is not supported
description: updated
Changed in nova (Ubuntu Artful):
status: New → Triaged
importance: Undecided → Medium
Changed in nova (Ubuntu):
status: Triaged → Invalid
importance: Low → Undecided
Changed in nova (Ubuntu Artful):
assignee: nobody → Corey Bryant (corey.bryant)
Changed in nova (Ubuntu):
assignee: Corey Bryant (corey.bryant) → nobody
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Corey, or anyone else affected,

Accepted nova into artful-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nova/2:16.1.0-0ubuntu1 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 on 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-artful to verification-done-artful. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-artful. In either case, without details of your testing we will not be able to proceed.

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

Changed in nova (Ubuntu Artful):
status: Triaged → Fix Committed
tags: added: verification-needed verification-needed-artful
Revision history for this message
Corey Bryant (corey.bryant) wrote :

The new version of the package has been verified to build successfully.

tags: added: verification-done-verification-needed-artful
removed: verification-needed verification-needed-artful
tags: added: verification-done-artful
removed: verification-done-verification-needed-artful
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nova - 2:16.1.0-0ubuntu1

---------------
nova (2:16.1.0-0ubuntu1) artful; urgency=medium

  [ James Page ]
  * Add Depends nova-compute-kvm -> ipxe-qemu to ensure that required
    ROM's are installed for all architectures (LP: #1754015).

  [ Corey Bryant ]
  * New stable point release for OpenStack Pike (LP: #1758046).
  * d/p/aarch64-libvirt-compat.patch: Rebased and updated to mock
    os.path.exists return value to true in order to simulate
    _has_uefi_support returning true (LP: #1758060).

 -- Corey Bryant <email address hidden> Thu, 22 Mar 2018 08:44:37 -0400

Changed in nova (Ubuntu Artful):
status: Fix Committed → Fix Released
Revision history for this message
Chris Halse Rogers (raof) wrote : Update Released

The verification of the Stable Release Update for nova 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 regressions.

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.