Hi,

I just want to say that there is already a feature wrt KVM migration
where the target process is started with the exact machine version of
the source process (check commits 82e3bf8, 4ab7559, e6ba332 back in
stable-2.8). So if the source process runs with 2.0 and we upgrade to
2.1 then the target process will run with -M pc-i440fx-2.0 and the
migration should succeed! I know there where some incompatibilities
between 1.1 and 1.5 for example but that was QEMU fault/bug.

This patch set gets the version from the node that is the output of kvm
--version. This is NOT the version of the source process but the version
of the installed version (an kvm upgrade after starting the instance).
Thus the check might be wrong. And as I see the default behavior now
is to make the migration fail if no extra option is passed. This
overrides the machine_version feature (which I know is only for the
KVM hypervisor) but is working correctly.

Sorry for the late comment but I hope it helps.

Cheers,
dimara


* 'Klaus Aehlig' via ganeti-devel <[email protected]> [2014-10-02 
10:11:35 +0200]:

> (Resending as due to recent commits to the repo the old
>  last patch would not apply cleanly any more.)
> 
> Change the default behaviour of Ganeti to check for hypervisor
> compatibility. Also add an option to override that, if needed.
> 
> Klaus Aehlig (10):
>   Add a function comparing hypervisor versions
>   Add unit test for HVVersionsLikelySafeForMigration
>   Make hypervisor check fatal
>   Add opcode Parameter pIgnoreHVVersion
>   Add pIgnoreHVVersions to OpInstanceMigrate
>   Honor ignore_hvversions in LUInstanceMigrate
>   Add --ignore-hvversions option
>   Add --ignore-hvversions to gnt-instance migrate
>   Add --ignore-hvversions to gnt-instance migrate man page
>   Document HV version checks in NEWS
> 
>  NEWS                                     |  3 +++
>  lib/cli.py                               |  7 +++++++
>  lib/client/gnt_instance.py               |  6 ++++--
>  lib/cmdlib/instance_migration.py         | 22 +++++++++++++++++++---
>  lib/utils/version.py                     | 18 ++++++++++++++++++
>  man/gnt-instance.rst                     |  6 +++++-
>  src/Ganeti/HTools/Cluster.hs             |  4 +++-
>  src/Ganeti/OpCodes.hs                    |  1 +
>  src/Ganeti/OpParams.hs                   |  6 ++++++
>  test/hs/Test/Ganeti/OpCodes.hs           |  2 +-
>  test/py/ganeti.utils.version_unittest.py | 18 ++++++++++++++++++
>  11 files changed, 85 insertions(+), 8 deletions(-)
> 
> -- 
> 2.1.0.rc2.206.gedb03e5

Attachment: signature.asc
Description: Digital signature

Reply via email to