[cinder] InstanceLocalityFilter doesn't work

Bug #1517438 reported by Ivan Kolodyazhny
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mirantis OpenStack
Fix Released
Medium
Yuriy Nesenenko

Bug Description

Condition [1] will be always true and CinderException will be raised. The root cause is here [2]: we check for client-side extensions instead of server-side one

[1] https://github.com/openstack/cinder/blob/master/cinder/scheduler/filters/instance_locality_filter.py#L96
[2] https://github.com/openstack/cinder/blob/master/cinder/compute/nova.py#L161

Upstream bug: https://bugs.launchpad.net/cinder/+bug/1517431

Tags: area-cinder
Ivan Kolodyazhny (e0ne)
Changed in mos:
status: New → Confirmed
importance: Undecided → Medium
assignee: nobody → Ivan Kolodyazhny (e0ne)
milestone: none → 8.0
Changed in mos:
assignee: Ivan Kolodyazhny (e0ne) → Yuriy Nesenenko (ynesenenko)
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/cinder (openstack-ci/fuel-8.0/liberty)

Fix proposed to branch: openstack-ci/fuel-8.0/liberty
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/14381

Changed in mos:
status: Confirmed → In Progress
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix merged to openstack/cinder (openstack-ci/fuel-8.0/liberty)

Reviewed: https://review.fuel-infra.org/14381
Submitter: Pkgs Jenkins <email address hidden>
Branch: openstack-ci/fuel-8.0/liberty

Commit: 088cae0afa4d3d73e08fef0df1b74e8ecb6e1b6d
Author: Ivan Kolodyazhny <email address hidden>
Date: Wed Dec 2 17:36:59 2015

Fix InstanceLocalityFilter scheduler filter

Commit I51254e20151fe1d6771897e64a8d0bdf2d674d03 introduces new error:
We use novaclient extensions instead of Nova API extensions. In such
case, InstanceLocalityFilter won't find Nova ExtendedServerAttributes
extension and will raise CinderException.

This patch fixes novaclient usage in a right way.

Change-Id: Idee7c82858559d93defba0af5d856771b62a624d
Closes-Bug: #1517438

Changed in mos:
status: In Progress → Fix Committed
tags: added: cinder
summary: - InstanceLocalityFilter doesn't work
+ [cinder] InstanceLocalityFilter doesn't work
tags: added: area-cinder
removed: cinder
Revision history for this message
Evgeny Sikachev (esikachev) wrote :

verified on 478 iso

Changed in mos:
status: Fix Committed → Fix Released
Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Fix proposed to openstack/cinder (9.0/mitaka)

Fix proposed to branch: 9.0/mitaka
Change author: Ivan Kolodyazhny <email address hidden>
Review: https://review.fuel-infra.org/18525

Revision history for this message
Fuel Devops McRobotson (fuel-devops-robot) wrote : Change abandoned on openstack/cinder (9.0/mitaka)

Change abandoned by Ivan Kolodyazhny <email address hidden> on branch: 9.0/mitaka
Review: https://review.fuel-infra.org/18525

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.