tempest SimpleReadOnlyKeystoneClientTest fails with "AttributeError: 'NoneType' object has no attribute 'startswith'"

Bug #1217159 reported by Angus Salkeld
34
This bug affects 6 people
Affects Status Importance Assigned to Milestone
python-keystoneclient
Invalid
Undecided
Unassigned
tempest
Fix Released
Critical
Matthew Treinish

Bug Description

ft161.2: tempest.cli.simple_read_only.test_keystone.SimpleReadOnlyKeystoneClientTest.test_admin_catalog_list_StringException: Empty attachments:
  stderr
  stdout

Traceback (most recent call last):
  File "tempest/cli/simple_read_only/test_keystone.py", line 49, in test_admin_catalog_list
    self.assertTrue(svc['__label'].startswith('Service:'),
AttributeError: 'NoneType' object has no attribute 'startswith'

Revision history for this message
Dolph Mathews (dolph) wrote :
Revision history for this message
Dolph Mathews (dolph) wrote :

Removed duplicate - I think this is a discrete issue

Changed in tempest:
status: New → Confirmed
importance: Undecided → Critical
Changed in tempest:
milestone: none → havana-3
Mark McLoughlin (markmc)
summary: - tempest SimpleReadOnlyKeystoneClientTest fails
+ tempest SimpleReadOnlyKeystoneClientTest fails with "AttributeError:
+ 'NoneType' object has no attribute 'startswith'"
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest (master)

Fix proposed to branch: master
Review: https://review.openstack.org/44135

Revision history for this message
David Kranz (david-kranz) wrote :

The previous (abandoned) patch was for more debug output and not a proposed fix.

Revision history for this message
Dolph Mathews (dolph) wrote :

Better logging has been merged in https://review.openstack.org/#/c/43923/5/ which should provide more insight into this issue the next time it occurs.

Changed in python-keystoneclient:
status: New → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/44160

Changed in tempest:
assignee: nobody → Matthew Treinish (treinish)
status: Confirmed → In Progress
Revision history for this message
Matthew Treinish (treinish) wrote :
Download full text (8.2 KiB)

Sample output from a failing run:

Service: compute
+-------------+-----------------------------------------------------------+
| Property | Value |
+-------------+-----------------------------------------------------------+
| adminURL | http://127.0.0.1:8774/v2/b85086183d1241c092e40682f537ad66 |
| id | ac361376d11c4b089f2b606ee5ae1c45 |
| internalURL | http://127.0.0.1:8774/v2/b85086183d1241c092e40682f537ad66 |
| publicURL | http://127.0.0.1:8774/v2/b85086183d1241c092e40682f537ad66 |
| region | RegionOne |
+-------------+-----------------------------------------------------------+
Service: volumev2
+-------------+-----------------------------------------------------------+
| Property | Value |
+-------------+-----------------------------------------------------------+
| adminURL | http://127.0.0.1:8776/v2/b85086183d1241c092e40682f537ad66 |
| id | f7b17f02e30242e4a75a2ff64bb3ab7f |
| internalURL | http://127.0.0.1:8776/v2/b85086183d1241c092e40682f537ad66 |
| publicURL | http://127.0.0.1:8776/v2/b85086183d1241c092e40682f537ad66 |
| region | RegionOne |
+-------------+-----------------------------------------------------------+
Service: computev3
+-------------+----------------------------------+
| Property | Value |
+-------------+----------------------------------+
| adminURL | http://127.0.0.1:8774/v3 |
| id | 971471446f3d413cb9b75f1c7af18b93 |
| internalURL | http://127.0.0.1:8774/v3 |
| publicURL | http://127.0.0.1:8774/v3 |
| region | RegionOne |
+-------------+----------------------------------+
Service: s3
+-------------+----------------------------------+
| Property | Value |
+-------------+----------------------------------+
| adminURL | http://127.0.0.1:3333 |
| id | c65df1fc46e14d8c863455fc787fa6b4 |
| internalURL | http://127.0.0.1:3333 |
| publicURL | http://127.0.0.1:3333 |
| region | RegionOne |
+-------------+----------------------------------+
Service: image
+-------------+----------------------------------+
| Property | Value |
+-------------+----------------------------------+
| adminURL | http://127.0.0.1:9292 |
| id | 552aa96f7f2b4774ad1a09fb63e2f481 |
| internalURL | http://127.0.0.1:9292 |
| publicURL | http://127.0.0.1:9292 |
| region | RegionOne |
+-------------+----------------------------------+
Service: metering
+-------------+----------------------------------+
| Property | Value |
+-------------+----------------------------------+
| adminURL | http://127.0.0.1:8777 |
| id | df4de58a504148a3acfce748a2e75542 |
| internalURL | http://127.0.0.1:8777 |
| p...

Read more...

Changed in python-keystoneclient:
status: Invalid → Incomplete
Revision history for this message
Matthew Treinish (treinish) wrote :

So after rechecking the parser it's not the '--' that are causing the failure here but the 3 tables being under one service label.

If this is the expected behavior for keystone client then we will just have to improve the parser.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/44178

Revision history for this message
Dolph Mathews (dolph) wrote :

It looks like tempest is creating services without names? I believe that's currently allowed in keystone, as they're only optional labels (the 'type' is what matters- identity, compute, etc).

Revision history for this message
Matthew Treinish (treinish) wrote :

Ok, so that means the fix in https://review.openstack.org/44178 is valid then. There might also be another bug in the tempest client code somewhere then, because I don't think any of the tests in tempest ever intend to create a service without a name. I'll investigate that part later though. (and open a separate bug for that)

Changed in python-keystoneclient:
status: Incomplete → Invalid
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/44178
Committed: http://github.com/openstack/tempest/commit/49681808e702b91fd083d5511399b85ebb74d012
Submitter: Jenkins
Branch: master

commit 49681808e702b91fd083d5511399b85ebb74d012
Author: Matthew Treinish <email address hidden>
Date: Wed Aug 28 19:07:30 2013 -0400

    Fix test_admin_catalog_list

    This commit fixes test_admin_catalog_list() so that it tests more
    specific details of the admin catalog. It also removes the test's
    dependency on each table in the output having a service label. This
    dependency was occasionally causing a failure if the test ran at the
    same time as the keystone api tests.

    Fixes bug 1217159

    Change-Id: I4446abb86a3f1f6514fc9de1d00c06f82943b9d9

Changed in tempest:
status: In Progress → Fix Committed
Sean Dague (sdague)
Changed in tempest:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.