Updates:
        Status: Fixed

Comment #3 on issue 640 by [email protected]: Upgrade: ensure all hypervisors exist in the config file, with default hvparams if needed.
http://code.google.com/p/ganeti/issues/detail?id=640

commit 4fdfdb269516397693001ae9141f562eea8754f5
Author: Michele Tartara <[email protected]>
Date:   Thu Dec 19 19:29:24 2013 +0100

    Update the NEWS file with the Issue 640 fix

    Add an entry in the NEWS file describing the fix of Issue 640.

    Signed-off-by: Michele Tartara <[email protected]>
    Reviewed-by: Helga Velroyen <[email protected]>

commit 6ee8fdd382b3fc617c68d2333610e23e68a824f4
Author: Michele Tartara <[email protected]>
Date:   Thu Dec 19 18:27:38 2013 +0100

    Ensure that all the hypervisors exist in the config file

All the hypervisors are supposed to exist in the config file, but it might not be so after upgrades from old versions. This patch ensures that all the missing
    hypervisors are added with their default values to the config file.

Also, some tests are adapted, because now they receive the default values instead of an empty dictionary when they are working using a minimal cluster
    config as their input.

    Fixes Issue 640.

    Signed-off-by: Michele Tartara <[email protected]>
    Reviewed-by: Helga Velroyen <[email protected]>

commit 52a988f208ab5aeba8c2d1907a67876dc3ecfcef
Author: Michele Tartara <[email protected]>
Date:   Thu Dec 19 18:13:13 2013 +0100

    Fix testEncodeInstance test input

The input of the testEncodeInstance test was not adherent to the actual format of the Ganeti configuration file: kvm has no HV_BLOCKDEV_PREFIX, and "hvparams" inside an instance should only contain the values of the hypervisor parameters, not the hypervisor name, which is already declared in the "hypervisor" field, and which was not correctly aligned with the parameters in the "hvparams"
    section.

All these problems are now fixed, and the assertions are changed accordingly.

    Signed-off-by: Michele Tartara <[email protected]>


--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

Reply via email to