[Documentation] Migration in cinder, lvm-rbd example is not correct

Bug #1969913 reported by Vladislav Belogrudov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Low
Vladislav Belogrudov

Bug Description

This bug tracker is for errors with the documentation, use the following as a template and remove or add fields as you see fit. Convert [ ] into [x] to check boxes:

- [x] This doc is inaccurate in this way: ___lvm <-> rbd migration for live VMs just works ___
- [ ] This is a doc addition request.
- [ ] I have a fix to the document that I can paste below including example: input and output.

If you have a troubleshooting or support issue, use the following resources:

 - The mailing list: https://lists.openstack.org
 - IRC: 'openstack' channel on OFTC

-----------------------------------
Release: 20.1.0.dev57 on 2020-06-02 19:31:34
SHA: 3e068b5ce0990a576b55fea952319884c5c0ed6d
Source: https://opendev.org/openstack/cinder/src/doc/source/contributor/migration.rst
URL: https://docs.openstack.org/cinder/latest/contributor/migration.html

Changed in cinder:
importance: Undecided → Medium
tags: added: documentation lvm migration rbd
removed: doc
Changed in cinder:
status: New → Incomplete
status: Incomplete → New
Changed in cinder:
importance: Medium → Low
tags: added: low-hanging-fruit
summary: - Migration in cinder, lvm-rbd example is not correct
+ [Documentation] Migration in cinder, lvm-rbd example is not correct
Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Greetings Vladislav Belogrudov,
Do you want us to update the documentation just to be sure, or do you have another goal?

Would you please tell us how you reproduce the scenario?

Cheers,
Sofia

Changed in cinder:
status: New → Incomplete
Revision history for this message
Vladislav Belogrudov (vlad-belogrudov) wrote :

I will correct the documentation.

Interesting enough, it did not work before :)

Migration from Train + Ceph Luminous worked only outwards. So cinder "retype --migration-policy on-demand rbdvol1 lvm-1" worked, but "retype --migration-policy on-demand lvmvol1 rbd-1" did not. There was a fix for Ussuri to allow migration into Ceph, it requires

libvirt >= 6
qemu >= 4.2

The bugfix:

https://review.opendev.org/c/openstack/nova/+/696834/
..
To allow swap_volume to support RBD volumes we also need the domain to
use the recently introduced -blockdev support within libvirt >= 6.0.0
and QEMU >= 4.2.0. New MIN_LIBVIRT_BLOCKDEV and MIN_QEMU_BLOCKDEV
version constants are introduced and used to determine when to switch to
the virDomainBlockCopy method of moving between volumes.

[1] https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainBlockRebase
[2] https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainBlockCopy

Changed in cinder:
assignee: nobody → Vladislav Belogrudov (vlad-belogrudov)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/cinder/+/839872

Changed in cinder:
status: Incomplete → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.opendev.org/c/openstack/cinder/+/839872
Committed: https://opendev.org/openstack/cinder/commit/7ee223837ec99bf85d8a4b220457e6c1302acc3a
Submitter: "Zuul (22348)"
Branch: master

commit 7ee223837ec99bf85d8a4b220457e6c1302acc3a
Author: Vladislav Belogrudov <email address hidden>
Date: Fri Apr 29 10:45:37 2022 +0300

    Fix example of failed migrarion for LVM->RBD

    Migration from and to RBD devices works for in-use volumes since
    Ussuri release. Also as of now a migration of in-use volumes for
    non-active VMs is prohibited for any backends.

    Related-Bug: #1673090
    Related-Bug: #1868996
    Closes-Bug: #1969913
    Change-Id: Id639c412a6f5ad24ab9c80416c56c1d99b3f104c

Changed in cinder:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/cinder 21.0.0.0rc1

This issue was fixed in the openstack/cinder 21.0.0.0rc1 release candidate.

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.