Live migration failed: Unable to parse URI qemu+ssh://nova_migration@fd00:

Bug #1786058 reported by Sergii Golovatiuk on 2018-08-08
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Medium
Sergii Golovatiuk
Ocata
Medium
Lee Yarwood
Pike
Medium
Lee Yarwood
Queens
Medium
Lee Yarwood
Rocky
Medium
Sahid Orentino

Bug Description

Unable to parse URI qemu+ssh://nova_migration@fd00:fd00:fd00:2000::1e/system?keyfile=/etc/nova/migration/identity

IPv6 URI should be wrapped in [], to avoid ambiguity wrt port number.

nova/virt/libvirt/driver.py needs some logic to wrap IPv6 addresses to brackets. Otherwise live migration will fail due to malformed URI

Changed in nova:
assignee: nobody → Sergii Golovatiuk (sgolovatiuk)
Changed in nova:
status: New → In Progress
Changed in nova:
assignee: Sergii Golovatiuk (sgolovatiuk) → Matt Riedemann (mriedem)

Reviewed: https://review.openstack.org/589548
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=8b019d6f1e7893a7e308bd79c879e94d3400e007
Submitter: Zuul
Branch: master

commit 8b019d6f1e7893a7e308bd79c879e94d3400e007
Author: Sergii Golovatiuk <email address hidden>
Date: Tue Aug 7 19:49:37 2018 +0300

    libvirt: Always escape IPv6 addresses when used in migration URI

    If IPv6 is passed to URI it should be wrapped within square brackets.
    This patch detects IPv6 to form migration URI properly. Domain name, IPv4 or
    already bracketed IPv6 address will pass as is

    Extend tests to include collapsed IPv6 addresses and IPv6 addresses with
    port

    Change-Id: I1201db996ea6ceaebd49479b298d74585a78b006
    Closes-Bug: #1786058

Changed in nova:
status: In Progress → Fix Released
Matt Riedemann (mriedem) on 2018-08-22
Changed in nova:
importance: Undecided → Medium
tags: added: ipv6 libvirt live-migration
Changed in nova:
assignee: Matt Riedemann (mriedem) → Sergii Golovatiuk (sgolovatiuk)

Reviewed: https://review.openstack.org/599419
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=7ee5499e5507c45249945fa5baf7665bc2f1cc02
Submitter: Zuul
Branch: stable/rocky

commit 7ee5499e5507c45249945fa5baf7665bc2f1cc02
Author: Sergii Golovatiuk <email address hidden>
Date: Tue Aug 7 19:49:37 2018 +0300

    libvirt: Always escape IPv6 addresses when used in migration URI

    If IPv6 is passed to URI it should be wrapped within square brackets.
    This patch detects IPv6 to form migration URI properly. Domain name, IPv4 or
    already bracketed IPv6 address will pass as is

    Extend tests to include collapsed IPv6 addresses and IPv6 addresses with
    port

    Change-Id: I1201db996ea6ceaebd49479b298d74585a78b006
    Closes-Bug: #1786058
    (cherry picked from commit 8b019d6f1e7893a7e308bd79c879e94d3400e007)

This issue was fixed in the openstack/nova 18.0.1 release.

Reviewed: https://review.openstack.org/603737
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=551a7a459c6503158cb4512fec8b2dc9c0641abe
Submitter: Zuul
Branch: stable/queens

commit 551a7a459c6503158cb4512fec8b2dc9c0641abe
Author: Sergii Golovatiuk <email address hidden>
Date: Tue Aug 7 19:49:37 2018 +0300

    libvirt: Always escape IPv6 addresses when used in migration URI

    If IPv6 is passed to URI it should be wrapped within square brackets.
    This patch detects IPv6 to form migration URI properly. Domain name, IPv4 or
    already bracketed IPv6 address will pass as is

    Extend tests to include collapsed IPv6 addresses and IPv6 addresses with
    port

    Change-Id: I1201db996ea6ceaebd49479b298d74585a78b006
    Closes-Bug: #1786058
    (cherry picked from commit 8b019d6f1e7893a7e308bd79c879e94d3400e007)
    (cherry picked from commit 7ee5499e5507c45249945fa5baf7665bc2f1cc02)

Change abandoned by Lee Yarwood (<email address hidden>) on branch: stable/pike
Review: https://review.openstack.org/603738
Reason: Apologies, finally getting around to looking at this now and as Icdcf681c1d71a09b88b029f80d13cd5015dacd56 landed in stable/queens we can't use it prior to this in Nova. Closing this out.

Change abandoned by Lee Yarwood (<email address hidden>) on branch: stable/ocata
Review: https://review.openstack.org/603740
Reason: Apologies, finally getting around to looking at this now and as Icdcf681c1d71a09b88b029f80d13cd5015dacd56 landed in stable/queens we can't use it prior to this in Nova. Closing this out.

This issue was fixed in the openstack/nova 17.0.7 release.

This issue was fixed in the openstack/nova 19.0.0.0rc1 release candidate.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers