Libvirt patches needed for 15.04 ppc64le

Bug #1468247 reported by bugproxy
24
This bug affects 3 people
Affects Status Importance Assigned to Milestone
libvirt (Ubuntu)
Fix Released
Undecided
Unassigned
Utopic
Won't Fix
Undecided
Unassigned
Vivid
Fix Released
Medium
Chris J Arges

Bug Description

[Impact]
virsh edit may fail on ppc64el due to schema issues.

[Test Case]
Execute the following on a ppc64el machine:
$ virsh edit <domain>

[Fix]
$ git describe 450a59ba569f571e499d618be4bfde064122f1fc 5e4f49ab8aa2dc7e71b8482e2f15f2a7de1c1006 bdbe723fcd8b55678b0ad984134f7b79d6fbeb89
v1.2.14-220-g450a59b
v1.2.12-146-g5e4f49a
v1.2.12-145-gbdbe723

--

virsh edit <domain>
fails with the following error :
# virsh edit 7
error: XML document failed to validate against schema: Unable to validate doc against /usr/share/libvirt/schemas/domain.rng
Extra element os in interleave
Element domain failed to validate content

---uname output---
3.19.0-18-generic #18-Ubuntu SMP Tue May 19 18:35:08 UTC 2015 ppc64le ppc64le ppc64le GNU/Linux

Following upstream patches need to be pulled in to fix this:

1. commit 450a59ba569f571e499d618be4bfde064122f1fc
Author: Martin Kletzander <email address hidden>
Date: Thu Apr 9 10:49:02 2015 +0200

    schema: Allow multiple machines for VMs and unite architectures

    Use the same pattern for all machine types on all archs and same archs
    for hvm and exe types.

    Signed-off-by: Martin Kletzander <email address hidden>

Additionally, the following fixes are required for ppc64le : (not related to this bug, but need to be taken in)

1. commit 5e4f49ab8aa2dc7e71b8482e2f15f2a7de1c1006
Author: Prerna Saxena <email address hidden>
Date: Sun Feb 15 09:54:15 2015 +0530

    PowerPC : Forbid NULL CPU model with 'host-model' mode.

    PowerPC : Forbid NULL CPU model with 'host-model' mode in qemu command line.

    This ensures that an XML such as following:
    ...
      <cpu mode='host-model'>
        <model fallback='allow'/>
      </cpu>
    ...

    will not generate a '-cpu host,compat=(null)' command line with qemu-system-ppc64
2. commit bdbe723fcd8b55678b0ad984134f7b79d6fbeb89
Author: Prerna Saxena <email address hidden>
Date: Sun Feb 15 09:48:00 2015 +0530

    PowerPC : Make 'qemu-system-ppc64' the default emulator on ppc64[le].

    PowerPC : Explicitly associate 'qemu-system-ppc64' as the
     default emulator for all 64-bit PowerPC guests ( both Big & Little Endian )

    Signed-off-by: Prerna Saxena <email address hidden>

bugproxy (bugproxy)
tags: added: architecture-ppc64le bugnameltc-126685 severity-high targetmilestone-inin---
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

Thank you for taking the time to report this bug and helping to make Ubuntu better. It seems that your bug report is not filed about a specific source package though, rather it is just filed against Ubuntu in general. It is important that bug reports be filed about source packages so that people interested in the package can find the bugs about it. You can find some hints about determining what package your bug might be about at https://wiki.ubuntu.com/Bugs/FindRightPackage. You might also ask for help in the #ubuntu-bugs irc channel on Freenode.

To change the source package that this bug is filed about visit https://bugs.launchpad.net/ubuntu/+bug/1468247/+editstatus and add the package name in the text box next to the word Package.

[This is an automated message. I apologize if it reached you inappropriately; please just reply to this message indicating so.]

