Bug 1829545 - Stop using deprecated "migrate_set_speed" QMP command
Summary: Stop using deprecated "migrate_set_speed" QMP command
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux Advanced Virtualization
Classification: Red Hat
Component: libvirt
Version: 8.2
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: rc
: 8.3
Assignee: Jiri Denemark
QA Contact: Fangge Jin
URL:
Whiteboard:
Depends On:
Blocks: 1829697
TreeView+ depends on / blocked
 
Reported: 2020-04-29 18:22 UTC by Peter Krempa
Modified: 2020-11-17 17:49 UTC (History)
8 users (show)

Fixed In Version: libvirt-6.5.0-1.el8
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1829697 (view as bug list)
Environment:
Last Closed: 2020-11-17 17:48:27 UTC
Type: Bug
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Peter Krempa 2020-04-29 18:22:40 UTC
Description of problem:
"migrate_set_speed" is deprecated since qemu-2.8 and should be replaced by 'migrate-set-parameters". Libvirt still uses it in qemuDomainMigrateSetMaxSpeed (non-postcopy case), qemuMigrationSrcRun (always), qemuMigrationSrcToFile (always).

When fixing this, please also adjust the test case to mark the command as okay to be removed.

Comment 1 Jiri Denemark 2020-06-10 18:26:39 UTC
Patches sent upstream for review: https://www.redhat.com/archives/libvir-list/2020-June/msg00444.html

Comment 2 Jiri Denemark 2020-06-17 08:27:57 UTC
Fixed upstream by

commit 92b8dbc66a2c181382973835c1e9ad16cac2ff29
Refs: v6.4.0-123-g92b8dbc66a
Author:     Jiri Denemark <jdenemar>
AuthorDate: Wed Jun 10 16:13:15 2020 +0200
Commit:     Jiri Denemark <jdenemar>
CommitDate: Wed Jun 17 10:25:33 2020 +0200

    qemu: Avoid deprecated migrate_set_speed QMP command

    The same functionality can be achieved using migrate-set-parameters QMP
    command with max-bandwidth parameter.

    https://bugzilla.redhat.com/show_bug.cgi?id=1829545

    Signed-off-by: Jiri Denemark <jdenemar>
    Reviewed-by: Peter Krempa <pkrempa>

Comment 6 Fangge Jin 2020-07-31 14:13:46 UTC
Verify with:
libvirt-client-6.5.0-1.module+el8.3.0+7323+d54bb644.x86_64
qemu-kvm-5.0.0-2.module+el8.3.0+7379+0505d6ca.x86_64


Results:
migrate-set-parameters is used

Steps:
1)# virsh migrate-setspeed rhel7-min 10

2)# virsh migrate-setspeed rhel7-min 10 --postcopy

3) # virsh migrate rhel7-min qemu+ssh://10.0.150.207/system --live --verbose  --p2p --migrateuri tcp://10.0.150.207 --bandwidth 15

4) # virsh managedsave rhel7-min


Still need to verify with qemu-kvm version that doesn't support set migrate speed via migrate-set-parameters

Comment 7 Fangge Jin 2020-08-27 09:44:45 UTC
(In reply to Fangge Jin from comment #6)
> Still need to verify with qemu-kvm version that doesn't support set migrate
> speed via migrate-set-parameters

To test this scenario, I will have to test upgrade from RHEL7.3 to RHELAV-8.3, which is not supported.
So this bug can be marked as verified.

Comment 10 errata-xmlrpc 2020-11-17 17:48:27 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (virt:8.3 bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2020:5137


Note You need to log in before you can comment on or make changes to this bug.