Updates:
        Status: Fixed

Comment #6 on issue 355 by thoma...@google.com: hail should make sure a network is available before connecting an instance to a nodegroup
http://code.google.com/p/ganeti/issues/detail?id=355

Fixed with the following commits:

commit b69cc7edb1054b398619000fc22639d028de8c9b
Author: Thomas Thrainer <thoma...@google.com>
Date:   Thu May 23 09:46:42 2013 +0200

    Add NEWS entry for hail honoring networks

    The entry also mentions the required change in the IAllocator protocol.

    Signed-off-by: Thomas Thrainer <thoma...@google.com>
    Reviewed-by: Guido Trotter <ultrot...@google.com>

commit 017160ed075e24684001b5e5ac0d89900c88e689
Author: Thomas Thrainer <thoma...@google.com>
Date:   Thu May 16 14:56:26 2013 +0200

    Add tests for network-aware allocation

    hail-alloc-invalid-network defines a cluster with two nodegroups and an
    allocation request which does not fit on any of the groups. Group 1 has
    invalid disk-templates while Group 2 is not connected to the right
    networks.

    hail-alloc-restricted-network defines a cluster wih two nodegroups.
    Nodegroup Group 1 is only used as last_resort group, but is chosen by
    hail as only this group is connected to the right networks.

    Signed-off-by: Thomas Thrainer <thoma...@google.com>
    Reviewed-by: Guido Trotter <ultrot...@google.com>

commit b1142361bf5fd3ccdb56c410acbfa98b861e36c0
Author: Thomas Thrainer <thoma...@google.com>
Date:   Thu May 16 16:58:00 2013 +0200

    Honor network connections in hail

    Before trying to allocate nodes in node groups, node groups are now
    filtered based on the networks they are connected to an the networks
    which are required by the new instance.

    Signed-off-by: Thomas Thrainer <thoma...@google.com>
    Reviewed-by: Guido Trotter <ultrot...@google.com>

commit 908c2f67f9433785c64a285cdc0ae3a2896cc617
Author: Thomas Thrainer <thoma...@google.com>
Date:   Tue May 21 08:08:04 2013 +0200

    Parse NIC data from allocation request in hail

    Add a NIC type and extend the Instance type by a list of NIC's. Parse
    the NIC's in allocation requests and store them for now. Later patches
    will make use of this field in order to ensure that the requested
    instance is only placed in node groups wich are connected to those
    networks.

    Signed-off-by: Thomas Thrainer <thoma...@google.com>
    Reviewed-by: Guido Trotter <ultrot...@google.com>

commit a0be8f1a137566e9ed0f864eeeb785a0ab3bbc00
Author: Thomas Thrainer <thoma...@google.com>
Date:   Fri May 24 10:03:02 2013 +0200

    Support group networks in Text backend

    The Text backend now parses network UUID (comma separated) and
    serializes them in the same form.
    The test data is adapted to the new format.

    Signed-off-by: Thomas Thrainer <thoma...@google.com>
    Reviewed-by: Guido Trotter <ultrot...@google.com>

commit c8b199db6839bc8c77729478e02cef9324c2a186
Author: Thomas Thrainer <thoma...@google.com>
Date:   Fri May 17 13:39:07 2013 +0200

    Parse node group networks

    Extend the Group by the network ids it is connected to. Adapt
    the IAlloc backend such that the networks are parsed correctly.
    This also required the adaption of test data.

    Signed-off-by: Thomas Thrainer <thoma...@google.com>
    Reviewed-by: Guido Trotter <ultrot...@google.com>

commit 2daca99b5be21ea4667637721ee2c65a14444520
Author: Thomas Thrainer <thoma...@google.com>
Date:   Thu May 23 09:39:33 2013 +0200

    Update IAllocator interface documentation

    Document the newly added network field of node groups, and also update
    the missing documentation for the other fields of node groups.

    Signed-off-by: Thomas Thrainer <thoma...@google.com>
    Reviewed-by: Guido Trotter <ultrot...@google.com>

--
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