]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Remove redundant db.volume_update() in volume manager create_volume()
authorZhiteng Huang <zhiteng.huang@intel.com>
Tue, 20 Nov 2012 17:02:37 +0000 (01:02 +0800)
committerZhiteng Huang <zhiteng.huang@intel.com>
Tue, 20 Nov 2012 17:55:12 +0000 (01:55 +0800)
commit6da9686ca64ef2a559d7120910d15cb495bc3b50
tree2bc24d0f39c9f269067616cb44bccbae84a0bcac
parent55c09da1b0ff4e2a147b07fcfa986a76815c7703
Remove redundant db.volume_update() in volume manager create_volume()

Since scheduler has already updated db to set the 'host' field and
recorded 'scheduled_at' time, it's not necessary to do db.volume_update()
again in volume manager create_volume() method.  Also let volume api
to update db when creating volume from snapshot while snapshot_same_host
is true, thus the behavior is consistent (updating db when placement
decision is made).

Change-Id: I43aaee8e2a5d0ceadb98bd608cc147467488349e
cinder/tests/test_volume.py
cinder/volume/api.py
cinder/volume/manager.py