Unable to attach an EBS volume

Bug #750565 reported by Carlos Perelló Marín
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libvirt (Ubuntu)
Fix Released
High
Serge Hallyn
Maverick
Invalid
High
Serge Hallyn

Bug Description

With Maverick's UEC / Euclayptus, there is no way to mount EBS volumes because libvirt gives the error:

sdh uses scsi:1, rather than scsi:0.

The error and the patch applied upstream can be seen at: http://osdir.com/ml/libvir-list/2011-01/msg01268.html

As you can see it's a single line change, the affected file for the maverick version is: src/qemu/qemu_driver.c (seems like they did some code refactoring on the version used on that email )

I applied the patch by hand, recompiled and installed it and the ebs mount worked.

===============================
SRU Justification:
1. impact: VMs are not able to mount drives
2. How bug was addressed: a patch was cherrypicked from upstream which fixes the mistaken assignment of 0 rather than the chosen controller.
3. Minimal patch: see the linked bzr tree
4. To reproduce: mount a scsi device as /dev/sdh.
 1. # virsh attach-disk --target sdb ...
 2. # virsh attach-disk --target sdh ...
5. Regression potential: only one line is changed, following a patch from upstream.
===============================

Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

Confirmed based on the fact that it is confirmed upstream :)

Patch needs to be re-generated against qemu_driver.c as qemu_hotplug.c doesn't yet exist in maverick's libvirt. I will create the patch and SRU.

For SRU justification, from the linked upstream email:

Steps to reproduce this bug:
> 1. # virsh attach-disk --target sdb ...
> 2. # virsh attach-disk --target sdh ...
> error: Failed to attach disk
> error: operation failed: target scsi:0 already exists
>
> sdh uses scsi:1, rather than scsi:0.
>

Changed in libvirt (Ubuntu):
status: New → Confirmed
importance: Undecided → High
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

@Carlos, could you test the .debs posted in comment #2? If they fix the issue for you then I'll push for SRU.

Changed in libvirt (Ubuntu):
status: Confirmed → In Progress
assignee: nobody → Serge Hallyn (serge-hallyn)
Revision history for this message
Carlos Perelló Marín (carlos) wrote :

Confirmed, it fixes the issue.

Thanks for the speed fixing it!

Changed in libvirt (Ubuntu Maverick):
status: New → In Progress
Changed in libvirt (Ubuntu):
status: In Progress → Fix Released
Changed in libvirt (Ubuntu Maverick):
importance: Undecided → High
assignee: nobody → Serge Hallyn (serge-hallyn)
description: updated
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted libvirt into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in libvirt (Ubuntu Maverick):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
JC Hulce (soaringsky) wrote :

This bug affects Ubuntu 10.10, Maverick Meerkat. Maverick has reached end-of-life and is no longer supported, so I am closing the bugtask for Maverick. Please upgrade to a newer version of Ubuntu.
More information here: https://lists.ubuntu.com/archives/ubuntu-announce/2012-April/000158.html

Changed in libvirt (Ubuntu Maverick):
status: Fix Committed → Invalid
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.