← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1810963] [NEW] Neutron Tempest plugin creates new users/projects even if use_dynamic_credentials is set to False

 

Public bug reported:

Hello together.

I run Neutron tempest test plugin and have some test cases which fail because of OverQuota errors. 
The reason is that the limits/quota for new users/projects is 0 in my environment. Thus, OverQuota is raised. If neutron tempest plugin would care about the use_dynamic_credentials = False then OverQuota would not be raised because it would use my configured users/projects below with high Quota.
Other suggestion: if isolated tenant is really necessary for a certain test then this test should be skipped if use_dynamic_credentials = False but force_tenant_isolation = True is required.

I configured tempest with 3 users and corresponding projects to perform
the tests:

- username: 'tempestuser1'
  password: MYSUPERSECRETPASSWORD
  tenant_name: 'tempest1'
  project_name: 'tempest1'
- username: 'tempestuser2'
  password: MYSUPERSECRETPASSWORD
  tenant_name: 'tempest2'
  project_name: 'tempest2'
- username: 'admin'
  password: MYSUPERSECRETPASSWORD
  tenant_name: 'admin'
  project_name: 'admin'
  types:
  - admin

I was able to debug the test run and found some code which overwrite
use_dynamic_credentials = False:

https://github.com/openstack/neutron-tempest-
plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/base.py#L1022

https://github.com/openstack/neutron-tempest-
plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/test_auto_allocated_topology.py#L33

https://github.com/openstack/neutron-tempest-
plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/test_qos.py#L603

https://github.com/openstack/neutron-tempest-
plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/admin/test_floating_ips_admin_actions.py#L28

https://github.com/openstack/neutron-tempest-
plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/admin/test_shared_network_extension.py#L177

above code forces to use dynamic_credentials because of:

https://github.com/openstack/tempest/blob/7e4087fb104bc3ac9608c524f2b7c875205ccbbb/tempest/common/credentials_factory.py#L137

** Affects: neutron
     Importance: Undecided
         Status: New

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

Title:
  Neutron Tempest plugin creates new users/projects even if
  use_dynamic_credentials is set to False

Status in neutron:
  New

Bug description:
  Hello together.

  I run Neutron tempest test plugin and have some test cases which fail because of OverQuota errors. 
  The reason is that the limits/quota for new users/projects is 0 in my environment. Thus, OverQuota is raised. If neutron tempest plugin would care about the use_dynamic_credentials = False then OverQuota would not be raised because it would use my configured users/projects below with high Quota.
  Other suggestion: if isolated tenant is really necessary for a certain test then this test should be skipped if use_dynamic_credentials = False but force_tenant_isolation = True is required.

  I configured tempest with 3 users and corresponding projects to
  perform the tests:

  - username: 'tempestuser1'
    password: MYSUPERSECRETPASSWORD
    tenant_name: 'tempest1'
    project_name: 'tempest1'
  - username: 'tempestuser2'
    password: MYSUPERSECRETPASSWORD
    tenant_name: 'tempest2'
    project_name: 'tempest2'
  - username: 'admin'
    password: MYSUPERSECRETPASSWORD
    tenant_name: 'admin'
    project_name: 'admin'
    types:
    - admin

  I was able to debug the test run and found some code which overwrite
  use_dynamic_credentials = False:

  https://github.com/openstack/neutron-tempest-
  plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/base.py#L1022

  https://github.com/openstack/neutron-tempest-
  plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/test_auto_allocated_topology.py#L33

  https://github.com/openstack/neutron-tempest-
  plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/test_qos.py#L603

  https://github.com/openstack/neutron-tempest-
  plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/admin/test_floating_ips_admin_actions.py#L28

  https://github.com/openstack/neutron-tempest-
  plugin/blob/06ef854082cb1f62038167e7e654181aa8727369/neutron_tempest_plugin/api/admin/test_shared_network_extension.py#L177

  above code forces to use dynamic_credentials because of:

  https://github.com/openstack/tempest/blob/7e4087fb104bc3ac9608c524f2b7c875205ccbbb/tempest/common/credentials_factory.py#L137

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


Follow ups