Incomplete mapping in cinder volume plugin

Bug #1578659 reported by Lei Zhang
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Searchlight
Fix Released
Medium
Annapoornima Koppad

Bug Description

In cinder volume plugin, we have a field named 'migration_status' appears in admin only fields and facets, but it's not include in volume mapping.

https://github.com/openstack/searchlight/blob/master/searchlight/elasticsearch/plugins/cinder/volumes.py#L35
https://github.com/openstack/searchlight/blob/master/searchlight/elasticsearch/plugins/cinder/volumes.py#L125

Lei Zhang (lei-a-zhang)
summary: - Inconsistent mapping in cinder volume plugin
+ Incomplete mapping in cinder volume plugin
Revision history for this message
Steve McLellan (sjmc7) wrote :

These fields do exist in API responses so just need mapping correctly (as not_analyzed fields).

Changed in searchlight:
importance: Undecided → Medium
tags: added: low-hanging-fruit
Changed in searchlight:
assignee: nobody → Annapoornima Koppad (annakoppad)
Changed in searchlight:
status: New → Confirmed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to searchlight (master)

Reviewed: https://review.openstack.org/372953
Committed: https://git.openstack.org/cgit/openstack/searchlight/commit/?id=88fa0167bd9c45410080e66eb97d56d3f899d1d2
Submitter: Jenkins
Branch: master

commit 88fa0167bd9c45410080e66eb97d56d3f899d1d2
Author: Rick Aulino <email address hidden>
Date: Tue Sep 20 00:49:37 2016 -0600

    Cinder mapping incorrect.

    The Cinder volume API field 'migration_status' is
    missing from it's mapping. Add it.

    Closes-Bug #1578659

    Change-Id: I5f15fc96c53f7b204329e5fbd6b7312959d53509

Changed in searchlight:
status: Confirmed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to searchlight (stable/newton)

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

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

Reviewed: https://review.openstack.org/375115
Committed: https://git.openstack.org/cgit/openstack/searchlight/commit/?id=6550b193e4033d4d968ca6980ad2dda94e85146d
Submitter: Jenkins
Branch: stable/newton

commit 6550b193e4033d4d968ca6980ad2dda94e85146d
Author: Rick Aulino <email address hidden>
Date: Tue Sep 20 00:49:37 2016 -0600

    Cinder mapping incorrect.

    The Cinder volume API field 'migration_status' is
    missing from it's mapping. Add it.

    Closes-Bug #1578659

    Change-Id: I5f15fc96c53f7b204329e5fbd6b7312959d53509
    (cherry picked from commit 88fa0167bd9c45410080e66eb97d56d3f899d1d2)

tags: added: in-stable-newton
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.