yahoo-eng-team team mailing list archive

[OpenStack Infra <1525430@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.openstack.org/256883
Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=a28677983df0748fb11cf1fe0ae7650834b02d3f
Submitter: Jenkins
Branch:    master

commit a28677983df0748fb11cf1fe0ae7650834b02d3f
Author: James Arendt <james.arendt@xxxxxx>
Date:   Wed Dec 9 22:58:41 2015 -0800

    Pass config info in neutron-lbaas api v2 tempest tests
    
    Modify resource_setup to pass on additional
    configuration data from tempest.conf to base ServiceClient.
    This allows the tests to run when using a modified tempest.conf,
    such as a non DevStack environment with a unique region and
    endpoint_type specified.
    
    Change-Id: I17ec0566a0bf9d13642e8376af88929c584089f1
    Closes-Bug: #1525430


** Changed in: neutron
       Status: In Progress => Fix Released

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/1525430

Title:
  neutron-lbaas tempest v2 api tests not passing config to serviceclient

Status in neutron:
  Fix Released

Bug description:
  Current lbaas tempest v2 api tests do not pass on certain configuration information
  to the underlyiing ServiceClient used to make a connection, so tests do not
  work if values like the endpoint_type or region for the network are changed
  in the tempest.conf.  Similarly cannot change control values like the build_timeout.

  For example, run tempest tests from the neuton_lbaas tree via OS_TEST_PATH
  with TEMPEST_CONFIG_DIR set to a tempest.conf that has modified network
  values:
  endpoint_type = internalURL

  The existing code will not honor this endpoint_type for the client.  Fix is to pass
  on the info.  I.e. instead of the current code:
          client_args = [auth_provider, 'network', 'regionOne']

          cls.load_balancers_client = (
              load_balancers_client.LoadBalancersClientJSON(*client_args))

  Read configuration and pass on the parameters if they exist
  thru to the ServiceClient base class underling the client connection with
  method signature:
      def __init__(self, auth_provider, service, region,
                   endpoint_type=None, build_interval=None, build_timeout=None,
                   disable_ssl_certificate_validation=None, ca_certs=None,
                   trace_requests=None):

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/1525430/+subscriptions