tests.identity.admin.test_services:ServicesTestBase.test_create_get_delete_service() fails due to incorrect parameter (XML only)

Bug #1061740 reported by Jay Pipes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
Critical
Jay Pipes

Bug Description

======================================================================
ERROR: GET Service
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jpipes/repos/tempest/tempest/tests/identity/admin/test_services.py", line 62, in test_create_get_delete_service
    resp, _ = self.client.delete_service(service_data['id'])
UnboundLocalError: local variable 'service_data' referenced before assignment
-------------------- >> begin captured logging << --------------------
tempest.common.rest_client: ERROR: Request URL: http://127.0.0.1:35357/v2.0/OS-KSADM/services
tempest.common.rest_client: ERROR: Request Body: <?xml version="1.0" encoding="UTF-8"?>
<service xmlns="http://docs.openstack.org/identity/api/ext/OS-KSADM/v1.0" type="type--9743" description="description-769556" name="service-722762"/>
tempest.common.rest_client: ERROR: Response Headers: {'date': 'Thu, 04 Oct 2012 16:42:05 GMT', 'vary': 'X-Auth-Token', 'content-length': '284', 'status': '500', 'content-type': 'application/xml'}
tempest.common.rest_client: ERROR: Response Body: {'message': "An unexpected error prevented the server from fulfilling your request. create_service() got an unexpected keyword argument 'service'", 'code': '500', 'title': 'Internal Server Error'}

There is a Keystone bug about this (it should return a 400 not a 500), but Tempest needs to be fixed as well. It seems that something may have changed recently regarding the Keystone API and the create_service() internal call.

Revision history for this message
Jay Pipes (jaypipes) wrote :
Revision history for this message
Jay Pipes (jaypipes) wrote :

Note this is ONLY seen in the XML format. JSON format works fine.

summary: tests.identity.admin.test_services:ServicesTestBase.test_create_get_delete_service()
- fails due to incorrect parameter
+ fails due to incorrect parameter (XML only)
Revision history for this message
Jay Pipes (jaypipes) wrote :

Marking just the XML test skipped until LP #1061738 is resolved.

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/14049

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

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

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/14066
Committed: http://github.com/openstack/tempest/commit/444c3e64ed1347adcfc4204c1e585e2d91022722
Submitter: Jenkins
Branch: master

commit 444c3e64ed1347adcfc4204c1e585e2d91022722
Author: Jay Pipes <email address hidden>
Date: Thu Oct 4 19:26:35 2012 -0400

    Refactor list servers negative test and fix remaining gate bugs

    test_list_servers_negative had so many issues with it, it's hard
    to list them here... but at least the following has been fixed:

    * No longer will the test flatly delete all instances the tenant
      and alt tenant have during every test method run (!!)
    * Fixtures -- 2 active and 1 deleted server -- are created in the
      class' setUpClass() method instead of having test methods
      constantly re-launch instances -- this reduces the test time
      down about 3 minutes
    * Removed all non-negative tests
    * Removed all tests that were skipped due to bugs that have been
    * Skip the XML create_service() test for Identity Admin tests
      because of the upstream Keystone bug 1061738

    This patch also includes patches for bug fixes that Sean Gallagher
    contributed but needed to be rebased into this one in order
    to get the full gate to pass.

    Change-Id: I0b38cf045520d93031c7916dae2479daf69d0411
    fixes: LP bug #1059802
    fixes: LP bug #1060373
    fixes: LP bug #1061740
    fixes: LP bug #1061167

Changed in tempest:
status: In Progress → Fix Released
Sean Dague (sdague)
Changed in tempest:
milestone: none → havana-3
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.