Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      this is necessary for HP Cloud list_nodes to get public ip addresses (probably other OpenStack clouds too?)

      Problem is that on _to_node function of compute/drivers/openstack.py
      response (api_node['addresses']) is something like
      {'private': [

      {'version': 4, 'addr': '10.5.236.143'}

      ,

      {'version': 4, 'addr': '15.185.109.64'}

      ]}

      So it goes immediately on the private ip addresses (which is wrong)

      public_networks_labels = ['public', 'internet']

      public_ips, private_ips = [], []
      for label, values in api_node['addresses'].items():
      ips = [v['addr'] for v in values]

      if label in public_networks_labels:
      public_ips.extend(ips)

      1. 503_openstack.patch
        3 kB
        Markos Gogoulos

        Activity

        Hide
        mgogoulos Markos Gogoulos added a comment -

        patch attached

        Show
        mgogoulos Markos Gogoulos added a comment - patch attached
        Hide
        jira-bot ASF subversion and git services added a comment -

        Commit 859a0a4f769c32761944e35bea0efac88add8207 in libcloud's branch refs/heads/trunk from Markos Gogoulos
        [ https://git-wip-us.apache.org/repos/asf?p=libcloud.git;h=859a0a4 ]

        LIBCLOUD-503 OpenStack public ip addresses fix

        Closes #235

        Signed-off-by: Tomaz Muraus <tomaz@apache.org>

        Show
        jira-bot ASF subversion and git services added a comment - Commit 859a0a4f769c32761944e35bea0efac88add8207 in libcloud's branch refs/heads/trunk from Markos Gogoulos [ https://git-wip-us.apache.org/repos/asf?p=libcloud.git;h=859a0a4 ] LIBCLOUD-503 OpenStack public ip addresses fix Closes #235 Signed-off-by: Tomaz Muraus <tomaz@apache.org>
        Hide
        kami Tomaz Muraus added a comment -

        Merged into trunk.

        Thanks.

        Show
        kami Tomaz Muraus added a comment - Merged into trunk. Thanks.

          People

          • Assignee:
            Unassigned
            Reporter:
            mgogoulos Markos Gogoulos
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development