[Tempest] [SSL] Tempest test "test_volume_list_pagination" and "test_volume_list_details_pagination" FAIL in MOS with TLS

Bug #1490266 reported by Oleksiy Butenko
20
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Status tracked in 10.0.x
10.0.x
Confirmed
High
Alexey Deryugin
9.x
In Progress
High
Yuriy Nesenenko

Bug Description

Tempest test "test_volume_list_pagination" and "test_volume_list_details_pagination" FAIL in MOS with TLS

Steps:
1. Deploy env with TLS (Controller - 1, Compute + Ceph - 1, Neutron with VLAN segmentation)
2. Install mos-tempest-runner on the Fuel master node (https://github.com/Mirantis/mos-tempest-runner)
3. run_tests tempest.api.volume.v2.test_volumes_list

Expected result:
All tests passed.

Actual result:
tempest.api.volume.v2.test_volumes_list.VolumesV2ListTestJSON.test_volume_list_details_pagination[id-e9138a2c-f67b-4796-8efa-635c196d01de] - FAIL
tempest.api.volume.v2.test_volumes_list.VolumesV2ListTestJSON.test_volume_list_pagination[id-af55e775-8e4b-4feb-8719-215c43b0238c] - FAIL

Tempest traceback:
http://paste.openstack.org/show/433517/

{"build_id": "2015-08-29_21-15-00", "build_number": "253", "release_versions": {"2015.1.0-7.0": {"VERSION": {"build_id": "2015-08-29_21-15-00", "build_number": "253", "api": "1.0", "fuel-library_sha": "bc04a7092d92400c79e6ea6ede25e7b67c6a6355", "nailgun_sha": "3189ccfb8c1dac888e351f535b03bdbc9d392406", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d7027952870a35db8dc52f185bb1158cdd3d1ebd", "openstack_version": "2015.1.0-7.0", "fuel-agent_sha": "1e8f38bbb864ed99aa8fe862b6367e82afec3263", "production": "docker", "python-fuelclient_sha": "9643fa07f1290071511066804f962f62fe27b512", "astute_sha": "53c86cba593ddbac776ce5a3360240274c20738c", "fuel-ostf_sha": "83048d68609854324ceeaf04242e68d658cfb55d", "release": "7.0", "fuelmain_sha": "0e54d68392b359bc122e5bbba9249c729eeaf579"}}}, "auth_required": true, "api": "1.0", "fuel-library_sha": "bc04a7092d92400c79e6ea6ede25e7b67c6a6355", "nailgun_sha": "3189ccfb8c1dac888e351f535b03bdbc9d392406", "feature_groups": ["mirantis"], "fuel-nailgun-agent_sha": "d7027952870a35db8dc52f185bb1158cdd3d1ebd", "openstack_version": "2015.1.0-7.0", "fuel-agent_sha": "1e8f38bbb864ed99aa8fe862b6367e82afec3263", "production": "docker", "python-fuelclient_sha": "9643fa07f1290071511066804f962f62fe27b512", "astute_sha": "53c86cba593ddbac776ce5a3360240274c20738c", "fuel-ostf_sha": "83048d68609854324ceeaf04242e68d658cfb55d", "release": "7.0", "fuelmain_sha": "0e54d68392b359bc122e5bbba9249c729eeaf579"}

Changed in mos:
importance: Undecided → Medium
tags: added: cinder tempest
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

Looks like Tempest issue, we need to check configuration and fix it on the Tempest side.

Yaroslav, could you please take a look?

Changed in mos:
milestone: none → 7.0
status: New → Confirmed
tags: added: non-release
no longer affects: mos/8.0.x
no longer affects: mos/7.0.x
Changed in mos:
assignee: nobody → Yaroslav Lobankov (ylobankov)
Revision history for this message
Yaroslav Lobankov (ylobankov) wrote :

Moving to 8.0 because it's a medium issue.

Changed in mos:
milestone: 7.0 → 8.0
Changed in mos:
assignee: Yaroslav Lobankov (ylobankov) → Oleksiy Butenko (obutenko)
Revision history for this message
Roman Bogorodskiy (novel) wrote :

Actually, I'm not quite sure that's a Tempest issue. If you forget about Tempest for a second a do something like:

$ cinder --debug --os-volume-api-version 2 list --limit 1

You'll see that even if it makes requests through HTTPS, all the hrefs are using HTTP. I guess it happens because the load balancer talks to the actual cinder-api instances through HTTP, not HTTPS. However, I think that this behavior is not quite correct and it should use HTTPS in hrefs, so Tempest tests are failing for reason.

However, I'm not sure if that's a bug in cinder code or configuration. Most likely it can be solved by defining 'osapi_volume_base_URL' though.

Revision history for this message
Roman Bogorodskiy (novel) wrote :

FYI, in my environment I reconfigured cinder to set 'osapi_volume_base_URL' in cinder.conf's DEFAULT section to match the value of the volume API endpoint. After that cinder started to return proper refs. That's probably a thing that should be configured automatically at deployment time.

Revision history for this message
Oleksiy Butenko (obutenko) wrote :

Roman, thank you for information!

Changed in mos:
assignee: Oleksiy Butenko (obutenko) → MOS Cinder (mos-cinder)
Ivan Kolodyazhny (e0ne)
Changed in mos:
status: Confirmed → Won't Fix
Revision history for this message
Timur Nurlygayanov (tnurlygayanov) wrote :

So, we have the workaround for the issue in MOS 8.0 and we can't fix this medium issue in MOS 8.0 already but we should fix it for MOS 9.0 release.

summary: - Tempest test "test_volume_list_pagination" and
+ [Tempest] [SSL] Tempest test "test_volume_list_pagination" and
"test_volume_list_details_pagination" FAIL in MOS with TLS
Changed in mos:
assignee: MOS Cinder (mos-cinder) → Alexey Deryugin (velovec)
tags: added: swarm-blocker
tags: added: area-cinder
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (9.0/mitaka)

Related fix proposed to branch: 9.0/mitaka
Change author: Yuriy Nesenenko <email address hidden>
Review: https://review.fuel-infra.org/19570

Revision history for this message
Dina Belova (dbelova) wrote :

Added move-to-10.0 tag due to the fact bug was transferred from 9.0 to 10.0

tags: added: move-to-10.0
Revision history for this message
Oleksiy Butenko (obutenko) wrote :

remove tag move-to-10.0, tempest, non-release. It's not tempest issue. And we need fix it in 9.0 - change to High

tags: removed: move-to-10.0 non-release tempest
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (9.0/mitaka)

Change abandoned by Yuriy Nesenenko <email address hidden> on branch: 9.0/mitaka
Review: https://review.fuel-infra.org/19570
Reason: wrong aprouch

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix proposed to openstack/cinder (9.0/mitaka)

Related fix proposed to branch: 9.0/mitaka
Change author: Yuriy Nesenenko <email address hidden>
Review: https://review.fuel-infra.org/20056

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Related fix merged to openstack/cinder (9.0/mitaka)

Reviewed: https://review.fuel-infra.org/20056
Submitter: Pkgs Jenkins <email address hidden>
Branch: 9.0/mitaka

Commit: 2ddf86a8fadaf359327a68d3c361db88f696b027
Author: Yuriy Nesenenko <email address hidden>
Date: Mon Apr 25 13:42:14 2016

Handle SSL termination proxies for version list

Cinder list with pagination contains wrong scheme
for 'next' link in case of SSL endpoints. This
patch fixes it and returns the correct scheme in
version URLs if service is behind an SSL termination
proxy.

Change-Id: If5aab9cc25a2e7c66a0bb13b5f7488a667b30309
Closes-Bug: #1569355
Related-Bug: #1490266

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.