Updates:
        Status: Fixed

Comment #1 on issue 1073 by [email protected]: ssconf_hvparams_* not distributed with ssconf
https://code.google.com/p/ganeti/issues/detail?id=1073

Fixed in

commit 1f8b080ae9fe4e8b2095b07003dc990e8a69894b
Author: Petr Pudlak <[email protected]>
Date:   Wed Apr 22 19:09:47 2015 +0200

    Include hypervisor parameters in SSConf

    This was omitted after the refactoring of SSConf to Haskell, now being
    added. Fixes #1073.

    Signed-off-by: Petr Pudlak <[email protected]>
    Reviewed-by: Klaus Aehlig <[email protected]>

commit 7c6deb9fcf93a3957032700ba67d4c2b761a88fc
Author: Petr Pudlak <[email protected]>
Date:   Wed Apr 22 17:48:27 2015 +0200

    Add SSConf keys for hypervisor parameters

    .. and a function for converting hypervisor types to the keys.

    Signed-off-by: Petr Pudlak <[email protected]>
    Reviewed-by: Klaus Aehlig <[email protected]>

commit 812366793041fbea3976a2d6bc59185203b3f335
Author: Petr Pudlak <[email protected]>
Date:   Wed Apr 22 13:10:29 2015 +0200

    Use Hypervisor as the key in ClusterHvParams

    .. instead of String. This imposes a type-level restriction that the
    keys of the map are just hypervisor names.

    Note about 'Arbitrary GroupDiskParams': Since GroupDiskParams and
    ClusterHvParams were both synonyms for 'Container (Container JSValue)',
    the Arbitrary instance worked for both. After fixing the type of
    ClousterHvParams, its instance become different from GroupDiskParams,
    hence the latter needs the addition.

    Signed-off-by: Petr Pudlak <[email protected]>
    Reviewed-by: Klaus Aehlig <[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