'virsh migrate' fails in a password authentication of SSH

Bug #1365947 reported by Seyeong Kim
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
libvirt (Ubuntu)
Fix Released
Medium
Seyeong Kim
Precise
Won't Fix
Medium
Unassigned
Trusty
Fix Released
Medium
Unassigned

Bug Description

[SRU justification]

[Impact]

I can't input password properly when virsh migrate from one to the others by ssh

[Test Case]

there are two hosts

host1 ( has vm1 )
host2 ( has nothing )

connect to host1 by ssh

and type below command

# virsh --connect qemu+ssh://user1@host1/system migrate --live \
 vm-name qemu+ssh://user2@host2/system

and need to input password
but i can't

it seems hang at this time.

[Regression Potential]

[Other Info]
upstream patch
http://libvirt.org/git/?p=libvirt.git;a=commit;h=7eabd5503e3de147b703c1a1e6dff81cdc46d1e7
http://libvirt.org/git/?p=libvirt.git;a=commit;h=676cb4f4e762b8682a06c6dab1f690fbcd939550

Seyeong Kim (seyeongkim)
affects: openjdk-7 (Ubuntu) → libvirt (Ubuntu)
Changed in libvirt (Ubuntu):
assignee: nobody → Seyeong Kim (xtrusia)
status: New → In Progress
Seyeong Kim (seyeongkim)
description: updated
Revision history for this message
Serge Hallyn (serge-hallyn) wrote : Re: [Bug 1365947] Re: 'virsh migrate' fails in a password authentication of SSH

Thanks for submitting this bug and posting patches! Do the patches
definately fix it for you? (I had always thought the cause was
different, so was surprised to see the patches are handling network
timeout)

Setting the priority to medium since there is a workaround (using
ssh keys)

 importance: medium

Changed in libvirt (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Seyeong Kim (seyeongkim) wrote :

Hello

There was a poll function problem waiting STDIN on PTS env ( on ssh ). so looks hang.

I emailed patch (not this) to the maintainer of libvirt, but they told me about this commits ( attached ).

then I applied this patch and tested on my machines. and It worked.

and the customer (NEC) also tested this.

Thanks.

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

Thanks. so those should be fixed in the 1.2.8 version we are trying to upload (awaiting FFE) to utopic now. Note also that commit 7eabd55. introduced a regression fixed by c285ffc4.

We can SRU these into trusty once they are in utopic.

Revision history for this message
Seyeong Kim (seyeongkim) wrote :
Revision history for this message
Seyeong Kim (seyeongkim) wrote :
Revision history for this message
Seyeong Kim (seyeongkim) wrote :

Thanks.

got c285ffc4, applied patch is uploaded.

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

thank you. I've gone with keeping the three patches separate, so that in 3-4 years it's easier to tell what has and has not been applied based on the git tree, however having your patch to sanity-check and make sure we got the same result was still very useful.

Changed in libvirt (Ubuntu Trusty):
importance: Undecided → Medium
Changed in libvirt (Ubuntu Precise):
importance: Undecided → Medium
Revision history for this message
Chris J Arges (arges) wrote :

The patches: c285ffc4, 7eabd55, 676cb4f4 are all in 1.2.8 and that version is in Utopic so marking Fix Released for that version.

Changed in libvirt (Ubuntu):
status: In Progress → Fix Released
Changed in libvirt (Ubuntu Trusty):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Chris J Arges (arges) wrote : Please test proposed package

Hello Seyeong, or anyone else affected,

Accepted libvirt into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/libvirt/1.2.2-0ubuntu13.1.6 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!

Revision history for this message
Seyeong Kim (seyeongkim) wrote :

I confirmed the packages work well on my test machines

Thanks

tags: added: verification-done
removed: verification-needed
Revision history for this message
Adam Conrad (adconrad) wrote : Update Released

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

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

This bug was fixed in the package libvirt - 1.2.2-0ubuntu13.1.6

---------------
libvirt (1.2.2-0ubuntu13.1.6) trusty-proposed; urgency=medium

  * 9029-ovs-delete-port-if-it-exists-when-adding-new-one: cherrypick commit
    33445ce from upstream (LP: #1343262)
  * fix migration failure with ssh password authentication (LP: #1365947)
    - 9030-virsh-add-keepalive-in-new-vshconnect-fn
    - 9031-cmdmigrate-move-vshconnect-before-vshwatchjob
    - 9032-virsh-initialize-vsh-data-in-cmdmigrate
  * libvirt-bin.postinst: check for confiles whichhave been removed rather
    than fail package install (LP: #1375910)
  * Support incoming migration from 12.04 hosts (LP: #1374622)
    - debian/patches/support-incoming-qemu-kvm: add a flag to
      /etc/libvirt/qemu.conf to specify whether pc-1.0 came from a 12.04 host
    - Add a note in README.Debian.
 -- Serge Hallyn <email address hidden> Tue, 30 Sep 2014 13:54:31 -0500

Changed in libvirt (Ubuntu Trusty):
status: Fix Committed → Fix Released
Revision history for this message
Serge Hallyn (serge-hallyn) wrote :

This code has changed quite a bit since precise, and I don't want to accidentally cause a regression. Given that there is a workaround (using ssh keys) I am going to drop the priority of this and mark it wont-fix. If someone can describe a case where using ssh keys cannot work as a workaround, we can consider re-raising the priority.

Changed in libvirt (Ubuntu Precise):
status: New → Won't Fix
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.