[tempest-lib] REST client can not parse 'list' in json response in any case

Bug #1539927 reported by Victor Ryzhenkin
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tempest
Fix Released
Medium
Ken'ichi Ohmichi

Bug Description

Using master devstack.

If response json contains only 'list' type object, REST client _parse_resp method will raise an exception
'AttributeError: 'list' object has no attribute 'keys''

The example of failed job here: http://logs.openstack.org/84/274284/3/check/gate-tempest-dsvm-murano-api/fcf8a33/testr_results.html.gz

Changed in tempest:
status: New → Confirmed
assignee: nobody → Victor Ryzhenkin (vryzhenkin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tempest-lib (master)

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

Changed in tempest:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on tempest-lib (master)

Change abandoned by Victor Ryzhenkin (<email address hidden>) on branch: master
Review: https://review.openstack.org/274356
Reason: Will reupload the change to corresponding repo.

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

Changed in tempest:
assignee: Victor Ryzhenkin (vryzhenkin) → Ken'ichi Ohmichi (oomichi)
Changed in tempest:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tempest (master)

Reviewed: https://review.openstack.org/461124
Committed: https://git.openstack.org/cgit/openstack/tempest/commit/?id=69a8edc1ac7eb9260094f972fe1677205c9a8f0e
Submitter: Jenkins
Branch: master

commit 69a8edc1ac7eb9260094f972fe1677205c9a8f0e
Author: Ken'ichi Ohmichi <email address hidden>
Date: Fri Apr 28 11:41:20 2017 -0700

    Prevent error in _parse_resp when nullable list

    - Add if contiditon to check, that body object hasattr 'keys'
    - Add unit test for this case

    NOTE: The original patch is Ifd063ed2329ec14c123a128b9520babb54ece69c
          and this patch moves it to Tempest from deprecated tempest-lib.
          In addition, this patch moves the test path because the path was
          not match to the corresponding module.
          One more thing is that this issue happened on Murano side and
          they needed to have some workaround on their side. It would be
          nice to fix this root issue on Tempest side.

    Co-Authored-By: Victor Ryzhenkin <email address hidden>
    Closes-Bug: #1539927
    Change-Id: I46cee5f3910ec9dfe383c6466f711e4a9554bb60

Changed in tempest:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tempest 16.1.0

This issue was fixed in the openstack/tempest 16.1.0 release.

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.