Comment 2 for bug 1721837

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

commit e88d3b2c8205a4865d64419623de10d9159eeea2
Author: John Griffith <email address hidden>
Date: Fri Oct 6 17:56:52 2017 +0000

    Fix migration 112 to use live_data_migration API

    In the 112 db migration I was being lazy and generating and
    updating the newly added UUID column. I also didn't update
    the Service object (left that for the follow on patch).

    Turns out we're going to want/need the online_data_migration
    pieces for some follow up work and we might as well do this at
    least a little bit more efficiently/correctly now.

    This patch modifies the db migration to NOT try and populate
    the newly added UUID fields in the Service table, and it
    updates the Service object properly including adding the gen UUID
    components to service.create()

    This also fixes up what we had in place for the online_data_migration
    code in cmd/manage.py; note that we had a framework there/started but
    it wasn't being used up to this point.

    Change-Id: I6696a15cd2c8fbf851a59b8d6d60ae1981bb1b89
    Closes-Bug: #1721837