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 -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : yahoo-eng-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp
