delta/openstack/nova.git/nova/objects/block_device.py, branch 23.2.2 opendev.org: openstack/nova.git fup: Merge duplicate volume attachment checks 2021-02-04T14:05:31+00:00 Lee Yarwood lyarwood@redhat.com 2021-02-01T11:35:23+00:00 81b689e5bbae446e0b9da7ad604640aa7abe47e4 This is a follow up to I2881d77d52bcbde9f3ac6a6ddfb4a22a9bd45c8a where it was noted that we could merge these checks into one using a single db query for all active volume attachments for a given volume. This requires changes to the BlockDeviceMappingList object and was thus left as a separate change to allow the original to be backported. Change-Id: I859be652eb8fff591899a223193bd9efc8830cf3 
This is a follow up to I2881d77d52bcbde9f3ac6a6ddfb4a22a9bd45c8a where
it was noted that we could merge these checks into one using a single db
query for all active volume attachments for a given volume. This
requires changes to the BlockDeviceMappingList object and was thus left
as a separate change to allow the original to be backported.

Change-Id: I859be652eb8fff591899a223193bd9efc8830cf3
Remove 'bdm_(update_or_create|destroy)_at_top' 2019-04-16T17:26:17+00:00 Stephen Finucane sfinucan@redhat.com 2019-04-04T15:16:47+00:00 25545d7f554d406062dee260b3231be771014606 These were being called within the 'BlockDeviceMapping' object. Remove them in their entirety, given that nothing should be calling them now. Note that this effectively breaks cells v1 but that shouldn't matter, given that it's no longer possible to use the thing. Part of blueprint remove-cells-v1 Change-Id: I13fb0575317015128853ac093bccffe233f3c79f Signed-off-by: Stephen Finucane <sfinucan@redhat.com> 
These were being called within the 'BlockDeviceMapping' object. Remove
them in their entirety, given that nothing should be calling them now.
Note that this effectively breaks cells v1 but that shouldn't matter,
given that it's no longer possible to use the thing.

Part of blueprint remove-cells-v1

Change-Id: I13fb0575317015128853ac093bccffe233f3c79f
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
Add volume_type field to BlockDeviceMapping object 2018-09-28T07:29:24+00:00 zhangbailin zhangbailin@inspur.com 2018-09-24T06:13:52+00:00 f180025669257ed209959d5a95ddb3c0efac0165 Add ``volume_type`` field to BlockDeviceMapping object to be used in new instance block_device_mapping_v2.volume_type API extension. Implements: blueprint boot-instance-specific-storage-backend Change-Id: I66ff84585642ff8d40ecfefef3342349eb49f83d 
Add ``volume_type`` field to BlockDeviceMapping object to be used
in new instance block_device_mapping_v2.volume_type API extension.

Implements: blueprint boot-instance-specific-storage-backend
Change-Id: I66ff84585642ff8d40ecfefef3342349eb49f83d
Use nova.db.api directly 2018-07-10T14:56:27+00:00 Chris Dent cdent@anticdent.org 2018-02-11T21:58:04+00:00 def4b17934a3b2cf783d0177d6a9632916dfd10f nova/db/__init__.py was importing * from nova.db.api. This meant that any time any code anywhere within the nova.db package was imported then nova.db.api was too, leading to a cascade of imports that may not have been desired. Also, in general, code in __init__.py is a pain. Therefore, this change adjusts code that so that either: * nova.db.api is used directly * nova.db.api is imported as 'db' In either case, the functionality remains the same. The primary goal of this change was to make it possible to import the model files without having to import the db api. Moving the model files to a different place in the directory hierarchy was considered, but given that "code in __init__.py is a pain" this mode was chosen. This looks like a very large change, but it is essentially adjusting package names, many in mocks. Change-Id: Ic1fd7c87ceda05eeb96735da2a415ef37060bb1a 
nova/db/__init__.py was importing * from nova.db.api. This meant that
any time any code anywhere within the nova.db package was imported
then nova.db.api was too, leading to a cascade of imports that may
not have been desired. Also, in general, code in __init__.py is a pain.

Therefore, this change adjusts code that so that either:

* nova.db.api is used directly
* nova.db.api is imported as 'db'

In either case, the functionality remains the same.

The primary goal of this change was to make it possible to import the
model files without having to import the db api. Moving the model files
to a different place in the directory hierarchy was considered, but
given that "code in __init__.py is a pain" this mode was chosen.

This looks like a very large change, but it is essentially adjusting
package names, many in mocks.

Change-Id: Ic1fd7c87ceda05eeb96735da2a415ef37060bb1a
Add an online migration for BDM.uuid 2018-01-02T17:29:36+00:00 Matthew Booth mbooth@redhat.com 2017-12-02T05:42:52+00:00 0c441e636ba9d287909584b6ddf15eab5d479f0e Part of bp local-disk-serial-numbers Change-Id: I4b33751b6793f60c6f2703c379c36387c49d866d 
Part of bp local-disk-serial-numbers

Change-Id: I4b33751b6793f60c6f2703c379c36387c49d866d
Make BlockDeviceMapping object support uuid 2017-12-17T16:47:30+00:00 Dan Smith dansmith@redhat.com 2015-11-06T19:30:15+00:00 8df136dab5861caa6289cc5a254e8b401736e079 Part of bp local-disk-serial-numbers Co-Authored-By: Lee Yarwood <lyarwood@redhat.com> Co-Authored-By: Matthew Booth <mbooth@redhat.com> Partial-Bug: #1489581 Change-Id: I3b25debb0bcfd4e211734307c8d363f2b5dbc655 
Part of bp local-disk-serial-numbers

