Uploaded image for project: 'Libcloud'
  1. Libcloud
  2. LIBCLOUD-165

Rackspace LB/DNS Drivers: Support Auth 2.0 Methods

    Details

    • Type: New Feature
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.1
    • Component/s: None
    • Labels:
      None

      Description

      Currently the Rackspace Loadbalancer and DNS providers are not returned by the Rackspace Auth Service Catalog, so we do not support them with auth 2.0 authentication.

      This adds a workaround to allow these drivers to work with auth 2.0 – we should remove this workaround once these are properly returned by the service catalog.

      1. libcloud-165.patch
        14 kB
        Dave King
      2. libcloud-165-attempt-2.patch
        14 kB
        Dave King

        Activity

        Hide
        tildedave Dave King added a comment -

        First attempt at a patch.

        Show
        tildedave Dave King added a comment - First attempt at a patch.
        Hide
        kami Tomaz Muraus added a comment -

        Ran the tests and they fail:

        ======================================================================
        ERROR: test_gets_auth_2_0_endpoint_defaults_to_ord_region (test.loadbalancer.test_rackspace.RackspaceUKLBTests)
        ----------------------------------------------------------------------
        Traceback (most recent call last):
        File "/w/lc/trunk/test/loadbalancer/test_rackspace.py", line 88, in test_gets_auth_2_0_endpoint_defaults_to_ord_region
        driver.connection._populate_hosts_and_request_paths()
        File "/w/lc/trunk/libcloud/common/openstack.py", line 404, in _populate_hosts_and_request_paths
        osa.authenticate()
        File "/w/lc/trunk/libcloud/common/openstack.py", line 113, in authenticate
        return self.authenticate_2_0_with_password()
        File "/w/lc/trunk/libcloud/common/openstack.py", line 187, in authenticate_2_0_with_password
        return self.authenticate_2_0_with_body(reqbody)
        File "/w/lc/trunk/libcloud/common/openstack.py", line 193, in authenticate_2_0_with_body
        method='POST')
        File "/w/lc/trunk/libcloud/common/base.py", line 562, in request
        headers=headers)
        File "/w/lc/trunk/test/_init_.py", line 161, in request
        meth = getattr(self, meth_name)
        AttributeError: 'RackspaceLBMockHttp' object has no attribute '_v2_0_tokens'

        Probably just need to re-sync your branch with trunk and re-create the patch.

        Show
        kami Tomaz Muraus added a comment - Ran the tests and they fail: ====================================================================== ERROR: test_gets_auth_2_0_endpoint_defaults_to_ord_region (test.loadbalancer.test_rackspace.RackspaceUKLBTests) ---------------------------------------------------------------------- Traceback (most recent call last): File "/w/lc/trunk/test/loadbalancer/test_rackspace.py", line 88, in test_gets_auth_2_0_endpoint_defaults_to_ord_region driver.connection._populate_hosts_and_request_paths() File "/w/lc/trunk/libcloud/common/openstack.py", line 404, in _populate_hosts_and_request_paths osa.authenticate() File "/w/lc/trunk/libcloud/common/openstack.py", line 113, in authenticate return self.authenticate_2_0_with_password() File "/w/lc/trunk/libcloud/common/openstack.py", line 187, in authenticate_2_0_with_password return self.authenticate_2_0_with_body(reqbody) File "/w/lc/trunk/libcloud/common/openstack.py", line 193, in authenticate_2_0_with_body method='POST') File "/w/lc/trunk/libcloud/common/base.py", line 562, in request headers=headers) File "/w/lc/trunk/test/_ init _.py", line 161, in request meth = getattr(self, meth_name) AttributeError: 'RackspaceLBMockHttp' object has no attribute '_v2_0_tokens' Probably just need to re-sync your branch with trunk and re-create the patch.
        Hide
        tildedave Dave King added a comment -

        Update patch to trunk.

        I also updated the tests to use the new connection kwargs in the driver args.

        Show
        tildedave Dave King added a comment - Update patch to trunk. I also updated the tests to use the new connection kwargs in the driver args.
        Hide
        kami Tomaz Muraus added a comment -

        Merged, thanks.

        Show
        kami Tomaz Muraus added a comment - Merged, thanks.

          People

          • Assignee:
            Unassigned
            Reporter:
            tildedave Dave King
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development