[10.0] [Tempest] Cinder volume details pagination tests are failed

Bug #1665361 reported by Yury Tregubov
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
High
Nikita Karpin

Bug Description

On mos10.0 two tests for volume details pagination are failed with following error:

Traceback (most recent call last):

  File "tempest/api/volume/v2/test_volumes_list.py", line 174, in test_volume_list_details_pagination

    self._test_pagination('volumes', ids=self.volume_id_list, detail=True)

  File "tempest/api/volume/v2/test_volumes_list.py", line 144, in _test_pagination

    self.assertEqual(next, response.response['content-location'])

  File "/root/.rally/tempest/for-deployment-f0d3250b-dd7b-40c6-b2fa-6ac07f758585/.venv/local/lib/python2.7/site-packages/testtools/testcase.py", line 411, in assertEqual

    self.assertThat(observed, matcher, message)

  File "/root/.rally/tempest/for-deployment-f0d3250b-dd7b-40c6-b2fa-6ac07f758585/.venv/local/lib/python2.7/site-packages/testtools/testcase.py", line 498, in assertThat

    raise mismatch_error

testtools.matchers._impl.MismatchError: !=:

reference = u'http://public.fuel.local:8776/v2/bb73d1eeecee4f8694012820b8d77417/volumes/detail?limit=1&marker=af6b17fc-2935-44a7-b27f-87680c5cf0fc'

actual = 'https://public.fuel.local:8776/v2/bb73d1eeecee4f8694012820b8d77417/volumes/detail?limit=1&marker=af6b17fc-2935-44a7-b27f-87680c5cf0fc'

So 'https' got instead of 'http'. The tests are failed only on envs with SSL.
Probably it is a problem of the tests. But there were no such fails with mos 9.2
Plase take a look.

Tempest logs are attached.

Diagnostic snapshot is: https://docs.google.com/a/mirantis.com/uc?id=0B1Crk-sAvGanRzRGeEp3VExLZXM&export=download&urp=https://drive.google.com/file/d/0B1Crk-sAvGanRzRGe&pli=1

Revision history for this message
Yury Tregubov (ytregubov) wrote :
Changed in fuel:
milestone: none → 10.1
importance: Undecided → High
Revision history for this message
Yury Tregubov (ytregubov) wrote :
Changed in fuel:
assignee: nobody → MOS Cinder (mos-cinder)
status: New → Confirmed
Ivan Kolodyazhny (e0ne)
Changed in fuel:
assignee: MOS Cinder (mos-cinder) → Ivan Kolodyazhny (e0ne)
Revision history for this message
Ivan Kolodyazhny (e0ne) wrote :

It's a configuration issue. We have to set 'enable_proxy_headers_parsing = true' in [oslo_middleware] section in cinder.conf

Changed in fuel:
assignee: Ivan Kolodyazhny (e0ne) → MOS Packaging Team (mos-packaging)
Nikita Karpin (mkarpin)
Changed in fuel:
assignee: MOS Packaging Team (mos-packaging) → Nikita Karpin (mkarpin)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (master)

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

Changed in fuel:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/435507
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=fcf9c5cdf98b11b91c1919b5167f740cd7276583
Submitter: Jenkins
Branch: master

commit fcf9c5cdf98b11b91c1919b5167f740cd7276583
Author: Mykyta Karpin <email address hidden>
Date: Fri Feb 17 18:02:49 2017 +0200

    Enable proxy headers parsing

    As cinder is placed behind the proxy we need to enable
    proxy headers parsing, so cinder is aware about about
    protocol used to connect to endpoint.

    Change-Id: If68de7d52617803b379ed564f07ed6c947315dbd
    Closes-Bug: #1665361

Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-library (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/435891

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/newton)

Reviewed: https://review.openstack.org/435891
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=4ce627bc0b2abb315b79689e99de80711f7c4972
Submitter: Jenkins
Branch: stable/newton

commit 4ce627bc0b2abb315b79689e99de80711f7c4972
Author: Mykyta Karpin <email address hidden>
Date: Fri Feb 17 18:02:49 2017 +0200

    Enable proxy headers parsing

    As cinder is placed behind the proxy we need to enable
    proxy headers parsing, so cinder is aware about about
    protocol used to connect to endpoint.

    Change-Id: If68de7d52617803b379ed564f07ed6c947315dbd
    Closes-Bug: #1665361
    (cherry picked from commit fcf9c5cdf98b11b91c1919b5167f740cd7276583)

tags: added: in-stable-newton
Revision history for this message
Yury Tregubov (ytregubov) wrote :

The affected tests are passed sine mos 10.0 #1405.

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-library 11.0.0.0rc1

This issue was fixed in the openstack/fuel-library 11.0.0.0rc1 release candidate.

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.