Migration_status being used to hold target info

Bug #1706729 reported by Kendall Nelson on 2017-07-26
This bug affects 1 person
Affects Status Importance Assigned to Milestone

Bug Description

There are several places where migration_status is being used to store info about the migration, i.e. target: + vol_id. migration_status should only be holding string statuses: migrating, error, success, completing, none, etc.





This doesn't cause issues at the moment, but it needs to be fixed before the migration_status field can be made into an ovo enum. There are a few drivers that depend on this logic and so they will also need to be updated.

tags: added: nec
tags: added: drivers
description: updated
description: updated
description: updated
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

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

Changed in cinder:
assignee: nobody → Shunei Shiono (s-shiono)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/511737
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=a638f483fb41df4f4acc0b0f42d289ba711333e5
Submitter: Zuul
Branch: master

commit a638f483fb41df4f4acc0b0f42d289ba711333e5
Author: Shunei Shiono <email address hidden>
Date: Fri Oct 13 16:02:26 2017 +0900

    NEC driver: Never use target info in migration_status

    NEC driver uses target info in migration_status at volume creation.
    The target info will be deleted from migration_status in future
    cinder version. This patch change the driver not to use target info
    in migration_status.

    Change-Id: I04b188d1300979b3b4b070781c6c2105f638b649
    Partial-Bug: #1706729

Shunei Shiono (s-shiono) on 2017-10-31
Changed in cinder:
assignee: Shunei Shiono (s-shiono) → nobody
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers