[Spice-commits] Changes to 'migrate-script-imprv'

[GitLab Mirror]

New branch 'migrate-script-imprv' available with the following commits:
commit 7d65426a6714923f1d18aae29136c0c4230929a2
Author: Victor Toso <me at victortoso.com>
Date:   Fri Oct 4 16:40:47 2019 +0200

    migrate.py: add seamless migration option
    
    Now we always add seamless-migration option to qemu command line and
    disabled (off) by default although seamless migration is supported for
    over 7 years already!
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>

commit 8f1e3f2c7d1b98635b0e43e9a22f6b29d02912a2
Author: Victor Toso <me at victortoso.com>
Date:   Fri Oct 4 16:56:15 2019 +0200

    migrate.py: cosmetic changes
    
    Make one argument per line;
    Add space after and before '=';
    Indent with arguments;
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>

commit c303126867486acc2dfdb2f973fd5e601046b81c
Author: Victor Toso <me at victortoso.com>
Date:   Fri Oct 4 15:41:57 2019 +0200

    migrate.py: add option for hostname change
    
    This is used in the migrate message and being able to tweak it allows
    clients from different machines to test this local migration test.
    
    Example of usage:
        ./migrate.py --hostname 192.168.122.1 --wait-user-connect
    
    In a VM with access to 192.168.122.1:
        remote-viewer spice://192.168.122.1:5911
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>

commit e47d2238b3bcee753f8d6b68f080b6753a3b1eb0
Author: Victor Toso <me at victortoso.com>
Date:   Fri Oct 4 13:45:02 2019 +0200

    migrate.py: style, break line of qmp's command
    
    Preparatory patch for adding support to change hostname.
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>

commit b048fcd370ac6541f3d93732136ed3e53a4f7f66
Author: Victor Toso <me at victortoso.com>
Date:   Fri Oct 4 14:02:56 2019 +0200

    migrate.py: cosmetic changes on start_qemu()
    
    This function can be tweak quite a bit due all possibilities around
    QEMU configuration. This patch removes incoming_args and extra_args
    and move their values to the right place in the function.
    
    I'm also applying a more extensive coding style for parameters, so
    when we need to do changes it should happen with fewer lines.
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>

commit 696d09c103bb7072721de102a797389c9bfc5303
Author: Victor Toso <me at victortoso.com>
Date:   Mon Sep 30 11:22:37 2019 +0200

    migrate.py: simplify wait_active()
    
    We are only interested in the running state of qmp (or VCPUS) here
    which is what we check. Moving this check to the try branch allows
    removing some extra checks.
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>

commit 64b58825cd88373cb0703ba61d67722a4f744c7a
Author: Victor Toso <me at victortoso.com>
Date:   Fri Oct 4 14:12:18 2019 +0200

    migrate.py: migrate count starts with 1
    
    As this is not some index but a counter. The first print happens after
    first migration was done, meaning 1 instead of 0
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>

commit 8825a2edfc5ad6a80ed87bb426ed03c7bdee471f
Author: Victor Toso <me at victortoso.com>
Date:   Fri Oct 4 15:26:09 2019 +0200

    migrate.py: fix events race
    
    Currently, when we run migrate.py with --client option we get stuck
    waiting for SPICE_CONNECTED event on target qmp (while receiving only
    QMP). Problem is, only after event SPICE_INITIALIZED in the source qmp
    is that the client will be able to do migration (to properly exchange
    spice messages).
    
    So, it is a must that if we have a test where a client is connected,
    to wait for SPICE_INITIALIZED event in the source qmp.
    
    To clarify, this patches fixes the following test:
    
        ./migrate.py --client spicy
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>