tags: added: bot-comment
Prerna (prerna)
affects: ubuntu → libvirt (Ubuntu)
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in libvirt (Ubuntu):
status: New → Confirmed
Chris J Arges (arges)
Changed in libvirt (Ubuntu):
assignee: nobody → Chris J Arges (arges)
Changed in libvirt (Ubuntu Vivid):
assignee: nobody → Chris J Arges (arges)
Changed in libvirt (Ubuntu):
importance: Undecided → Medium
Changed in libvirt (Ubuntu Vivid):
importance: Undecided → Medium
Changed in libvirt (Ubuntu):
status: Confirmed → In Progress
Changed in libvirt (Ubuntu Vivid):
status: New → In Progress
Chris J Arges (arges)
description: updated
Changed in libvirt (Ubuntu):
assignee: Chris J Arges (arges) → nobody
status: In Progress → Fix Released
importance: Medium → Undecided
Revision history for this message
Chris J Arges (arges) wrote :

Debdiff with above fixes and builds against vivid's libvirt.
Once the current package in proposed gets worked through we should merge this with other libvirt fixes and upload into the next version.

Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2015-07-01 18:19 EDT-------
Hey can we have this on utopic as well?

bugproxy (bugproxy)
tags: added: targetmilestone-inin1504
removed: targetmilestone-inin---
Revision history for this message
Chris J Arges (arges) wrote :

The backport to Utopic is not a clean cherry-pick to 1.2.8.
The following require a bit more support: 450a59ba569f571e499d618be4bfde064122f1fc 5e4f49ab8aa2dc7e71b8482e2f15f2a7de1c1006

Any idea for additional support patches? I'd like to confirm the failure in Utopic, and understand if there is a better strategy for this version.

Thanks,

Revision history for this message
Chris J Arges (arges) wrote : Please test proposed package

Hello bugproxy, or anyone else affected,

Accepted libvirt into vivid-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/libvirt/1.2.12-0ubuntu14.1 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 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 to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

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

Changed in libvirt (Ubuntu Vivid):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
bugproxy (bugproxy) wrote : fix-lp1468247-vivid.debdiff

Default Comment by Bridge

Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in libvirt (Ubuntu Utopic):
status: New → Confirmed
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2015-07-15 03:19 EDT-------
Hi Chris,
In response to comment #5:
1) 450a59ba569f57 : This is required. Reason :Qemu updates result in newer pseries* machine types being introduced, which are subsequently not recognized by libvirt.
I had proposed this patch to the community to fix this : https://www.redhat.com/archives/libvir-list/2015-April/msg00200.html
However, they relaxed the schema completely through commit 450a59ba569f57.
You could try introducing the fix at the above URL as a private patch in utopic, if the said upstream commit is difficult. It will address the problem at hand.
2) 5e4f49ab8aa2dc7e : This is a bug fix, and needs inclusion. Are you sure it is not getting properly patched in utopic ? It seems like a one-line change.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package libvirt - 1.2.12-0ubuntu14.1

---------------
libvirt (1.2.12-0ubuntu14.1) vivid; urgency=medium

  [ Seyeong Kim ]
  * d/p/fix_libvirtd_killed_by_sigsegv.patch: fix incorrect backport
    (LP: #1464175)

  [ Edward Hope-Morley ]
  * Add post-start to upstart (/etc/init/libvirt-bin.conf) and
    sysv (/etc/init.d/libvirt-bin) to ensure libvirt-sock
    created before up (LP: #1455608)

  [ Chris J Arges ]
  * Add d/p/PowerPC-Make-qemu-system-ppc64-the-default-emulator-.patch,
    d/p/PowerPC-Forbid-NULL-CPU-model-with-host-model-mode.patch,
    d/p/schema-Allow-multiple-machines-for-VMs-and-unite-arc.patch
    To fix issues with libvirt on ppc64el.(LP: #1468247)

 -- Chris J Arges <email address hidden> Wed, 08 Jul 2015 12:51:16 -0500

Changed in libvirt (Ubuntu Vivid):
status: Fix Committed → Fix Released
Changed in libvirt (Ubuntu Utopic):
status: Confirmed → Won't Fix
tags: added: verification-done
removed: verification-needed
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.