Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[LIBCLOUD-614] multiple bugfixes and improvements to GCE #360

Closed
wants to merge 8 commits into from

Commits on Sep 16, 2014

  1. GCE: healthcheck: add description param to ex_create_healthcheck(); t…

    …argetpool: multiple bugfixes
    
    Fix apache#1:
    
    targetpool = driver.ex_get_targetpool('tpname')
    targetpool.add_node(node)
    node.destroy() # targetpool still contains the node
    targetpool = driver.ex_get_targetpool('tpname')
        # targetpool.nodes contains node uri string (in addition to possible other node objects)
        # as produced by _to_targetpool because the node is in the pool but is destroyed
    targetpool.remove_node(node) # raises an exception, removes the node nevertheless.
    
    Expected behavior: remove the node, return true
    
    Fix apache#2:
    targetpool.add(node)
    targetpool.add(node)
        # targetpool.nodes contains 2 copies of node
        # actual targetpool resource on the GCE side doesn't contain 2 copies
    
    Expected behavior: no duplicates in targetpool.nodes, the node list matches GCE side
    
    Fix/Improvement apache#3:
    Allow specifying nodes by fully-qualified node uri in add_node and remove_node.
    
    if tp.nodes:
      tp.remove_node(tp.nodes[0]) # fails if the node in the list doesn't exist
    
    exoected behavior: should be able to remove any node from the list
    
    GCE allows adding non-existent nodes to the targetpool and doesn't automatically
    remove nodes from the pool if you delete them. libcloud should support doing the same.
    Phreedom committed Sep 16, 2014
    Configuration menu
    Copy the full SHA
    7911570 View commit details
    Browse the repository at this point in the history
  2. GCE: fix ex_targetpool_add_healthcheck and ex_targetpool_remove_healt…

    …hcheck.
    
    The requests were malformed and thus did nothing.
    Phreedom committed Sep 16, 2014
    Configuration menu
    Copy the full SHA
    d84a823 View commit details
    Browse the repository at this point in the history
  3. GCE: add a parameter to ex_create_address to promote specific ephemer…

    …al addresses to static ones.
    Phreedom committed Sep 16, 2014
    Configuration menu
    Copy the full SHA
    5beec36 View commit details
    Browse the repository at this point in the history
  4. GCE: add description parameter to ex_create_forwarding_rule, improve …

    …tests; fix documentation and test fixtures for health checks
    Phreedom committed Sep 16, 2014
    Configuration menu
    Copy the full SHA
    49de63c View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    b457589 View commit details
    Browse the repository at this point in the history
  6. GCE: ex_create_firewall: allow creation og firewalls with sourceRange…

    …s = [], preserve behavior for sourceRanges = None, preserve default value.
    
    GCE documentation states that firewall allows traffic
    if it matches either sourceRanges or sourceTags values.
    Thus, sourceRanges = [] is a valid parameter value.
    
    Although it would be better to simply set ["0.0.0.0/0"]
    as the default, for minimal API breakage, None value is
    still supported.
    Phreedom committed Sep 16, 2014
    Configuration menu
    Copy the full SHA
    bcafb9b View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    784427f View commit details
    Browse the repository at this point in the history

Commits on Sep 19, 2014

  1. GCE: lint

    Phreedom committed Sep 19, 2014
    Configuration menu
    Copy the full SHA
    39aecd1 View commit details
    Browse the repository at this point in the history