Co-Authored-By: Lee Yarwood <lyarwood@redhat.com>
Co-Authored-By: Matthew Booth <mbooth@redhat.com>
Partial-Bug: #1489581
Change-Id: I3b25debb0bcfd4e211734307c8d363f2b5dbc655
Fix AttributeError in BlockDeviceMapping.obj_load_attr 2017-10-25T14:09:54+00:00 Matt Riedemann mriedem.os@gmail.com 2017-10-24T20:45:57+00:00 1ca191fcc4d809c991c23dedc951bbe7206edf1d The BDM has no uuid attribute so the debug message in here would result in an AttributeError. This has been around since the creation of this object, and the debug log message was probably copied from the Instance object. This was only exposed in Pike when this code started lazy-loading the instance field: I1dc54a38f02bb48921bcbc4c2fdcc2c946e783c1 So this change fixes that bug and adds tests for obj_load_attr. Change-Id: I8b55227b1530a76c2f396c035384abd89237d936 Closes-Bug: #1726871 
The BDM has no uuid attribute so the debug message in here
would result in an AttributeError. This has been around since
the creation of this object, and the debug log message was
probably copied from the Instance object.

This was only exposed in Pike when this code started
lazy-loading the instance field:

  I1dc54a38f02bb48921bcbc4c2fdcc2c946e783c1

So this change fixes that bug and adds tests for obj_load_attr.

Change-Id: I8b55227b1530a76c2f396c035384abd89237d936
Closes-Bug: #1726871
Remove translation of log messages 2017-06-21T06:01:19+00:00 Ngo Quoc Cuong cuongnq@vn.fujitsu.com 2017-05-23T01:06:43+00:00 5a769f2a01d67f516c291e7900a5c257063bdfbe The i18n team has decided not to translate the logs because it seems like it not very useful; operators prefer to have them in English so that they can search for those strings on the internet. Partially fix on nova/keymgr, nova/network, nova/notifications, nova/objects and nova/pci other paths will be fixed on next commits Change-Id: Ie3a83fef0dc689b9d37ac43e047ce5d48f567adc 
The i18n team has decided not to translate the logs because it
seems like it not very useful; operators prefer to have them in
English so that they can search for those strings on the internet.

Partially fix on nova/keymgr, nova/network, nova/notifications,
nova/objects and nova/pci other paths will be fixed on next commits

Change-Id: Ie3a83fef0dc689b9d37ac43e047ce5d48f567adc
objects: Add attachment_id to BlockDeviceMapping 2017-03-16T20:35:01+00:00 Lee Yarwood lyarwood@redhat.com 2017-02-23T20:22:24+00:00 d8c532d538e6d808afc2debf455732800d7d830b This change introduces attachment_id into the BlockDeviceMapping Object as required by the new cinderv3 attachment API introduced in Ocata : Ie15233c99d91de67279b56d27a5508c5ea98d769 Implements: blueprint cinder-new-attach-apis Change-Id: Ieb43099a31689e1b509cbffb9aeb7cc9948f1a18 
This change introduces attachment_id into the BlockDeviceMapping Object
as required by the new cinderv3 attachment API introduced in Ocata :

Ie15233c99d91de67279b56d27a5508c5ea98d769

Implements: blueprint cinder-new-attach-apis
Change-Id: Ieb43099a31689e1b509cbffb9aeb7cc9948f1a18
Add tag column to vifs and bdm 2016-04-01T15:46:54+00:00 Artom Lifshitz alifshit@redhat.com 2016-01-05T22:57:53+00:00 9ba07fdf9feb5881a4b050f23066573f5d8753c8 This patch series adds the ability for a user to specify a tag to be applied to a network interface or a disk when booting an instance. This tag is then exposed through the metadata API. For example, a user boots a VM with two network interfaces. One is connected to a private network, the other to the public Internet. There is currently no direct way to tell which interface is which. Specifying tags allows the user to distinguish between the two interfaces. This patch contains the database migration to add a 'tag' column to the virtual_interfaces and block_device_mapping tables as well as the corresponding change to the VirtualInterface and BlockDeviceMapping objects. Implements: blueprint virt-device-role-tagging Co-authored-by: Vladik Romanovsky <vromanso@redhat.com> Change-Id: Ic8be3de4e970116772f9b6ce01c55e26b829e6cb 
This patch series adds the ability for a user to specify a tag to be
applied to a network interface or a disk when booting an instance.
This tag is then exposed through the metadata API. For example, a user
boots a VM with two network interfaces. One is connected to a private
network, the other to the public Internet. There is currently no
direct way to tell which interface is which. Specifying tags allows
the user to distinguish between the two interfaces.

This patch contains the database migration to add a 'tag' column to
the virtual_interfaces and block_device_mapping tables as well as the
corresponding change to the VirtualInterface and BlockDeviceMapping
objects.

Implements: blueprint virt-device-role-tagging
Co-authored-by: Vladik Romanovsky <vromanso@redhat.com>
Change-Id: Ic8be3de4e970116772f9b6ce01c55e26b829e6cb