yahoo-eng-team team mailing list archive

[Assaf Muller <amuller@xxxxxxxxxx>]

Public bug reported:

I recently dropped Tempest 'plumbing' code from Neutron and imported it
instead from Tempest: https://review.openstack.org/#/c/269771/

Since then, we've had two instances of changes to Tempest breaking the Neutron API tests:
https://bugs.launchpad.net/neutron/+bug/1554362, and:
https://review.openstack.org/#/c/284911/

Neutron imports the following files from Tempest (That aren't a stable
interface e.g. in tempest.lib) that Manila and Ironic (As examples of
projects that have Tempest plugins) do not:

from tempest.common import cred_provider
from tempest.common import custom_matchers
from tempest.common import tempest_fixtures
from tempest import manager
from tempest.services.identity.v2.json.tenants_client import TenantsClient

Instead of waiting for the next breakage, we should go through each such import and figure out a strategy to stop using it:
1) Perhaps there is already equivalent code in tempest.lib?
2) If not, can we help move it to tempest.lib?
3) Failing that, we can always copy the needed functionality in to Neutron

** Affects: neutron
     Importance: High
         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/1555356

Title:
  Neutron too coupled with Tempest

Status in neutron:
  New

Bug description:
  I recently dropped Tempest 'plumbing' code from Neutron and imported
  it instead from Tempest: https://review.openstack.org/#/c/269771/

  Since then, we've had two instances of changes to Tempest breaking the Neutron API tests:
  https://bugs.launchpad.net/neutron/+bug/1554362, and:
  https://review.openstack.org/#/c/284911/

  Neutron imports the following files from Tempest (That aren't a stable
  interface e.g. in tempest.lib) that Manila and Ironic (As examples of
  projects that have Tempest plugins) do not:

  from tempest.common import cred_provider
  from tempest.common import custom_matchers
  from tempest.common import tempest_fixtures
  from tempest import manager
  from tempest.services.identity.v2.json.tenants_client import TenantsClient

  Instead of waiting for the next breakage, we should go through each such import and figure out a strategy to stop using it:
  1) Perhaps there is already equivalent code in tempest.lib?
  2) If not, can we help move it to tempest.lib?
  3) Failing that, we can always copy the needed functionality in to Neutron

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