openstack
/
tempest
Code Issues Proposed changes

Remove test_get_service_by_service_and_host_name 

By default, devstack runs in superconductor mode, meaning there
are 2 conductor services, one is in the cell0 database and one
is in the cell1 database. These get reported with the same service
binary "nova-conductor" and on the same host (since it's single
node devstack). This can cause failures in this test which
randomly picks a service from the list and then does a query by
hostname and binary, and if it picks nova-conductor, it can fail
because it's expecting one but there can be two conductors in this
type of deployment.

This test shouldn't even exist in tempest since it can be (and is)
tested in the nova tree and it's not an interop guideline since
it's an admin-only API by default.

Change-Id: I40a776959768f4422dd7960680a877db96cc3b9b
Closes-Bug: #1771817
This commit is contained in:
Matt Riedemann 2018-05-17 09:26:46 -04:00
parent e4f4d8bee1
commit 6d593b9817
1 changed files with 0 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tempest/api/compute/admin/test_services.py
View File

@ -56,15 +56,3 @@ class ServicesAdminTestJSON(base.BaseV2ComputeAdminTest):
# sort the lists before comparing, to take out dependency
# on order.
self.assertEqual(sorted(s1), sorted(s2))
@decorators.idempotent_id('39397f6f-37b8-4234-8671-281e44c74025')
def test_get_service_by_service_and_host_name(self):
services = self.client.list_services()['services']
host_name = services[0]['host']
binary_name = services[0]['binary']
services = self.client.list_services(host=host_name,
binary=binary_name)['services']
self.assertEqual(1, len(services))
self.assertEqual(host_name, services[0]['host'])
self.assertEqual(binary_name, services[0]['binary'])